[PATCH] D118987: [analyzer] Add failing test case demonstrating buggy taint propagation

2022-02-24 Thread Balázs Benics via Phabricator via cfe-commits
steakhal marked an inline comment as done. steakhal added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/GenericTaintChecker.cpp:782 +if (PropDstArgs.contains(I)) { + LLVM_DEBUG(llvm::dbgs() << "PreCall<"; Call.dump(llvm::dbgs()); +

[PATCH] D118987: [analyzer] Add failing test case demonstrating buggy taint propagation

2022-02-24 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added inline comments. Comment at: clang/lib/StaticAnalyzer/Checkers/GenericTaintChecker.cpp:782 +if (PropDstArgs.contains(I)) { + LLVM_DEBUG(llvm::dbgs() << "PreCall<"; Call.dump(llvm::dbgs()); + llvm::dbgs() There

[PATCH] D118987: [analyzer] Add failing test case demonstrating buggy taint propagation

2022-02-23 Thread Balázs Benics via Phabricator via cfe-commits
steakhal closed this revision. steakhal added a comment. Committed as fa0a80e017ebd58a71bdb4e4493bb022f80fe791 . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118987/new/

[PATCH] D118987: [analyzer] Add failing test case demonstrating buggy taint propagation

2022-02-23 Thread Balázs Benics via Phabricator via cfe-commits
steakhal added a comment. In D118987#3332940 , @jrtc27 wrote: > In D118987#3319697 , @steakhal > wrote: > >> It seems like the `clang-ve-ninja` doesn't really want to accept any patches >> from me :D >> I hope

[PATCH] D118987: [analyzer] Add failing test case demonstrating buggy taint propagation

2022-02-18 Thread Jessica Clarke via Phabricator via cfe-commits
jrtc27 added a comment. In D118987#3319697 , @steakhal wrote: > It seems like the `clang-ve-ninja` doesn't really want to accept any patches > from me :D > I hope it's not personal. Let's be friends bot, please. > > Link to the breakage:

[PATCH] D118987: [analyzer] Add failing test case demonstrating buggy taint propagation

2022-02-14 Thread Balázs Benics via Phabricator via cfe-commits
steakhal reopened this revision. steakhal added a subscriber: simoll. steakhal added a comment. This revision is now accepted and ready to land. It seems like the `clang-ve-ninja` doesn't really want to accept any patches from me :D I hope it's not personal. Let's be friends bot, please. I'm

[PATCH] D118987: [analyzer] Add failing test case demonstrating buggy taint propagation

2022-02-14 Thread Balázs Benics via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG744745ae195f: [analyzer] Add failing test case demonstrating buggy taint propagation (authored by steakhal). Herald added a project: clang. Herald