From d09bbffceb2babe51000ff3042c09febc3afaa09 Mon Sep 17 00:00:00 2001 From: Minje Ahn Date: Fri, 16 Oct 2015 11:34:01 +0900 Subject: Fix prevent issue Change-Id: I0da7fbf08691fc8a262584d0e4f8edbfbd809b45 Signed-off-by: Minje Ahn --- server/include/thumb-server-internal.h | 2 -- server/thumb-server-internal.c | 9 ++++++--- 2 files changed, 6 insertions(+), 5 deletions(-) (limited to 'server') diff --git a/server/include/thumb-server-internal.h b/server/include/thumb-server-internal.h index 7869fe3..e3caa99 100755 --- a/server/include/thumb-server-internal.h +++ b/server/include/thumb-server-internal.h @@ -27,8 +27,6 @@ #ifndef _THUMB_DAEMON_INTERNAL_H_ #define _THUMB_DAEMON_INTERNAL_H_ -#define SAFE_FREE(src) { if(src) {free(src); src = NULL;}} - typedef enum { MEDIA_SERVER_PID = 1, OTHERS_PID = 0, diff --git a/server/thumb-server-internal.c b/server/thumb-server-internal.c index eaf1745..dd282a0 100755 --- a/server/thumb-server-internal.c +++ b/server/thumb-server-internal.c @@ -46,6 +46,7 @@ #define THUMB_DEFAULT_HEIGHT 240 #define THUMB_BLOCK_SIZE 512 #define THUMB_ROOT_UID "0" +#define THUMB_COMM_SOCK_PATH "/var/run/media-server/media_ipc_thumbcomm.socket" static __thread char **arr_path; static __thread uid_t *arr_uid; @@ -486,7 +487,7 @@ static gboolean __thumb_server_send_msg_to_agent(int msg_type) sock = sock_info.sock_fd; serv_addr.sun_family = AF_UNIX; - strcpy(serv_addr.sun_path, "/var/run/media-server/media_ipc_thumbcomm.socket"); + strncpy(serv_addr.sun_path, THUMB_COMM_SOCK_PATH, strlen(THUMB_COMM_SOCK_PATH)); /* Connecting to the thumbnail server */ @@ -546,6 +547,7 @@ gboolean _thumb_server_prepare_socket(int *sock_fd) if (ms_cynara_enable_credentials_passing(sock) != MS_MEDIA_ERR_NONE) { thumb_err("ms_cynara_enable_credentials_passing failed"); + close(sock); return FALSE; } @@ -559,7 +561,8 @@ static char* _media_thumb_get_default_path(uid_t uid) char *result_psswd = NULL; struct group *grpinfo = NULL; if (uid == getuid()) { - result_psswd = strdup(THUMB_DEFAULT_PATH); + if (THUMB_DEFAULT_PATH != NULL) + result_psswd = strndup(THUMB_DEFAULT_PATH, strlen(THUMB_DEFAULT_PATH)); grpinfo = getgrnam("users"); if (grpinfo == NULL) { thumb_err("getgrnam(users) returns NULL !"); @@ -757,7 +760,7 @@ int _media_thumb_process(thumbMsg *req_msg, thumbMsg *res_msg, uid_t uid) thumb_h = req_msg->thumb_height; thumb_path = res_msg->dst_path; thumb_path[0] = '\0'; - max_length = sizeof(res_msg->dst_path); + max_length = sizeof(res_msg->dst_path) -1; if (!g_file_test(origin_path, G_FILE_TEST_EXISTS | G_FILE_TEST_IS_REGULAR)) { thumb_err("origin_path does not exist in file system."); -- cgit v1.2.3