From e9c4a0470ef4102202d19a8ac405c7c23b6f758f Mon Sep 17 00:00:00 2001
From: ChenHengming <ChenHengming@6e48237b-75ef-9749-8fc9-41990f28c85a>
Date: Thu, 19 May 2016 08:21:21 +0000
Subject: [PATCH] [mpp_buffer]: fix legacy group runtime error on mpi test

git-svn-id: https://10.10.10.66:8443/svn/MediaProcessPlatform/trunk/mpp@780 6e48237b-75ef-9749-8fc9-41990f28c85a
---
 mpp/mpp_buffer_impl.cpp | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/mpp/mpp_buffer_impl.cpp b/mpp/mpp_buffer_impl.cpp
index 18fad37d..08acb1d8 100644
--- a/mpp/mpp_buffer_impl.cpp
+++ b/mpp/mpp_buffer_impl.cpp
@@ -117,17 +117,23 @@ static MppBufferGroupImpl *search_group_by_id_no_lock(struct list_head *list, RK
 
 MPP_RET deinit_group_no_lock(MppBufferGroupImpl *group)
 {
+    MppBufferGroupImpl *legacy = mpp_buffer_legacy_group();
+
     mpp_alloctor_put(&group->allocator);
     list_del_init(&group->list_group);
     mpp_free(group);
     service.group_count--;
-    /* if only legacy group left dump the legacy group */
-    if (service.group_count == 1) {
-        MppBufferGroupImpl *legacy = mpp_buffer_legacy_group();
-        if (legacy->count) {
-            mpp_log("found legacy group has buffer remain, start dumping\n");
-            mpp_buffer_group_dump(legacy);
-            abort();
+
+    if (group == legacy) {
+        service.mLegacyGroup = NULL;
+    } else {
+        /* if only legacy group left dump the legacy group */
+        if (service.group_count == 1) {
+            if (legacy->count) {
+                mpp_log("found legacy group has buffer remain, start dumping\n");
+                mpp_buffer_group_dump(legacy);
+                //abort();
+            }
         }
     }
     return MPP_OK;
@@ -532,7 +538,7 @@ MppBufferService::~MppBufferService()
     if (mLegacyGroup)
         mpp_buffer_group_deinit(mLegacyGroup);
 
-    // then remove the reset group
+    // then remove the remaining group
     if (!list_empty(&mListGroup)) {
         MppBufferGroupImpl *pos, *n;
         list_for_each_entry_safe(pos, n, &mListGroup, MppBufferGroupImpl, list_group) {