[PATCH xinput v2] Fix XRRCrtcInfo memory leaks
Peter Hutterer
peter.hutterer at who-t.net
Wed Feb 8 08:20:04 PST 2012
crtc_info isn't used until later, move down to the block it is used to avoid
leaking.
Reported-by: Aaron Plattner <aplattner at nvidia.com>
Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
---
src/transform.c | 5 +++--
1 files changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/transform.c b/src/transform.c
index cffc3af..6a31c83 100644
--- a/src/transform.c
+++ b/src/transform.c
@@ -142,7 +142,6 @@ map_output_xrandr(Display *dpy, int deviceid, const char *output_name)
int rc = EXIT_FAILURE;
XRRScreenResources *res;
XRROutputInfo *output_info;
- XRRCrtcInfo *crtc_info;
res = XRRGetScreenResources(dpy, DefaultRootWindow(dpy));
@@ -152,7 +151,6 @@ map_output_xrandr(Display *dpy, int deviceid, const char *output_name)
if (!output_info->crtc || output_info->connection != RR_Connected)
continue;
- crtc_info = XRRGetCrtcInfo (dpy, res, output_info->crtc);
if (strcmp(output_info->name, output_name) == 0)
{
found = 1;
@@ -163,11 +161,14 @@ map_output_xrandr(Display *dpy, int deviceid, const char *output_name)
/* crtc holds our screen info, need to compare to actual screen size */
if (found)
{
+ XRRCrtcInfo *crtc_info;
Matrix m;
matrix_set_unity(&m);
+ crtc_info = XRRGetCrtcInfo (dpy, res, output_info->crtc);
set_transformation_matrix(dpy, &m, crtc_info->x, crtc_info->y,
crtc_info->width, crtc_info->height);
rc = apply_matrix(dpy, deviceid, &m);
+ XRRFreeCrtcInfo(crtc_info);
} else
printf("Unable to find output '%s'. "
"Output may not be connected.\n", output_name);
--
1.7.7.5
More information about the xorg-devel
mailing list