common: fix minor issuessubmit/tizen/20160921.044915accepted/tizen/wearable/20160921.232934accepted/tizen/tv/20160921.232944accepted/tizen/mobile/20160921.232939accepted/tizen/ivi/20160921.232950accepted/tizen/common/20160921.162506

- memory leak
- uninitialized variable
- memory release sequence

Change-Id: I9a4271bf4a83029e5ce4dd9f38be1cdb2fa7682f
Signed-off-by: taeyoung <ty317.kim@samsung.com>

2 files changed, 3 insertions, 3 deletions

diff --git a/src/libdeviced/usbhost.c b/src/libdeviced/usbhost.c
index cf00e69c..1cb0d1d4 100644
--- a/src/libdeviced/usbhost.c
+++ b/src/libdeviced/usbhost.c
@@ -71,6 +71,7 @@ static int register_edbus_signal_handler(const char *path, const char *interface
 	handler = calloc(1, sizeof(*handler));
 	if (!handler) {
 		_E("No memory");
+		ret = -1;
 		goto out;
 	}
 	handler->name = name;
diff --git a/src/usbhost/usb-host.c b/src/usbhost/usb-host.c
index 83e26384..16d65df8 100644
--- a/src/usbhost/usb-host.c
+++ b/src/usbhost/usb-host.c
@@ -609,9 +609,8 @@ static int store_policy(void)
 
 	fd = open(POLICY_FILEPATH, O_WRONLY | O_CREAT, 0664);
 	if (fd < 0) {
-		ret = -errno;
 		_E("Could not open policy file for writing: %m");
-		goto out;
+		return -errno;
 	}
 
 	DD_LIST_FOREACH(access_list, elem, entry) {
@@ -684,8 +683,8 @@ static int read_policy(void)
 				value_str);
 		if (ret == EOF) {
 			_E("Error reading line: %m");
-			free(entry);
 			free(entry->creds.sec_label);
+			free(entry);
 			goto out;
 		}