[0.6.162] check resource manager daemon state 21/195721/1
authorEunhae Choi <eunhae1.choi@samsung.com>
Mon, 17 Dec 2018 12:48:27 +0000 (21:48 +0900)
committerEunhae Choi <eunhae1.choi@samsung.com>
Mon, 17 Dec 2018 12:48:30 +0000 (21:48 +0900)
- check 'resource manager ready' file before creating the rm handle
  to clarify the error case

Change-Id: I1177a825aed22c238110fbe24792ba29e1a156a6

packaging/libmm-player.spec
src/mm_player_priv.c

index 8e8b501..cb27e2f 100644 (file)
@@ -1,6 +1,6 @@
 Name:       libmm-player
 Summary:    Multimedia Framework Player Library
-Version:    0.6.161
+Version:    0.6.162
 Release:    0
 Group:      Multimedia/Libraries
 License:    Apache-2.0
index e6cef96..cddddb5 100644 (file)
@@ -4417,11 +4417,18 @@ _mmplayer_create_player(MMHandleType handle)
                goto ERROR;
        }
 
+       /* check resource manager state */
+       if (access(MM_RESOURCE_MANAGER_READY, F_OK) != 0) {
+               LOGE("resource manager is not ready");
+               ret = MM_ERROR_PLAYER_INTERNAL;
+               goto ERROR;
+       }
+
        /* initialize resource manager */
-       if (MM_RESOURCE_MANAGER_ERROR_NONE != mm_resource_manager_create(
-                       MM_RESOURCE_MANAGER_APP_CLASS_MEDIA, __resource_release_cb, player,
-                       &player->resource_manager)) {
-               LOGE("failed to initialize resource manager\n");
+       if (mm_resource_manager_create(MM_RESOURCE_MANAGER_APP_CLASS_MEDIA,
+               __resource_release_cb, player, &player->resource_manager)
+               != MM_RESOURCE_MANAGER_ERROR_NONE) {
+               LOGE("failed to initialize resource manager");
                ret = MM_ERROR_PLAYER_INTERNAL;
                goto ERROR;
        }
@@ -4486,6 +4493,8 @@ _mmplayer_create_player(MMHandleType handle)
        MMPLAYER_STATE_CHANGE_TIMEOUT(player) = player->ini.localplayback_state_change_timeout;
        MMPLAYER_SET_STATE(player, MM_PLAYER_STATE_NULL);
 
+       MMPLAYER_FLEAVE();
+
        return MM_ERROR_NONE;
 
 ERROR: