Revision: 48481
http://brlcad.svn.sourceforge.net/brlcad/?rev=48481&view=rev
Author: indianlarry
Date: 2012-01-12 11:22:01 +0000 (Thu, 12 Jan 2012)
Log Message:
-----------
Cleaned up coverity identified memory leaks. These relate to Run 6 CIDs:
1865-76.
Modified Paths:
--------------
brlcad/trunk/src/conv/step/BSplineCurve.cpp
brlcad/trunk/src/conv/step/CompositeCurve.cpp
brlcad/trunk/src/conv/step/ConnectedFaceSet.cpp
brlcad/trunk/src/conv/step/CurveBoundedSurface.cpp
brlcad/trunk/src/conv/step/Face.cpp
brlcad/trunk/src/conv/step/GlobalUncertaintyAssignedContext.cpp
brlcad/trunk/src/conv/step/Path.cpp
brlcad/trunk/src/conv/step/Polyline.cpp
brlcad/trunk/src/conv/step/PullbackCurve.cpp
brlcad/trunk/src/conv/step/Representation.cpp
brlcad/trunk/src/conv/step/STEPWrapper.cpp
Modified: brlcad/trunk/src/conv/step/BSplineCurve.cpp
===================================================================
--- brlcad/trunk/src/conv/step/BSplineCurve.cpp 2012-01-12 02:06:57 UTC (rev
48480)
+++ brlcad/trunk/src/conv/step/BSplineCurve.cpp 2012-01-12 11:22:01 UTC (rev
48481)
@@ -101,6 +101,8 @@
control_points_list.push_back(aCP);
} else {
std::cerr << CLASSNAME << ": Unhandled entity in attribute
'control_points_list'." << std::endl;
+ l->clear();
+ delete l;
return false;
}
}
Modified: brlcad/trunk/src/conv/step/CompositeCurve.cpp
===================================================================
--- brlcad/trunk/src/conv/step/CompositeCurve.cpp 2012-01-12 02:06:57 UTC
(rev 48480)
+++ brlcad/trunk/src/conv/step/CompositeCurve.cpp 2012-01-12 11:22:01 UTC
(rev 48481)
@@ -84,6 +84,8 @@
segments.push_back(aCCS);
} else {
std::cerr << CLASSNAME << ": Unhandled entity in attribute
'segments'." << std::endl;
+ l->clear();
+ delete l;
return false;
}
}
Modified: brlcad/trunk/src/conv/step/ConnectedFaceSet.cpp
===================================================================
--- brlcad/trunk/src/conv/step/ConnectedFaceSet.cpp 2012-01-12 02:06:57 UTC
(rev 48480)
+++ brlcad/trunk/src/conv/step/ConnectedFaceSet.cpp 2012-01-12 11:22:01 UTC
(rev 48481)
@@ -80,6 +80,8 @@
cfs_faces.push_back(aAF);
} else {
std::cerr << CLASSNAME << ": Unhandled entity in attribute
'cfs_faces'." << std::endl;
+ l->clear();
+ delete l;
return false;
}
}
Modified: brlcad/trunk/src/conv/step/CurveBoundedSurface.cpp
===================================================================
--- brlcad/trunk/src/conv/step/CurveBoundedSurface.cpp 2012-01-12 02:06:57 UTC
(rev 48480)
+++ brlcad/trunk/src/conv/step/CurveBoundedSurface.cpp 2012-01-12 11:22:01 UTC
(rev 48481)
@@ -88,6 +88,8 @@
boundaries.push_back(aAF);
} else {
std::cerr << CLASSNAME << ": Unhandled entity in attribute
'cfs_faces'." << std::endl;
+ l->clear();
+ delete l;
return false;
}
}
Modified: brlcad/trunk/src/conv/step/Face.cpp
===================================================================
--- brlcad/trunk/src/conv/step/Face.cpp 2012-01-12 02:06:57 UTC (rev 48480)
+++ brlcad/trunk/src/conv/step/Face.cpp 2012-01-12 11:22:01 UTC (rev 48481)
@@ -82,6 +82,8 @@
bounds.push_back(aFB);
} else {
std::cerr << CLASSNAME << ": Unhandled entity in attribute
'bounds'." << std::endl;
+ l->clear();
+ delete l;
return false;
}
}
Modified: brlcad/trunk/src/conv/step/GlobalUncertaintyAssignedContext.cpp
===================================================================
--- brlcad/trunk/src/conv/step/GlobalUncertaintyAssignedContext.cpp
2012-01-12 02:06:57 UTC (rev 48480)
+++ brlcad/trunk/src/conv/step/GlobalUncertaintyAssignedContext.cpp
2012-01-12 11:22:01 UTC (rev 48481)
@@ -84,6 +84,8 @@
uncertainty.push_back(aUMWU);
} else {
std::cerr << CLASSNAME << ": Unhandled entity in attribute
'uncertainty'." << std::endl;
+ l->clear();
+ delete l;
return false;
}
}
Modified: brlcad/trunk/src/conv/step/Path.cpp
===================================================================
--- brlcad/trunk/src/conv/step/Path.cpp 2012-01-12 02:06:57 UTC (rev 48480)
+++ brlcad/trunk/src/conv/step/Path.cpp 2012-01-12 11:22:01 UTC (rev 48481)
@@ -115,6 +115,8 @@
std::cerr << CLASSNAME
<< ": Unhandled entity in attribute 'edge_list'."
<< std::endl;
+ l->clear();
+ delete l;
return false;
}
}
Modified: brlcad/trunk/src/conv/step/Polyline.cpp
===================================================================
--- brlcad/trunk/src/conv/step/Polyline.cpp 2012-01-12 02:06:57 UTC (rev
48480)
+++ brlcad/trunk/src/conv/step/Polyline.cpp 2012-01-12 11:22:01 UTC (rev
48481)
@@ -81,6 +81,8 @@
points.push_back(aCP);
} else {
std::cerr << CLASSNAME << ": Unhandled entity in attribute
'points'." << std::endl;
+ l->clear();
+ delete l;
return false;
}
}
Modified: brlcad/trunk/src/conv/step/PullbackCurve.cpp
===================================================================
--- brlcad/trunk/src/conv/step/PullbackCurve.cpp 2012-01-12 02:06:57 UTC
(rev 48480)
+++ brlcad/trunk/src/conv/step/PullbackCurve.cpp 2012-01-12 11:22:01 UTC
(rev 48481)
@@ -993,6 +993,7 @@
}
} else {
std::cerr << "pullback_samples:Error: cannot evaluate curve at
parameter 0.0" << std::endl;
+ delete data;
return NULL;
}
} else {
Modified: brlcad/trunk/src/conv/step/Representation.cpp
===================================================================
--- brlcad/trunk/src/conv/step/Representation.cpp 2012-01-12 02:06:57 UTC
(rev 48480)
+++ brlcad/trunk/src/conv/step/Representation.cpp 2012-01-12 11:22:01 UTC
(rev 48481)
@@ -153,6 +153,8 @@
items.push_back(aRI);
} else {
std::cerr << CLASSNAME << ": Unhandled entity in attribute
'items'." << std::endl;
+ l->clear();
+ delete l;
return false;
}
}
Modified: brlcad/trunk/src/conv/step/STEPWrapper.cpp
===================================================================
--- brlcad/trunk/src/conv/step/STEPWrapper.cpp 2012-01-12 02:06:57 UTC (rev
48480)
+++ brlcad/trunk/src/conv/step/STEPWrapper.cpp 2012-01-12 11:22:01 UTC (rev
48481)
@@ -90,6 +90,7 @@
// LocalUnits::solidangle = 1.0;
ON_Brep *onBrep = aBrep->GetONBrep();
if (!onBrep) {
+ delete aBrep;
bu_exit(1, "ERROR: failure creating advanced boundary
representation from %s\n", stepfile.c_str());
} else {
ON_TextLog tl;
@@ -103,6 +104,9 @@
delete onBrep;
}
delete aBrep;
+ } else {
+ delete aBrep;
+ bu_exit(1, "ERROR: failure loading advanced boundary
representation from %s\n", stepfile.c_str());
}
}
}
This was sent by the SourceForge.net collaborative development platform, the
world's largest Open Source development site.
------------------------------------------------------------------------------
RSA(R) Conference 2012
Mar 27 - Feb 2
Save $400 by Jan. 27
Register now!
http://p.sf.net/sfu/rsa-sfdev2dev2
_______________________________________________
BRL-CAD Source Commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/brlcad-commits
