RESOLVED FIXED 236366
[CMake] Checks uses of execute_process() for correct command splitting 
https://bugs.webkit.org/show_bug.cgi?id=236366
Summary [CMake] Checks uses of execute_process() for correct command splitting
Adrian Perez
Reported 2022-02-09 07:43:59 PST
I fixed two instances in bug #236365 which were introduced by other of my recent patches, but there are other uses of execute_process() which may need a similar treatment: % rg -tcmake '\bexecute_process\(' Source/ | wc -l 23 % I am not sure if there are other CMake commands which behave in a similar way and would be good to check as well.
Attachments
Patch (4.41 KB, patch)
2022-02-10 06:48 PST, Adrian Perez 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Adrian Perez
Comment 1 2022-02-10 06:48:33 PST
Created attachment 451524 [details] Patch
Adrian Perez
Comment 2 2022-02-10 06:50:53 PST
(In reply to Adrian Perez from comment #1) > Created attachment 451524 [details] > Patch The rest of uses of execute_process() either take care of properly splitting command line arguments, or directly specify the command without gathering multiple arguments from the same variable -- in the end only two instances of its usage were removed.
EWS
Comment 3 2022-02-15 10:24:53 PST
Committed r289819 (247280@main): <https://commits.webkit.org/247280@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 451524 [details].
Radar WebKit Bug Importer
Comment 4 2022-02-15 10:25:19 PST
<rdar://problem/88973413>
Note You need to log in before you can comment on or make changes to this bug.