psd->data.bitmap is never set, but it's better to make sure we never
try to free() an XID.

Signed-off-by: Peter Harris <phar...@opentext.com>
---
 xts5/src/xim/response.c |    6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/xts5/src/xim/response.c b/xts5/src/xim/response.c
index b00ccdd..c554acb 100644
--- a/xts5/src/xim/response.c
+++ b/xts5/src/xim/response.c
@@ -455,8 +455,8 @@ void xim_response_pop_cb(pstk)
                                }
                                else
                                {
-                                       if(psd->data.bitmap != NULL)
-                                               free(psd->data.bitmap);
+                                       if(psd->data.bitmap != None)
+                                               XFreePixmap(Dsp, 
psd->data.bitmap);
                                }
                                break;
                        default:
@@ -1133,7 +1133,7 @@ static Pixmap read_pixmap()
                parse_skwhite(&presponse);
        }
 
-       return(NULL);
+       return(None);
 }
 
 /* read the callback data from a response file for */
-- 
1.7.10.4

_______________________________________________
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

Reply via email to