From: Chad Versace <chad.versace@linux.intel.com>
Date: Thu, 24 May 2012 06:42:15 +0000 (-0700)
Subject: compositor: Fix buggy snprintf that sets module path
X-Git-Tag: 20120702.1049~106
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=bf381907449862230c6ace4f0a2b543641a2af48;p=profile%2Fivi%2Fweston.git

compositor: Fix buggy snprintf that sets module path

If the MODULEDIR string contains '%', then
    snprintf(path, sizeof(path), MODULEDIR "/%s", name);
does not do what you want.

Signed-off-by: Chad Versace <chad.versace@linux.intel.com>
---

diff --git a/src/compositor.c b/src/compositor.c
index c68b5ba..d21459b 100644
--- a/src/compositor.c
+++ b/src/compositor.c
@@ -2777,7 +2777,7 @@ load_module(const char *name, const char *entrypoint, void **handle)
 	void *module, *init;
 
 	if (name[0] != '/')
-		snprintf(path, sizeof path, MODULEDIR "/%s", name);
+		snprintf(path, sizeof path, "%s/%s", MODULEDIR, name);
 	else
 		snprintf(path, sizeof path, "%s", name);