Avoid breaking on dead launch options

Bug: 28463187

Launch options array[2-4] are not initialized by the support library
in the current SDK.

They may have arbitrary values that invalide the checks that this CL
is removing, and cause a kernel launch to bail out unnecessarily.

This CL removes those checks, since the array.*Start and array.*End
fields in the launch options are not used anywhere in the current
runtime.

Change-Id: Ib8a2e551619f770b402f569acbde635ee047675a

1 files changed, 0 insertions, 9 deletions

diff --git a/rsScriptC.cpp b/rsScriptC.cpp
index d2d5b1fd..eaef8490 100644
--- a/rsScriptC.cpp
+++ b/rsScriptC.cpp
@@ -183,15 +183,6 @@ void ScriptC::runForEach(Context *rsc,
                          const void * usr,
                          size_t usrBytes,
                          const RsScriptCall *sc) {
-    // Make a copy of RsScriptCall and zero out extra fields that are absent
-    // in API levels below 23.
-    RsScriptCall sc_copy;
-    if (sc != nullptr && getApiLevel() < 23) {
-        memset(&sc_copy, 0, sizeof(sc_copy));
-        memcpy(&sc_copy, sc, 7*4);
-        sc = &sc_copy;
-    }
-
     if (slot >= mHal.info.exportedForEachCount) {
         rsc->setError(RS_ERROR_BAD_SCRIPT,
                       "The forEach kernel index is out of bounds");