hyprwm/hyprutils
1
0
Fork 0
mirror of https://github.com/hyprwm/hyprutils.git synced 2026-05-07 06:38:01 +02:00
Code Issues Projects Releases Packages Wiki Activity

process: free argv allocations on execvp failure (#106)

Browse source 
* process: free argv allocations on execvp failure

* fix execvp style
This commit is contained in:
Visal Vijay 2026-04-26 01:47:03 +05:30 • committed by GitHub
parent eedd60805c
commit fa3992be2d
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 7 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
src/os/Process.cpp
View file

@ -66,7 +66,7 @@ bool Hyprutils::OS::CProcess::runSync() {
std::vector<char*> argsC;
argsC.emplace_back(strdup(m_impl->binary.c_str()));
for (auto& arg : m_impl->args) {
// TODO: does this leak? Can we just pipe c_str() as the strings won't be realloc'd?
// Arguments are duplicated for execvp; if execvp fails, they must be freed.
argsC.emplace_back(strdup(arg.c_str()));
}
@ -78,7 +78,12 @@ bool Hyprutils::OS::CProcess::runSync() {
}
execvp(m_impl->binary.c_str(), argsC.data());
exit(1);
for (auto ptr : argsC) {
if (ptr)
free(ptr);
}
_exit(1);
} else {
// parent
close(outPipe[1]);