From: joncruz <joncruz@users.sourceforge.net>
Date: Wed, 31 May 2006 06:30:53 +0000 (+0000)
Subject: Fixing crash on odg save
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=2ed2602d2b0fc19510377df4e86eac87a0241518;p=inkscape.git

Fixing crash on odg save
---

diff --git a/ChangeLog b/ChangeLog
index bba0e3347..cfed5f1ab 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,9 @@
+2006-05-29  Jon A. Cruz  <jon@joncruz.org>
+
+	* src/extension/internal/odf.cpp:
+	Corrected deletes and uninitizlied variables.
+	Fixes bug# 1489961.
+
 2006-05-30 John Cliff <simarilius@yahoo.com>
 	src/trace/potrace/inkscape-potrace.cpp
 	Changed bitsPerSample from hardcoded to calculated for multi color traces
diff --git a/src/extension/internal/odf.cpp b/src/extension/internal/odf.cpp
index 16ff0f18f..1d3648048 100644
--- a/src/extension/internal/odf.cpp
+++ b/src/extension/internal/odf.cpp
@@ -101,27 +101,32 @@ class SVDMatrix
 {
 public:
 
-    SVDMatrix()
+    SVDMatrix() :
+        badval(0),
+        d(0),
+        rows(0),
+        cols(0),
+        size(0)
         {
-        d = (double *)0;
-        rows = cols = size = 0;
         }
 
-    SVDMatrix(unsigned int rowSize, unsigned int colSize)
+    SVDMatrix(unsigned int rowSize, unsigned int colSize) :
+        badval(0),
+        rows(rowSize),
+        cols(colSize),
+        size(rows * cols)
         {
-        rows = rowSize;
-        cols = colSize;
-        size = rows * cols;
         d = new double[size];
         for (unsigned int i=0 ; i<size ; i++)
             d[i] = 0.0;
         }
 
-    SVDMatrix(double *vals, unsigned int rowSize, unsigned int colSize)
+    SVDMatrix(double *vals, unsigned int rowSize, unsigned int colSize) :
+        badval(0),
+        rows(rowSize),
+        cols(colSize),
+        size(rows * cols)
         {
-        rows = rowSize;
-        cols = colSize;
-        size = rows * cols;
         d = new double[size];
         for (unsigned int i=0 ; i<size ; i++)
             d[i] = vals[i];
@@ -129,10 +134,16 @@ public:
 
     virtual ~SVDMatrix()
         {
-        delete d;
+        delete[] d;
+        d = 0;
         }
 
-    SVDMatrix(const SVDMatrix &other)
+    SVDMatrix(const SVDMatrix &other) :
+        badval(0),
+        d(0),
+        rows(0),
+        cols(0),
+        size(0)
         {
         assign(other);
         }
@@ -207,7 +218,10 @@ private:
      void assign(const SVDMatrix &other)
         {
         if (d)
-            delete d;
+            {
+            delete[] d;
+            d = 0;
+            }
         rows = other.rows;
         cols = other.cols;
         size = other.size;
@@ -259,9 +273,13 @@ public:
    @return     Structure to access U, S and V.
    */
 
-    SingularValueDecomposition (const SVDMatrix &mat)
+    SingularValueDecomposition (const SVDMatrix &mat) :
+        A(mat),
+        U(),
+        s(0),
+        s_size(0),
+        V()
         {
-        A = mat;
         calculate();
         }
 
@@ -284,8 +302,7 @@ public:
 
     /**
      *  Return the s[index] value
-     */
-    double getS(unsigned int index);
+     */    double getS(unsigned int index);
 
     /**
      * Two norm