From: Cheoleun Moon <chleun.moon@samsung.com>
Date: Wed, 17 Apr 2019 01:02:39 +0000 (+0900)
Subject: Fix code style
X-Git-Tag: submit/tizen/20190422.081829^0
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F72%2F203672%2F4;p=platform%2Fcore%2Fconnectivity%2Fnet-config.git

Fix code style

Change-Id: I541c2e466d81748420262960e71a1a33436f2821
Signed-off-by: Cheoleun Moon <chleun.moon@samsung.com>
---

diff --git a/src/ip-conflict-detect.c b/src/ip-conflict-detect.c
index 1873d8b..743c292 100755
--- a/src/ip-conflict-detect.c
+++ b/src/ip-conflict-detect.c
@@ -213,10 +213,9 @@ out:
 	return TRUE;
 }
 
-static void __close_channel_and_sock(struct sock_data* sd)
+static void __close_channel_and_sock(struct sock_data *sd)
 {
-	INFO("+");
-	GError* error = NULL;
+	GError *error = NULL;
 
 	if (sd == NULL)
 		return;
@@ -236,9 +235,8 @@ static void __close_channel_and_sock(struct sock_data* sd)
 	sd->chk_conflict_sd = -1;
 }
 
-static int __open_channel_and_sock(struct sock_data* sd)
+static int __open_channel_and_sock(struct sock_data *sd)
 {
-	INFO("+");
 	if (sd == NULL)
 		return -1;
 
@@ -250,16 +248,19 @@ static int __open_channel_and_sock(struct sock_data* sd)
 	sd->chk_conflict_sock_io = g_io_channel_unix_new(sd->chk_conflict_sd);
 	if (sd->chk_conflict_sock_io == NULL) {
 		INFO("Failed to create channel");
-		g_free(sd);
+		close(sd->chk_conflict_sd);
+		sd->chk_conflict_sd = -1;
 		return -1;
 	}
+
 	g_io_channel_set_close_on_unref(sd->chk_conflict_sock_io, TRUE);
-	if (G_IO_STATUS_NORMAL != g_io_channel_set_encoding(sd->chk_conflict_sock_io,
-				NULL, NULL))
+	if (G_IO_STATUS_NORMAL != g_io_channel_set_encoding(sd->chk_conflict_sock_io, NULL, NULL))
 		INFO("Failed to set encoding NULL on io channel");
+
 	if (G_IO_STATUS_NORMAL != g_io_channel_set_flags(sd->chk_conflict_sock_io,
 				G_IO_FLAG_NONBLOCK, NULL))
 		INFO("Failed to set flags on io channel");
+
 	sd->chk_conflict_data_id = g_io_add_watch(sd->chk_conflict_sock_io, G_IO_IN,
 			__netconfig_check_arp_receive, sd);
 	DBG("socket %d", sd->chk_conflict_sd);