Fix bug in WB combobox readout
authorH.G. Muller <h.g.muller@hccnet.nl>
Thu, 9 Feb 2012 11:51:25 +0000 (12:51 +0100)
committerH.G. Muller <h.g.muller@hccnet.nl>
Thu, 9 Feb 2012 11:51:25 +0000 (12:51 +0100)
The previous patch exposed a latent bug in the readout of the generic popup.

winboard/wsettings.c

index 10bb427..5098d54 100644 (file)
@@ -387,7 +387,7 @@ GetOptionValues(HWND hDlg, ChessProgramState *cps, Option *optionList)
                if(!success) break;\r
                new = -1;\r
                for(k=0; k<optionList[j].max; k++) {\r
-                   if(!strcmp(choices[k], newText)) new = k;\r
+                   if(choices[k] && !strcmp(choices[k], newText)) new = k;\r
                }\r
                if(!cps && new > 0) {\r
                    if(*(char**)optionList[j].target) free(*(char**)optionList[j].target);\r