[PATCH] D120236: Add more sources to Taint analysis

2022-02-21 Thread Balázs Benics via Phabricator via cfe-commits
steakhal added a comment. Also great to see that the docs are in-sync to the code, which is a great plus! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120236/new/ https://reviews.llvm.org/D120236 ___ cf

[PATCH] D120236: Add more sources to Taint analysis

2022-02-21 Thread Balázs Benics via Phabricator via cfe-commits
steakhal added a comment. Sorry for the wall-of-nits. Overall, looks great. Comment at: clang/docs/analyzer/checkers.rst:2358 Default sources defined by ``GenericTaintChecker``: -``fdopen``, ``fopen``, ``freopen``, ``getch``, ``getchar``, ``getchar_unlocked``, ``gets``, ``sca

[PATCH] D120236: Add more sources to Taint analysis

2022-02-21 Thread Endre Fülöp via Phabricator via cfe-commits
gamesh411 created this revision. gamesh411 added a reviewer: steakhal. Herald added subscribers: martong, Szelethus, dkrupp. Herald added a reviewer: Szelethus. gamesh411 requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Add more functions as