[ 3486814 ] sfcb crashes while freeing the cimxml response

zaneb · Feb 11, 2012 · c8c4a7d · c8c4a7d
1 parent 4d51ed8
commit c8c4a7d
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/ChangeLog b/ChangeLog
@@ -1,3 +1,8 @@
+2012-02-11 Narasimha Sharoff <nsharoff@us.ibm.com>
+
+	* cimXmlParser.c
+	[ 3486814 ] sfcb crashes while freeing the cimxml response
+
 2012-02-11 Narasimha Sharoff <nsharoff@us.ibm.com>
 
 	* sfcBroker.c

diff --git a/NEWS b/NEWS
@@ -13,7 +13,6 @@ New features:
 
 Bugs fixed: 
 
-- 3485438 segfault while retrying to deliver an indication
 - 3414700 assocClass ignored for assoc upcalls
 - 3416055 SIGSEGV sometimes does not print provider name
 - 3416167 CIM_InstDeletion not in class repo by default
@@ -23,6 +22,8 @@ Bugs fixed:
 - 3478930 Handle SIGPIPE in providerMgr
 - 3478933 Sequence Number reset issues
 - 3483294 InternalProvider Should Handle SFCB_RegisteredProfile
+- 3485438 segfault while retrying to deliver an indication
+- 3486814 sfcb crashes while freeing the cimxml response
 
 Changes in 1.3.13
 =================

diff --git a/cimXmlParser.c b/cimXmlParser.c
@@ -1741,7 +1741,9 @@ static void freeParamValues(XtokParamValues *op)
 {
   XtokParamValue * pv = op->first , *dv;
   while (pv) {
-    freeParamValue(pv);
+    /* 3486814 : type is invalid, value is null */
+    if (pv->type != USHRT_MAX)
+       freeParamValue(pv);
     dv = pv;
     pv = pv->next;
     free(dv);