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

[Jan Včelák via Phabricator via cfe-commits]

janvcelak added inline comments.


================
Comment at: clang/lib/StaticAnalyzer/Checkers/VforkChecker.cpp:109
       "execvpe",
+      "execve",
       nullptr
----------------
xazax.hun wrote:
> Well, this is not the case now, but I wonder if it would also make sense to sort this list alphabetically. 
I had this idea as well. The list is currently not sorted but seems to match the order of the functions as they are listed in the `exec(3)` manual page so I kept it.

Let me know what is preferred and I can update the diff.


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

https://reviews.llvm.org/D73629