diff options
| author | Nicholas Marriott <nicm@openbsd.org> | 2011-08-30 09:18:52 +0000 |
|---|---|---|
| committer | Nicholas Marriott <nicm@openbsd.org> | 2011-08-30 09:18:52 +0000 |
| commit | 423649b2c61eaa976490bfa331b35cec7de92d48 (patch) | |
| tree | 41d3325e448080e8b8422498adf0096a6c5dbbf8 /cmd-split-window.c | |
| parent | 2888843a188c796dd230c5eb15916cd381f68d56 (diff) | |
Plug memory leak, from Tiago Cunha.
Diffstat (limited to 'cmd-split-window.c')
| -rw-r--r-- | cmd-split-window.c | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/cmd-split-window.c b/cmd-split-window.c index e34645a4..31654771 100644 --- a/cmd-split-window.c +++ b/cmd-split-window.c @@ -58,7 +58,7 @@ cmd_split_window_exec(struct cmd *self, struct cmd_ctx *ctx) struct window *w; struct window_pane *wp, *new_wp = NULL; struct environ env; - char *cmd, *cwd, *cause; + char *cmd, *cwd, *cause, *new_cause; const char *shell; u_int hlimit, paneidx; int size, percentage; @@ -94,16 +94,18 @@ cmd_split_window_exec(struct cmd *self, struct cmd_ctx *ctx) if (args_has(args, 'l')) { size = args_strtonum(args, 'l', 0, INT_MAX, &cause); if (cause != NULL) { - ctx->error(ctx, "size %s", cause); + xasprintf(&new_cause, "size %s", cause); xfree(cause); - return (-1); + cause = new_cause; + goto error; } } else if (args_has(args, 'p')) { percentage = args_strtonum(args, 'p', 0, INT_MAX, &cause); if (cause != NULL) { - ctx->error(ctx, "percentage %s", cause); + xasprintf(&new_cause, "percentage %s", cause); xfree(cause); - return (-1); + cause = new_cause; + goto error; } if (type == LAYOUT_TOPBOTTOM) size = (wp->sy * percentage) / 100; |