Insight problem when environment variable DISPLAY=xxx.xxx.xxx.xxx:0.

Insight problem when environment variable DISPLAY=xxx.xxx.xxx.xxx:0.

[Nunzio Raciti]

[-- Attachment #1: Type: text/plain, Size: 523 bytes --]

Hi All,

I discovered a bug on Insight, probably exposed by new version of libc,
when the environment variable "DISPLAY" is set with a value like
"xxx.xxx.xxx.xxx:0."

The problem occurs exiting from the GUI without open other windows:
1- $ insight
2- check that there aren't other open windows (eg. console, stack....)
3- exit from the GUI (eg. file->exit)
4- if you are using glibc => 2.3.4

*** glibc detected *** double free or corruption (!prev): 0xXXXXXXX ***
Aborted

Attached is my proposal of patch.

Ciao,
Nunzio

[-- Attachment #2: insight_var_DISPLAY.patch --]
[-- Type: text/x-diff, Size: 641 bytes --]

diff U3 C:/Temp/insight-weekly-CVS-7.0.50-20091130/src/tk/generic/tkWindow.c C:/Temp/insight-patched/src/tk/generic/tkWindow.c
--- C:/Temp/insight-weekly-CVS-7.0.50-20091130/src/tk/generic/tkWindow.c	Tue Jan 21 22:24:46 2003
+++ C:/Temp/insight-patched/src/tk/generic/tkWindow.c	Wed May 25 09:44:50 2011
@@ -450,6 +450,11 @@
     if ((*p == '.') && (p[1] != '\0')) {
 	length = p - screenName;
 	screenId = strtoul(p+1, (char **) NULL, 10);
+    } else
+          /* Cover the "xxx.xxx.xxx.xxx:0." case */
+          if ((*p == '.') && (p[1] == '\0')) {
+	      length = p - screenName;
+	      screenId = 0;
     }
 
     /*