From: kibak.yoon <kibak.yoon@samsung.com>
Date: Thu, 3 Mar 2016 03:05:46 +0000 (+0900)
Subject: sensord: fix the svace issue because of using non-terminated string
X-Git-Tag: submit/tizen/20160315.065646^2~9
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=ad60f6010ca0b1713420e70a5af8f4866f8d3098;p=platform%2Fcore%2Fsystem%2Fsensord.git

sensord: fix the svace issue because of using non-terminated string

Change-Id: Ibcbee0bac176f72c647639f8fb950dca1cf8fe70
Signed-off-by: kibak.yoon <kibak.yoon@samsung.com>
---

diff --git a/src/shared/command_common.h b/src/shared/command_common.h
index a75415c..bb67f70 100644
--- a/src/shared/command_common.h
+++ b/src/shared/command_common.h
@@ -23,8 +23,8 @@
 #include <cpacket.h>
 #include <sensor_common.h>
 
-#define COMMAND_CHANNEL_PATH		"/tmp/sensord_command_socket"
-#define EVENT_CHANNEL_PATH			"/tmp/sensord_event_socket"
+#define COMMAND_CHANNEL_PATH		"/tmp/sensord_command_socket\0"
+#define EVENT_CHANNEL_PATH			"/tmp/sensord_event_socket\0"
 
 #define MAX_HANDLE			256
 #define MAX_HANDLE_REACHED	-2
diff --git a/src/shared/csocket.cpp b/src/shared/csocket.cpp
index 513c4b1..c6e7998 100644
--- a/src/shared/csocket.cpp
+++ b/src/shared/csocket.cpp
@@ -95,8 +95,7 @@ bool csocket::bind (const char *sock_path)
 
 	m_addr.sun_family = AF_UNIX;
 
-	strncpy(m_addr.sun_path, sock_path, sizeof(m_addr.sun_path));
-	m_addr.sun_path[sizeof(m_addr.sun_path)-1] = '\0';
+	strncpy(m_addr.sun_path, sock_path, strlen(sock_path));
 
 	length = strlen(m_addr.sun_path) + sizeof(m_addr.sun_family);
 
@@ -297,8 +296,7 @@ bool csocket::connect(const char *sock_path)
 
 	m_addr.sun_family = AF_UNIX;
 
-	strncpy(m_addr.sun_path, sock_path, sizeof(m_addr.sun_path));
-	m_addr.sun_path[sizeof(m_addr.sun_path)-1] = '\0';
+	strncpy(m_addr.sun_path, sock_path, strlen(sock_path));
 
 	addr_len = strlen(m_addr.sun_path) + sizeof(m_addr.sun_family);