Re: [PATCH pixman] '+=' is not a valid POSIX shell operator.

2013-06-09 Thread Matthieu Herrb
On Sat, Jun 08, 2013 at 10:22:25AM -0700, Matt Turner wrote: On Sat, Jun 8, 2013 at 9:07 AM, Matthieu Herrb matthieu.he...@laas.fr wrote: Signed-off-by: Matthieu Herrb matthieu.he...@laas.fr --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git

Re: [PATCH xaw3d 1/2] Restore recording of build options in Xaw3dP.h

2013-06-09 Thread Javier Pello
I'm inclined to push these, but they cause redefinition warnings during compilation: Well, I think I tested the patch and did not see those warnings, but now I am getting them as well. Since they are added to the .h file, perhaps they should be removed from XAW3D_CPPFLAGS and xaw3d.pc's

Re: [PATCH:libpciaccess 3/3] Update NetBSD support. From Michael Lorenz macal...@netbsd.org.

2013-06-09 Thread Alan Coopersmith
On 06/ 8/13 11:28 AM, Thomas Klausner wrote: On Fri, Jun 07, 2013 at 08:39:52PM -0700, Alan Coopersmith wrote: On 06/ 2/13 01:38 PM, Thomas Klausner wrote: --- configure.ac | 4 +- src/netbsd_pci.c | 471 +++ 2 files changed, 342

Re: [PATCH:libpciaccess 3/3] Update NetBSD support. From Michael Lorenz macal...@netbsd.org.

2013-06-09 Thread Thomas Klausner
On Sun, Jun 09, 2013 at 09:32:37AM -0700, Alan Coopersmith wrote: On 06/ 8/13 11:28 AM, Thomas Klausner wrote: On Fri, Jun 07, 2013 at 08:39:52PM -0700, Alan Coopersmith wrote: On 06/ 2/13 01:38 PM, Thomas Klausner wrote: --- configure.ac | 4 +- src/netbsd_pci.c | 471

[PATCH videoproto] Xvproto: fix xvEncodingInfo to match actual wire protocol

2013-06-09 Thread Julien Cristau
The padding is *before* the rate field, so the rate is placed on a 32bit boundary. This change adds explicit padding between height and rate, and removes extraneous padding after the rate field, which the server never sent and xlib never read. This changes sizeof(xvEncodingInfo). Hopefully

[PATCH v3 0/2] libX11: check size of GetReqExtra after XFlush

2013-06-09 Thread Kees Cook
Thanks for the feedback! I've split the patch into the two halves and updated return values, etc. I'm not too familiar with the Data API, so I left things as-is for the time-being. If the second patch isn't right, hopefully the first is still useful. :) Thanks, -Kees

[PATCH 2/2] libX11: check req when calling GetReqExtra

2013-06-09 Thread Kees Cook
This fixes the two callers of GetReqExtra to check that req is non-NULL to avoid crashing now that GetReqExtra does internal bounds-checking on the resulting buffer sizes. Additionally updates comment describing return values to use names instead of only literal values. Signed-off-by: Kees Cook

[PATCH 1/2] libX11: check size of GetReqExtra after XFlush

2013-06-09 Thread Kees Cook
Two users of GetReqExtra pass arbitrarily sized allocations from the caller (ModMap and Host). Adjust _XGetRequest() (called by the GetReqExtra macro) to double-check the requested length and invalidate req when this happens. Users of GetReqExtra passing lengths greater than the Xlib buffer size

Re: [PATCH:libpciaccess 3/3] Update NetBSD support. From Michael Lorenz macal...@netbsd.org.

2013-06-09 Thread Matthieu Herrb
On Sun, Jun 02, 2013 at 10:38:41PM +0200, Thomas Klausner wrote: --- configure.ac | 4 +- src/netbsd_pci.c | 471 +++ 2 files changed, 342 insertions(+), 133 deletions(-) Reviewed-by: Matthieu Herrb matthieu.he...@laas.fr diff