diff options
author | Sebastian Dransfeld <sd@tango.flipp.net> | 2014-09-25 09:22:22 +0200 |
---|---|---|
committer | Sebastian Dransfeld <sd@tango.flipp.net> | 2014-09-25 09:24:27 +0200 |
commit | 8204212f36d936f951accda61fd2f7f329390b71 (patch) | |
tree | c67cd2ee00d83bddec927a89f1bb6cd0b31e739d /src/lib/efreet/efreet_desktop_command.c | |
parent | 8e740ea765fc3171607f47e29c256802f5b4ca61 (diff) |
efreet: Fix leak
When doing efreet_desktop_command_append_multiple we add all files from
command, whilst still looping command->files. So the command was created
with all files for all files.
Set l to NULL, since we parse all elements in command->files in
efreet_desktop_command_append_multiple to break the loop.
Diffstat (limited to '')
-rw-r--r-- | src/lib/efreet/efreet_desktop_command.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/lib/efreet/efreet_desktop_command.c b/src/lib/efreet/efreet_desktop_command.c index 51e60c9957..15a4a5c105 100644 --- a/src/lib/efreet/efreet_desktop_command.c +++ b/src/lib/efreet/efreet_desktop_command.c | |||
@@ -381,6 +381,9 @@ efreet_desktop_command_build(Efreet_Desktop_Command *command) | |||
381 | &len, command, *p); | 381 | &len, command, *p); |
382 | if (!exec) goto error; | 382 | if (!exec) goto error; |
383 | file_added = 1; | 383 | file_added = 1; |
384 | /* Set l to NULL to break the loop, since we parse all command->files | ||
385 | * in efreet_desktop_command_append_multiple */ | ||
386 | l = NULL; | ||
384 | } | 387 | } |
385 | break; | 388 | break; |
386 | case 'i': | 389 | case 'i': |