[PATCH weston 11/11] desktop-shell: Refactor launcher configuration reading to avoid leaking

Rob Bradford robert.bradford at intel.com
Fri Jul 26 08:29:45 PDT 2013


From: Rob Bradford <rob at linux.intel.com>

---
 clients/desktop-shell.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/clients/desktop-shell.c b/clients/desktop-shell.c
index eb550c8..ad324ba 100644
--- a/clients/desktop-shell.c
+++ b/clients/desktop-shell.c
@@ -1205,15 +1205,13 @@ panel_add_launchers(struct panel *panel, struct desktop *desktop)
 
 		if (icon != NULL && path != NULL) {
 			panel_add_launcher(panel, icon, path);
+			count++;
 		} else {
 			fprintf(stderr, "invalid launcher section\n");
-			continue;
 		}
 
 		free(icon);
 		free(path);
-
-		count++;
 	}
 
 	if (count == 0) {
-- 
1.8.3.1



More information about the wayland-devel mailing list