commit 8461119713e2cd5c706e9af02b4cc5840eeca7df
author Linux Build Service Account <lnxbuild@quicinc.com> Wed Jun 14 22:52:55 2017 -0700
committer Gerrit - the friendly Code Review server <code-review@localhost> Wed Jun 14 22:52:55 2017 -0700
tree 8befc3382004d51f19fb09f69ab947fd4e464021
parent 85343fb4a2dad38988171797984632fae77e228d
parent 0a4a8af5e7833ef7f73cd5a8cf9979d29fd15b6c

Merge "sdm: color-manager: Clear feature list when add new feature"

sdm/include/private/color_params.h

diff --git a/sdm/include/private/color_params.h b/sdm/include/private/color_params.h
index 0a53832..08895ee 100644
--- a/sdm/include/private/color_params.h
+++ b/sdm/include/private/color_params.h
@@ -578,9 +578,13 @@
   // from ColorManager, containing all physical features to be programmed and also compute
   // metadata/populate into T.
   inline DisplayError AddFeature(uint32_t feature_id, PPFeatureInfo *feature) {
-    if (feature_id < kMaxNumPPFeatures)
+    if (feature_id < kMaxNumPPFeatures) {
+      if (feature_[feature_id]) {
+        delete feature_[feature_id];
+        feature_[feature_id] = NULL;
+      }
       feature_[feature_id] = feature;
-
+    }
     return kErrorNone;
   }