Don't bother freeing import dialog memory. Thus avoid crashes.
authorJohn Darrington <john@darrington.wattle.id.au>
Sat, 29 Aug 2015 12:42:29 +0000 (14:42 +0200)
committerJohn Darrington <john@darrington.wattle.id.au>
Sat, 29 Aug 2015 12:42:29 +0000 (14:42 +0200)
A number of users have complained that the import dialog assistant
crashes on exit.  This seems to be because some callback is accessing
the data after it has been freed.

This change "fixes" that by simply not ever freeing the data.  This
will cause a memory leak.  However during my investigations, I have
found that this dialog leaks so much anyway that these few bytes are
hardly significant.

A proper fix will hopefully be forthcoming.

Affects bug #45805

src/ui/gui/text-data-import-dialog.c

index 1527028c28ec06934244c0ecc065401e6f1b88ea..34127492fa9393ec03a7a157342cd07d71360fd1 100644 (file)
@@ -138,10 +138,6 @@ text_data_import_assistant (PsppireDataWindow *dw)
       destroy_formats_page (ia);
       destroy_separators_page (ia);
     }
-
-  destroy_assistant (ia);
-  destroy_file (ia);
-  free (ia);
 }
 
 /* Emits PSPP syntax to S that applies the dictionary attributes