fix(process_util): Memory leak

This commit is contained in:
Michael Carlberg 2016-12-14 15:02:56 +01:00
parent b11a662d81
commit 16592ce514

View File

@ -27,17 +27,22 @@ namespace process_util {
* Execute command * Execute command
*/ */
void exec(char* cmd, char** args) { void exec(char* cmd, char** args) {
execvp(cmd, args); if (cmd != nullptr) {
throw system_error("execvp() failed"); execvp(cmd, args);
throw system_error("execvp() failed");
}
} }
/** /**
* Execute command using shell * Execute command using shell
*/ */
void exec_sh(const char* cmd) { void exec_sh(const char* cmd) {
static const char* shell = env_util::get("SHELL", "/bin/sh").c_str(); static const string shell{env_util::get("SHELL", "/bin/sh").c_str()};
execlp(shell, shell, "-c", cmd, nullptr);
throw system_error("execvp() failed"); if (cmd != nullptr) {
execlp(shell.c_str(), shell.c_str(), "-c", cmd, nullptr);
throw system_error("execvp() failed");
}
} }
/** /**