From f0794b328873c23fe32dddc7f91de632939acac3 Mon Sep 17 00:00:00 2001
From: rizzo <rizzo@f38db490-d61c-443f-a65b-d21fe96a405b>
Date: Mon, 14 Jul 2008 16:50:54 +0000
Subject: [PATCH] free memory used by the x11 grabber when closing it.

git-svn-id: http://svn.digium.com/svn/asterisk/trunk@130733 f38db490-d61c-443f-a65b-d21fe96a405b
---
 channels/vgrabbers.c | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/channels/vgrabbers.c b/channels/vgrabbers.c
index 470a19f38..4afc12f61 100644
--- a/channels/vgrabbers.c
+++ b/channels/vgrabbers.c
@@ -69,6 +69,8 @@ struct grab_x11_desc {
 	struct fbuf_t	b;		/* geometry and pointer into the XImage */
 };
 
+static void *grab_x11_close(void *desc);	/* forward declaration */
+
 /*! \brief open the grabber.
  * We use the special name 'X11' to indicate this grabber.
  */
@@ -127,12 +129,7 @@ static void *grab_x11_open(const char *name, struct fbuf_t *geom, int fps)
 	return v;
 
 error:
-	/* XXX maybe XDestroy (v->image) ? */
-	if (v->dpy)
-		XCloseDisplay(v->dpy);
-	v->dpy = NULL;
-	ast_free(v);
-	return NULL;
+	return grab_x11_close(v);
 }
 
 static struct fbuf_t *grab_x11_read(void *desc)
@@ -170,7 +167,8 @@ static void *grab_x11_close(void *desc)
 {
 	struct grab_x11_desc *v = desc;
 
-	XCloseDisplay(v->dpy);
+	if (v->dpy)
+		XCloseDisplay(v->dpy);
 	v->dpy = NULL;
 	v->image = NULL;
 	ast_free(v);