commit 4efe3d38ee1d7b76c93a47ce6218b1f708fd4d9a
author Jason Sams <jsams@google.com> Wed Mar 18 18:28:06 2015 -0700
committer Jason Sams <jsams@google.com> Wed Mar 18 18:28:06 2015 -0700
tree c5bd8813b46f328f59c56793548efc0f49cb970f
parent c3851d58e6f0a97a3f5239d78515acd2be47b507

Add fatal error check

Prevent launching additional commands if the process is dying
This increases the chance the developer will get the error message
before we segfault somewhere.

Change-Id: I575906b22364c0d03859140570ca29bf8f336c01

rsScript.cpp

diff --git a/rsScript.cpp b/rsScript.cpp
index b89c96e..cb611af 100644
--- a/rsScript.cpp
+++ b/rsScript.cpp
@@ -48,6 +48,8 @@
         return;
     }
 
+    if (mRSC->hadFatalError()) return;
+
     mSlots[slot].set(a);
     mHasObjectSlots = true;
     mRSC->mHal.funcs.script.setGlobalBind(mRSC, this, slot, a);
@@ -59,6 +61,8 @@
         ALOGE("Script::setVar unable to set allocation, invalid slot index");
         return;
     }
+    if (mRSC->hadFatalError()) return;
+
     mRSC->mHal.funcs.script.setGlobalVar(mRSC, this, slot, (void *)val, len);
 }
 
@@ -69,6 +73,8 @@
               "%u >= %zu", slot, mHal.info.exportedVariableCount);
         return;
     }
+    if (mRSC->hadFatalError()) return;
+
     mRSC->mHal.funcs.script.getGlobalVar(mRSC, this, slot, (void *)val, len);
 }
 
@@ -79,6 +85,8 @@
               "%u >= %zu", slot, mHal.info.exportedVariableCount);
         return;
     }
+    if (mRSC->hadFatalError()) return;
+
     mRSC->mHal.funcs.script.setGlobalVarWithElemDims(mRSC, this, slot,
             (void *)val, len, e, dims, dimLen);
 }
@@ -90,6 +98,8 @@
               "%u >= %zu", slot, mHal.info.exportedVariableCount);
         return;
     }
+    if (mRSC->hadFatalError()) return;
+
     mHasObjectSlots = true;
     mRSC->mHal.funcs.script.setGlobalObj(mRSC, this, slot, val);
 }