[PATCH] D73629: [analyzer] vfork checker: allow execve after vfork

Jan Včelák via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 30 03:00:22 PST 2020


janvcelak added a comment.

In D73629#1847750 <https://reviews.llvm.org/D73629#1847750>, @NoQ wrote:

> We should add a test for this (cf. `test/Analysis/vfork.c`).


There is already a test to check if the whitelist works by making sure that call to `execl` doesn't generate the warning. I think there is no point adding `execve` into the tests unless we want to add a test for each of the functions in the list. However, let me know if you want it added.


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

https://reviews.llvm.org/D73629





More information about the cfe-commits mailing list