diff options
| author |  Daniel Lockyer <thisisdaniellockyer@gmail.com> | 2016-04-29 18:00:21 +0100 |
|---|---|---|
| committer | Daniel Lockyer <thisisdaniellockyer@gmail.com> | 2016-04-29 18:00:21 +0100 |
| commit | 235cfd93b22c24f851d6974a9b56bfe84023ac00 (patch) | |
| tree | 41312ed28947d432f1bbc972a92e6da494584324 /common | |
| parent | Merge pull request #614 from neosilky/memleak (diff) | |
| download | sway-235cfd93b22c24f851d6974a9b56bfe84023ac00.tar.gz sway-235cfd93b22c24f851d6974a9b56bfe84023ac00.tar.zst sway-235cfd93b22c24f851d6974a9b56bfe84023ac00.zip | |
common/stringop.c: a premature decrement meant an element of argv was never released
Diffstat (limited to 'common')
| -rw-r--r-- | common/stringop.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/common/stringop.c b/common/stringop.c index 61324458..432bee7f 100644 --- a/common/stringop.c +++ b/common/stringop.c | |||
| @@ -144,7 +144,7 @@ char **split_args(const char *start, int *argc) { | |||
| 144 | } | 144 | } |
| 145 | 145 | ||
| 146 | void free_argv(int argc, char **argv) { | 146 | void free_argv(int argc, char **argv) { |
| 147 | while (--argc > 0) { | 147 | while (argc-- > 0) { |
| 148 | free(argv[argc]); | 148 | free(argv[argc]); |
| 149 | } | 149 | } |
| 150 | free(argv); | 150 | free(argv); |