[PATCH:libXxf86vm 3/4] Unlock display before returning alloc error in XF86VidModeGetAllModeLines()

Peter Hutterer peter.hutterer at who-t.net
Sun Apr 14 18:55:16 PDT 2013


On Sat, Apr 13, 2013 at 06:48:20PM -0700, Alan Coopersmith wrote:
> Signed-off-by: Alan Coopersmith <alan.coopersmith at oracle.com>
> ---
>  src/XF86VMode.c |    2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/src/XF86VMode.c b/src/XF86VMode.c
> index 28c79c1..76276b6 100644
> --- a/src/XF86VMode.c
> +++ b/src/XF86VMode.c
> @@ -322,6 +322,8 @@ XF86VidModeGetAllModeLines(Display* dpy, int screen, int* modecount,
>              _XEatData(dpy, (rep.modecount) * sizeof(xXF86OldVidModeModeInfo));
>  	else
>              _XEatData(dpy, (rep.modecount) * sizeof(xXF86VidModeModeInfo));
> +	UnlockDisplay(dpy);
> +	SyncHandle();

looks like some indentation mishap here, otherwise series
Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>

Cheers,
   Peter


>          return False;
>      }
>      mdinfptr = (XF86VidModeModeInfo *) (
> -- 
> 1.7.9.2
> 
> _______________________________________________
> xorg-devel at lists.x.org: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
> 


More information about the xorg-devel mailing list