summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSebastian Dransfeld <sd@tango.flipp.net>2014-09-25 09:22:22 +0200
committerSebastian Dransfeld <sd@tango.flipp.net>2014-09-25 09:24:27 +0200
commit8204212f36d936f951accda61fd2f7f329390b71 (patch)
treec67cd2ee00d83bddec927a89f1bb6cd0b31e739d
parent8e740ea765fc3171607f47e29c256802f5b4ca61 (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.
-rw-r--r--src/lib/efreet/efreet_desktop_command.c3
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':