[Sblim-commit] Git: SFCC - Small Footprint CIM Client branch, master, updated. 6c78d522c560523af0f68868fa1ce98b1e4070b9

[Chris B. <buc...@us...>]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "SFCC - Small Footprint CIM Client".

The branch, master has been updated
       via  6c78d522c560523af0f68868fa1ce98b1e4070b9 (commit)
      from  b3f5edd206d53510beecb86d943c9d34bd2c77ac (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 6c78d522c560523af0f68868fa1ce98b1e4070b9
Author: buccella <buc...@li...>
Date:   Tue Jul 31 15:14:18 2012 -0400

    [ 3547832 ] memory leak in backend/cimxml/grammer.c

-----------------------------------------------------------------------

Summary of changes:
diff --git a/ChangeLog b/ChangeLog
index 8de2ebc..8ea9c4b 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2012-07-31  Chris Buccella  <buc...@li...>
+
+	* backend/cimxml/grammar.c:
+	[ 3547832 ] memory leak in backend/cimxml/grammer.c
+	(patch by Michal Minar)
+
 2012-06-07  Chris Buccella  <buc...@li...>
 
 	* backend/cimxml/client.c, cimc/cimc.h, cimc/cimcclient.c:
diff --git a/NEWS b/NEWS
index 86b4140..0ffe4a9 100644
--- a/NEWS
+++ b/NEWS
@@ -10,6 +10,7 @@ Bugs:
 - 3528819: Crash in getInstance when instance has NULL properties
 - 3530609: Various BEAM Errors
 - 3435363: Wrong call of curl_global_cleanup()
+- 3547832: memory leak in backend/cimxml/grammer.c
 
 Changes in 2.2.3
 ================
diff --git a/backend/cimxml/grammar.c b/backend/cimxml/grammar.c
index 8fb5f91..d109421 100644
--- a/backend/cimxml/grammar.c
+++ b/backend/cimxml/grammar.c
@@ -571,7 +571,7 @@ static void valueObjectWithPath(ParserControl *parm, parseUnion *stateUnion)
 static void valueObjectWithPathData(ParserControl *parm, parseUnion *stateUnion)
 {
 	parseUnion lvalp={0};
-	CMPIObjectPath *op;
+	CMPIObjectPath *op = NULL;
 	CMPIInstance *inst;
 	ct = localLex((parseUnion*)&stateUnion->xtokObjectWithPathData, parm);
 	if(ct == XTOK_CLASSPATH) {
@@ -590,6 +590,7 @@ static void valueObjectWithPathData(ParserControl *parm, parseUnion *stateUnion)
 		setInstQualifiers(inst, &stateUnion->xtokObjectWithPathData.inst.inst.qualifiers);
 		setInstProperties(inst, &stateUnion->xtokObjectWithPathData.inst.inst.properties);
 		simpleArrayAdd(parm->respHdr.rvArray,(CMPIValue*)&inst,CMPI_instance);
+		if (op) op->ft->release(op);
 	}
 	else {
 		parseError("XTOK_CLASSPATH or XTOK_INSTANCEPATH", ct, parm);
diff --git a/contributions.txt b/contributions.txt
index 0a2984c..4aad521 100644
--- a/contributions.txt
+++ b/contributions.txt
@@ -53,3 +53,7 @@ Josef Moellers, Fujitsu
 J. Bart Whiteley, Novell
 ------------------------
 03/29/2011 [ 1956407 ] CIM-XML/HTTP connection to unix socket
+
+Michal Minar
+------------
+07/31/2012 [ 3547832 ] memory leak in backend/cimxml/grammer.c


hooks/post-receive
-- 
SFCC - Small Footprint CIM Client