Fix coverity #29352

SuperTux · Feb 23, 2015 · 8beeeba · 8beeeba
1 parent 0ebb403
commit 8beeeba
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 7 deletions.
diff --git a/src/scripting/serialize.cpp b/src/scripting/serialize.cpp
@@ -99,8 +99,8 @@ void save_squirrel_table(HSQUIRRELVM vm, SQInteger table_idx, Writer& writer)
       }
       case OT_BOOL: {
         SQBool val;
-        sq_getbool(vm, -1, &val);
-        writer.write(key, val == SQTrue);
+        if(SQ_SUCCEEDED(sq_getbool(vm, -1, &val)))
+          writer.write(key, val == SQTrue);
         break;
       }
       case OT_STRING: {

diff --git a/src/scripting/squirrel_util.cpp b/src/scripting/squirrel_util.cpp
@@ -37,11 +37,12 @@ std::string squirrel2string(HSQUIRRELVM v, SQInteger i)
       break;
     case OT_BOOL: {
       SQBool p;
-      sq_getbool(v, i, &p);
-      if (p)
-        os << "true";
-      else
-        os << "false";
+      if SQ_SUCCEEDED(sq_getbool(v, i, &p)) {
+        if (p)
+          os << "true";
+        else
+          os << "false";
+      }
       break;
     }
     case OT_INTEGER: {