NoQ added a comment.

Note that the uncontrolled recursion checker will not be able to test this 
overwrite problem, because overwriting the argument region spoils the stack 
frame for it.



================
Comment at: include/clang/StaticAnalyzer/Core/PathSensitive/ProgramState.h:743
+    const Loc ArgLoc = getLValue(ArgDecl, SFC);
+    return getSVal(ArgLoc);
+  } else {
----------------
Hmm, this should work:
```
~ 743     StoreManager &storeMgr = stateMgr->getStoreManager();
+ 744     return storeMgr.getBinding(storeMgr.getInitialStore(SFC).getStore(),
+ 745                                ArgLoc);
```


https://reviews.llvm.org/D26760



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to