[0.6.105] delete rset without releasing it 61/212261/1 accepted/tizen/4.0/unified/20190821.120102 submit/tizen_4.0/20190820.075325
authorEunhye Choi <eunhae1.choi@samsung.com>
Mon, 19 Aug 2019 10:37:29 +0000 (19:37 +0900)
committerEunhye Choi <eunhae1.choi@samsung.com>
Mon, 19 Aug 2019 10:37:32 +0000 (19:37 +0900)
- After _rset_state_callback() is returned,
  murphy get crash by accessing invalid mem which
  was released during _resource_manager_deinit().
- _rset_state_cb is called by releasing resource
  in _resource_manager_deinit() and releasing rset
  is not required because it is going to be deleted.

Change-Id: I686b1dd1e593cc835f46e8f1cbf96bbf438fed52

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

index e7c50a9c9594f83656486ba8cf03886f54759d24..e42321c2a560ba9bf44fdc8f5812c608c0e7f939 100644 (file)
@@ -1,6 +1,6 @@
 Name:       libmm-player
 Summary:    Multimedia Framework Player Library
-Version:    0.6.104
+Version:    0.6.105
 Release:    0
 Group:      Multimedia/Libraries
 License:    Apache-2.0
index 81758e535a847ccdf379921b55349d67793d685a..f5126f00dbeb27ed19f80461f17aa01d2b4164d1 100644 (file)
@@ -523,10 +523,6 @@ int _mmplayer_resource_manager_deinit(MMPlayerResourceManager *resource_manager)
 
        MMPLAYER_RESOURCE_SET_LOCK(resource_manager);
        if (resource_manager->rset) {
-               if (resource_manager->rset->state == MRP_RES_RESOURCE_ACQUIRED) {
-                       if (mrp_res_release_resource_set(resource_manager->rset))
-                               LOGE("- could not release resource\n");
-               }
                mrp_res_delete_resource_set(resource_manager->rset);
                resource_manager->rset = NULL;
        }