Modify to do chmod() before listening server socket & remove unused user-service...

(Rarely application can not access socket because of no permission)

Change-Id: I9cc11ce3f733d0e2db43140e192f27b410a8bf63
Signed-off-by: Jiyong Min <jiyong.min@samsung.com>
(Cherry-picked from commit I0e8fb10f5dfd0c1b68e723db9a5a6c3ff4d884de)

diff --git a/packaging/capi-media-controller.spec b/packaging/capi-media-controller.spec
index 0a81201b33c6f101c199c98d3a9f59ecd7d3e8e6..3934c9e5c7fecb9ccbc572695ab6e49b99ec128c 100755 (executable)
--- a/packaging/capi-media-controller.spec
+++ b/packaging/capi-media-controller.spec
@@ -1,13 +1,12 @@
 Name:       capi-media-controller
 Summary:    A media controller library in Tizen Native API
-Version:    0.1.35
+Version:    0.1.36
 Release:    1
 Group:      Multimedia/API
 License:    Apache-2.0
 Source0:    %{name}-%{version}.tar.gz
 Source1:    mediacontroller.service
 Source2:    mediacontroller.socket
-Source3:    media-controller-user.service
 Source5:    500.media-controller.upgrade.sh
 BuildRequires:  cmake
 BuildRequires:  sqlite

diff --git a/packaging/media-controller-user.service b/packaging/media-controller-user.service
deleted file mode 100755 (executable)
index a7b5e1a..0000000
--- a/packaging/media-controller-user.service
+++ /dev/null
@@ -1,12 +0,0 @@
-[Unit]
-Description=Media controller user
-
-[Service]
-Type=idle
-CPUAccounting=true
-CPUQuota=5%
-ExecStartPre=/usr/bin/sleep 5
-ExecStart=/usr/bin/sh -c "/usr/bin/media-controller_create_db.sh"
-
-[Install]
-WantedBy=default.target

diff --git a/svc/media_controller_socket.c b/svc/media_controller_socket.c
index a2bfde97c911318365c21702b381d8b1baad6145..ebd313420b15cb367018e048c59c0b96d9ff60c4 100644 (file)
--- a/svc/media_controller_socket.c
+++ b/svc/media_controller_socket.c
@@ -104,6 +104,10 @@ int mc_ipc_create_server_socket(mc_msg_port_type_e port, int *sock_fd)
                return MEDIA_CONTROLLER_ERROR_INVALID_OPERATION;
        }
 
+       /*change permission of sock file*/
+       if (chmod(MC_IPC_PATH, 0666) < 0)
+               mc_stderror("chmod failed");
+
        mc_debug("bind success");
 
        /* Listening */
@@ -115,10 +119,6 @@ int mc_ipc_create_server_socket(mc_msg_port_type_e port, int *sock_fd)
 
        mc_debug("Listening...");
 
-       /*change permission of sock file*/
-       if (chmod(MC_IPC_PATH, 0666) < 0)
-               mc_stderror("chmod failed");
-
        *sock_fd = sock;
 
        return MEDIA_CONTROLLER_ERROR_NONE;