[all-commits] [llvm/llvm-project] 56eaf8: [analyzer] Done some changes to detect Uninitializ...

Shivam via All-commits all-commits at lists.llvm.org
Thu Mar 3 10:51:24 PST 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 56eaf869be27585bff7320505dfad32b5b3b6189
      https://github.com/llvm/llvm-project/commit/56eaf869be27585bff7320505dfad32b5b3b6189
  Author: Shivam <75530356+phyBrackets at users.noreply.github.com>
  Date:   2022-03-04 (Fri, 04 Mar 2022)

  Changed paths:
    M clang/lib/StaticAnalyzer/Checkers/CStringChecker.cpp

  Log Message:
  -----------
  [analyzer] Done some changes to detect Uninitialized read by the char array manipulation functions

Few weeks back I was experimenting with reading the uninitialized values from src , which is actually a bug but the CSA seems to give up at that point . I was curious about that and I pinged @steakhal on the discord and according to him this seems to be a genuine issue and needs to be fix. So I goes with fixing this bug and thanks to @steakhal who help me creating this patch. This feature seems to break some tests but this was the genuine problem and the broken tests also needs to fix in certain manner. I add a test but yeah we need more tests,I'll try to add more tests.Thanks

Reviewed By: steakhal, NoQ

Differential Revision: https://reviews.llvm.org/D120489




More information about the All-commits mailing list