libXt-1.0.8: fix dereferenc in TMprint.c

[walter harms]

hi list;
the current codes checks for
 eventWidget == NULL after accessing
eventWidget->core.tm.proc_table

now it checks for eventWidget before access eventWidget->core.tm.proc_table.


The org. patch is attacehd since i guess the tabs will be mangeld.

re,
 wh

Signed-off-by: walter harms <wha...@bfs.de>

--- libXt-1.0.8/src/TMprint.c.org       2010-09-21 23:42:08.000000000 +0200
+++ libXt-1.0.8/src/TMprint.c   2010-09-21 23:44:44.000000000 +0200
@@ -782,12 +782,16 @@
     PrintRec           stackPrints[STACKPRINTSIZE];
     PrintRec           *prints;
     TMShortCard                numPrints, maxPrints;
-    TMBindData bindData = (TMBindData) eventWidget->core.tm.proc_table;
+    TMBindData bindData ;
     TMComplexBindProcs complexBindProcs;

     if ((eventWidget == NULL) ||
-       ((xlations = eventWidget->core.tm.translations) == NULL) ||
-       (bindData->simple.isComplex == False))
+       (eventWidget->core.tm.translations == NULL) )
+      return;
+
+    xlations = eventWidget->core.tm.translations;
+    bindData = (TMBindData) eventWidget->core.tm.proc_table;
+    if (bindData->simple.isComplex == False)
       return;

     sb->current = sb->start = __XtMalloc((Cardinal)1000);

--- libXt-1.0.8/src/TMprint.c.org	2010-09-21 23:42:08.000000000 +0200
+++ libXt-1.0.8/src/TMprint.c	2010-09-21 23:44:44.000000000 +0200
@@ -782,12 +782,16 @@
     PrintRec		stackPrints[STACKPRINTSIZE];
     PrintRec		*prints;
     TMShortCard		numPrints, maxPrints;
-    TMBindData	bindData = (TMBindData) eventWidget->core.tm.proc_table;
+    TMBindData	bindData ;
     TMComplexBindProcs	complexBindProcs;
 
     if ((eventWidget == NULL) ||
-	((xlations = eventWidget->core.tm.translations) == NULL) ||
-	(bindData->simple.isComplex == False))
+	(eventWidget->core.tm.translations == NULL) )
+      return;
+
+    xlations = eventWidget->core.tm.translations;
+    bindData = (TMBindData) eventWidget->core.tm.proc_table;
+    if (bindData->simple.isComplex == False)
       return;
 
     sb->current = sb->start = __XtMalloc((Cardinal)1000);

_______________________________________________
xorg-devel@lists.x.org: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel