Fix that getting command did not work for permission denied

[Cause]
The server process should check the safety of the command.
The check-list of the command exist in the list of the client.
So it should be checked with the data of the client.

[Solution]
The '__mc_get_data' search the command with the check-list.
The '__mc_get_data' don't care about the pid(client & server).

Change-Id: I51d68b215006e92a04586e6d635fdd15999ff3cb
Signed-off-by: Jiyong Min <jiyong.min@samsung.com>
Conflicts:
	svc/media_controller_svc.c

Change-Id: I77c7335519cf70809b8c68c0bf0c7242801124d8

diff --git a/packaging/capi-media-controller.spec b/packaging/capi-media-controller.spec
index 34867092c08bef59fcfc0ab6624edc4dd2ac1823..0e837574eb71a1864698c817ef27a68886ebe521 100755 (executable)
--- a/packaging/capi-media-controller.spec
+++ b/packaging/capi-media-controller.spec
@@ -1,6 +1,6 @@
 Name:       capi-media-controller
 Summary:    A media controller library in Tizen Native API
-Version:    0.1.50
+Version:    0.1.51
 Release:    1
 Group:      Multimedia/API
 License:    Apache-2.0

diff --git a/svc/media_controller_svc.c b/svc/media_controller_svc.c
index 6849ea8e8f203f0ec7e89caabfb1eaa116ef53c2..4786d4c116c6586d28868d1921fefd9cbc67d270 100755 (executable)
--- a/svc/media_controller_svc.c
+++ b/svc/media_controller_svc.c
@@ -136,24 +136,23 @@ static void __mc_get_data(gpointer data, gpointer user_data)
        mc_list_data_set_t *_data = (mc_list_data_set_t *)data;
        mc_list_user_data *_user_data = (mc_list_user_data *)user_data;
        mc_comm_msg_s *_message = (mc_comm_msg_s *)_user_data->message;
-       if (_data->pid == _message->pid) {
-               if (_data->s_data != NULL) {
-                       unsigned int i = 0;
-                       unsigned int length_of_list = g_list_length(_data->s_data);
-                       char *value = NULL;
-                       for (i = 0; i < length_of_list; i++) {
-                               value = (char *)g_list_nth_data(_data->s_data, i);
-                               if ((value != NULL) && (strlen(value) == _message->msg_size)) {
-                                       if (strncmp(value, _message->msg, _message->msg_size) == 0) {
-                                               _data->s_data = g_list_remove(_data->s_data, value);
-                                               MC_SAFE_FREE(value);
-                                               _user_data->result++;
-                                       }
+
+       if (_data->s_data != NULL) {
+               unsigned int i = 0;
+               unsigned int length_of_list = g_list_length(_data->s_data);
+               char *value = NULL;
+               for (i = 0; i < length_of_list; i++) {
+                       value = (char *)g_list_nth_data(_data->s_data, i);
+                       if ((value != NULL) && (strlen(value) == _message->msg_size)) {
+                               if (strncmp(value, _message->msg, _message->msg_size) == 0) {
+                                       _data->s_data = g_list_remove(_data->s_data, value);
+                                       MC_SAFE_FREE(value);
+                                       _user_data->result++;
                                }
                        }
-                       if (g_list_length(_data->s_data) == 0)
-                               _data->s_data = NULL;
                }
+               if (g_list_length(_data->s_data) == 0)
+                       _data->s_data = NULL;
        }
 }