aboutsummaryrefslogtreecommitdiffstats
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
parentefreet: remove debug fprintf (diff)
downloadefl-8204212f36d936f951accda61fd2f7f329390b71.tar.gz
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)
&len, command, *p);
if (!exec) goto error;
file_added = 1;
+ /* Set l to NULL to break the loop, since we parse all command->files
+ * in efreet_desktop_command_append_multiple */
+ l = NULL;
}
break;
case 'i':