Fix crash on OK after Save Changes in Tags and Comment popup.
authorH.G. Muller <h.g.muller@hccnet.nl>
Thu, 21 Jul 2011 17:02:01 +0000 (19:02 +0200)
committerArun Persaud <arun@nubati.net>
Sun, 24 Jul 2011 05:33:13 +0000 (22:33 -0700)
The global variables tagsText and commentText were used to pass
a string obtained from a widget (and thus not malloc'ed), which
caused a crash when the GenericReadout tried to use those variables
for the new value. Fixed by using GenericReadout (which does malloc)
to read out the text,in stead of GetWidgetText.

xoptions.c

index bafa8bb..437ebae 100644 (file)
@@ -1395,7 +1395,7 @@ int NewComCallback(int n)
 
 void SaveChanges(int n)
 {
-    GetWidgetText(&currentOption[0], &commentText);
+    GenericReadout(0);
     ReplaceComment(commentIndex, commentText);
 }
 
@@ -1446,7 +1446,7 @@ int NewTagsCallback(int n)
 
 void changeTags(int n)
 {
-    GetWidgetText(&currentOption[1], &tagsText);
+    GenericReadout(1);
     if(bookUp) SaveToBook(tagsText); else
     ReplaceTags(tagsText, &gameInfo);
 }