On 02/10/2013 12:39 PM, Alan Coopersmith wrote:
Reported with other leaks found by cppcheck in bugzilla #50281
https://bugs.freedesktop.org/show_bug.cgi?id=50281
Signed-off-by: Alan Coopersmith <[email protected]>
---
glx/glxdri2.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/glx/glxdri2.c b/glx/glxdri2.c
index b26e501..e014529 100644
--- a/glx/glxdri2.c
+++ b/glx/glxdri2.c
@@ -939,6 +939,7 @@ __glXDRIscreenProbe(ScreenPtr pScreen)
&screen->fd, &driverName, &deviceName)) {
LogMessage(X_INFO,
"AIGLX: Screen %d is not DRI2 capable\n", pScreen->myNum);
+ free(screen);
return NULL;
Couldn't this just 'goto handle_error' instead? As far as I can tell,
that should handle things correctly (but would result in an additional
message in the log).
}
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
