[Sblim-commit] cmpi-devel CmpiImpl.cpp,1.12,1.13 NEWS,1.20,1.21

[Tyrel D. <ty...@us...>]

Update of /cvsroot/sblim/cmpi-devel
In directory vz-cvs-3.sog:/tmp/cvs-serv1813

Modified Files:
	CmpiImpl.cpp NEWS 
Log Message:
Fixed 3531598: several CmpiObjectPath methods leak memory


Index: NEWS
===================================================================
RCS file: /cvsroot/sblim/cmpi-devel/NEWS,v
retrieving revision 1.20
retrieving revision 1.21
diff -u -d -r1.20 -r1.21
--- NEWS	3 Jun 2012 04:51:09 -0000	1.20
+++ NEWS	3 Jun 2012 05:29:32 -0000	1.21
@@ -2,6 +2,7 @@
 ================
 
 Bugs Fixed:
+- 3531598 several CmpiObjectPath methods leak memory
 
 New Features:
 - 3531601 add CmpiString constructor for char data

Index: CmpiImpl.cpp
===================================================================
RCS file: /cvsroot/sblim/cmpi-devel/CmpiImpl.cpp,v
retrieving revision 1.12
retrieving revision 1.13
diff -u -d -r1.12 -r1.13
--- CmpiImpl.cpp	3 Jun 2012 04:51:09 -0000	1.12
+++ CmpiImpl.cpp	3 Jun 2012 05:29:32 -0000	1.13
@@ -1133,7 +1133,7 @@
      else
        throw CmpiStatus(rc.rc,name->charPtr());
    }
-   if (name) *name=*(new CmpiString(s));
+   if (name) *name=CmpiString(s);
    return d;
 };
 
@@ -1268,7 +1268,7 @@
    CMPIStatus rc={CMPI_RC_OK,NULL};
    CMPIString *s=getEnc()->ft->getNameSpace(getEnc(),&rc);
    if (rc.rc!=CMPI_RC_OK) throw CmpiStatus(rc);
-   return *new CmpiString(s);
+   return CmpiString(s);
 }
 
 void CmpiObjectPath::setNameSpace(const char* ns) {
@@ -1280,7 +1280,7 @@
    CMPIStatus rc={CMPI_RC_OK,NULL};
    CMPIString *s=getEnc()->ft->getHostname(getEnc(),&rc);
    if (rc.rc!=CMPI_RC_OK) throw CmpiStatus(rc);
-   return *new CmpiString(s);
+   return CmpiString(s);
 }
 
 void CmpiObjectPath::setHostname(const char* hn) {
@@ -1292,7 +1292,7 @@
    CMPIStatus rc={CMPI_RC_OK,NULL};
    CMPIString *s=getEnc()->ft->getClassName(getEnc(),&rc);
    if (rc.rc!=CMPI_RC_OK) throw CmpiStatus(rc);
-   return *new CmpiString(s);
+   return CmpiString(s);
 }
 
 void CmpiObjectPath::setClassName(const char* cn) {
@@ -1321,7 +1321,7 @@
    CMPIString *s;
    d._data=getEnc()->ft->getKeyAt(getEnc(),(int)pos,&s,&rc);
    if (rc.rc!=CMPI_RC_OK) throw CmpiStatus(rc);
-   if (name) *name=*(new CmpiString(s));
+   if (name) *name=CmpiString(s);
    return d;
 }
 
@@ -1643,7 +1643,7 @@
      else
        throw CmpiStatus(rc.rc,name->charPtr());
    }
-   if (name) *name=*(new CmpiString(s));
+   if (name) *name=CmpiString(s);
    return d;
 }