diff options
author | Bram Moolenaar <Bram@vim.org> | 2022-10-29 21:53:08 +0100 |
---|---|---|
committer | Bram Moolenaar <Bram@vim.org> | 2022-10-29 21:53:08 +0100 |
commit | 03d6e6f42b0deeb02d52c8a48c14abe431370c1c (patch) | |
tree | 293ef86f93e032c3111986f46ea7895ce30e89fb /src/ex_cmds.c | |
parent | 1577537f109d97a975fda9a899cacfb598617767 (diff) |
patch 9.0.0820: memory leak with empty shell commandv9.0.0820
Problem: Memory leak with empty shell command.
Solution: Free the empty string.
Diffstat (limited to 'src/ex_cmds.c')
-rw-r--r-- | src/ex_cmds.c | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/src/ex_cmds.c b/src/ex_cmds.c index 44333d514c..7bb7aa0380 100644 --- a/src/ex_cmds.c +++ b/src/ex_cmds.c @@ -957,12 +957,15 @@ do_bang( } } while (trailarg != NULL); - // Don't clear "prevcmd" if there is no command to run. + // Only set "prevcmd" if there is a command to run, otherwise keep te one + // we have. if (STRLEN(newcmd) > 0) { vim_free(prevcmd); prevcmd = newcmd; } + else + free_newcmd = TRUE; if (bangredo) // put cmd in redo buffer for ! command { @@ -986,6 +989,8 @@ do_bang( */ if (*p_shq != NUL) { + if (free_newcmd) + vim_free(newcmd); newcmd = alloc(STRLEN(prevcmd) + 2 * STRLEN(p_shq) + 1); if (newcmd == NULL) return; |