Changeset 143411 in webkit for trunk/Source/JavaScriptCore/interpreter/Interpreter.cpp

Timestamp:

Feb 19, 2013, 4:47:52 PM (12 years ago)

Author:

[email protected]

Message:

Silence some analyzer warnings
​https://bugs.webkit.org/show_bug.cgi?id=110281

Reviewed by Mark Hahnenberg.

The static analyzer believes that callerCodeBlock can be null,
based on other code performing null tests. This should not
ever be the case, but we'll add RELEASE_ASSERTs to make it
obvious if we're ever wrong.

• interpreter/Interpreter.cpp:

(JSC::getCallerInfo):

File:

• trunk/Source/JavaScriptCore/interpreter/Interpreter.cpp (modified) (1 diff)

trunk/Source/JavaScriptCore/interpreter/Interpreter.cpp

@@ -652 +652 @@
         } else
     #endif
+        {
+            RELEASE_ASSERT(callerCodeBlock);
             bytecodeOffset = callerCodeBlock->bytecodeOffset(callerFrame, callFrame->returnPC());
-#endif
-    }
-
+        }
+#endif
+    }
+
+    RELEASE_ASSERT(callerCodeBlock);
     lineNumber = callerCodeBlock->lineNumberForBytecodeOffset(bytecodeOffset);
     return callerFrame;