From: Ben Pfaff <blp@gnu.org>
Date: Wed, 5 Sep 2007 04:49:41 +0000 (+0000)
Subject: (cmd_crosstabs): Free xtab and the structures that it points to, to
X-Git-Tag: v0.6.0~285
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=9bf438308e197f904dcf3a6c9cd9117f35e611cd;p=pspp-builds.git

(cmd_crosstabs): Free xtab and the structures that it points to, to
plug a memory leak.  Fixes bug #18315.
---

diff --git a/src/language/stats/ChangeLog b/src/language/stats/ChangeLog
index 55c4a8e3..e9ec8fcd 100644
--- a/src/language/stats/ChangeLog
+++ b/src/language/stats/ChangeLog
@@ -1,3 +1,8 @@
+2007-09-04  Ben Pfaff  <blp@gnu.org>
+
+	* crosstabs.q (cmd_crosstabs): Free xtab and the structures that
+	it points to, to plug a memory leak.  Fixes bug #18315.
+
 2007-08-15  Jason Stover  <jhs@wonko.gcsu.edu>
 
 	* regression.q (identify_indep_vars): Print an error if dependent
diff --git a/src/language/stats/crosstabs.q b/src/language/stats/crosstabs.q
index e57d9343..8adf044c 100644
--- a/src/language/stats/crosstabs.q
+++ b/src/language/stats/crosstabs.q
@@ -191,11 +191,16 @@ int
 cmd_crosstabs (struct lexer *lexer, struct dataset *ds)
 {
   int result = internal_cmd_crosstabs (lexer, ds);
+  int i;
 
   free (variables);
   pool_destroy (pl_tc);
   pool_destroy (pl_col);
 
+  for (i = 0; i < nxtab; i++)
+    free (xtab[i]);
+  free (xtab);
+
   return result;
 }