[PATCH] D120489: [analyzer] Done some changes to detect Uninitialized read by the char array manipulation functions

Shivam Rajput via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Feb 28 11:27:17 PST 2022


phyBrackets added inline comments.


================
Comment at: clang/test/Analysis/bstring.c:310
 
-  clang_analyzer_eval(p == &dst[4]); // expected-warning{{TRUE}}
+  clang_analyzer_eval(p == &dst[4]); // no-warning (above is fatal)
 }
----------------
NoQ wrote:
> Hmm, given that your change suppresses some existing tests, maybe disable your checker on this file and copy the updated tests to a new file that would have the checker enabled?
I don't know how will it affect that but still that's an exception and from these exceptions test cases, we are updated about how it gonna fail on some tests cases and try to work on those exceptions.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D120489/new/

https://reviews.llvm.org/D120489



More information about the cfe-commits mailing list