From 04732d3a8b0e3b4565e8a2ae8e5f6b9e95f076fb Mon Sep 17 00:00:00 2001
From: Dong Anyuan <donganyuan@cn.fujitsu.com>
Date: Fri, 31 May 2019 10:29:42 +0900
Subject: [PATCH] Fix Coverity Scan CID 341064 (Null-checking
 mac_MainConfig->phr_Config suggests that it may be null, but it has already
 been dereferenced on all paths leading to the check.)

---
 openair2/RRC/LTE/rrc_eNB.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/openair2/RRC/LTE/rrc_eNB.c b/openair2/RRC/LTE/rrc_eNB.c
index a84efd334b..1dd3a26053 100644
--- a/openair2/RRC/LTE/rrc_eNB.c
+++ b/openair2/RRC/LTE/rrc_eNB.c
@@ -3931,10 +3931,9 @@ flexran_rrc_eNB_generate_defaultRRCConnectionReconfiguration(const protocol_ctxt
   free(mac_MainConfig->ul_SCH_Config);
   mac_MainConfig->ul_SCH_Config = NULL;
 
-  if(mac_MainConfig->phr_Config){
-    free(mac_MainConfig->phr_Config);
-    mac_MainConfig->phr_Config = NULL;
-  }
+  free(mac_MainConfig->phr_Config);
+  mac_MainConfig->phr_Config = NULL;
+
   free(mac_MainConfig);
   mac_MainConfig = NULL;
 }
-- 
GitLab