diff options
| author | Lukas Fleischer <lfleischer@calcurse.org> | 2018-08-25 10:22:29 +0200 |
|---|---|---|
| committer | Lukas Fleischer <lfleischer@calcurse.org> | 2018-08-25 10:22:29 +0200 |
| commit | 5efc334c1b0e8640e20b123d31f6ea52e20f9c2f (patch) | |
| tree | 4fc326fa853a187f0dfb34b86576ccce0c5f12dc /src | |
| parent | 7f448ee8cd8e5031ccc48604af2857bde94b0bae (diff) | |
| download | calcurse-5efc334c1b0e8640e20b123d31f6ea52e20f9c2f.tar.gz calcurse-5efc334c1b0e8640e20b123d31f6ea52e20f9c2f.zip | |
Fix memory leak in run_hook()
Fixes GitHub issue #139.
Signed-off-by: Lukas Fleischer <lfleischer@calcurse.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/hooks.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/hooks.c b/src/hooks.c index ef917e4..b34313c 100644 --- a/src/hooks.c +++ b/src/hooks.c @@ -40,7 +40,7 @@ int run_hook(const char *name) { - char *hook_path; + char *hook_path = NULL; char const *arg[2]; int pid, ret = -127; int prepare_wins = (ui_mode == UI_CURSES); @@ -50,7 +50,7 @@ int run_hook(const char *name) arg[1] = NULL; if (!io_file_exists(hook_path)) - return 0; + goto cleanup; if (prepare_wins) wins_prepare_external(); @@ -64,5 +64,7 @@ int run_hook(const char *name) if (prepare_wins) wins_unprepare_external(); +cleanup: + mem_free(hook_path); return ret; } |