Re: [PATCH:libX11] Bug 93184: read_EncodingInfo invalid free

2015-12-04 Thread walter harms
Am 04.12.2015 08:30, schrieb Alan Coopersmith: > Free the correct bits of memory if we run out and need to unwind > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=93184 > Signed-off-by: Alan Coopersmith > --- > modules/om/generic/omGeneric.c | 7 ---

Re: [PATCH:libX11] Bug 93184: read_EncodingInfo invalid free

2015-12-04 Thread Alan Coopersmith
On 12/ 4/15 12:05 AM, walter harms wrote: Am 04.12.2015 08:30, schrieb Alan Coopersmith: @@ -1895,7 +1895,8 @@ read_EncodingInfo( len = strlen(buf); font_data->name = Xmalloc(len + 1); if (font_data->name == NULL) { -Xfree(font_data); +

[PATCH:libX11] Plug additional _XDefaultOpenIM memory leaks in error paths

2015-12-04 Thread Alan Coopersmith
Rework code to store allocations directly into XIM struct instead of temporary local variables, so we can use _XCloseIM to unwind instead of duplicating it, and consistently jump to error handler on failure, instead of sometimes leaking and sometimes freeing. Signed-off-by: Alan Coopersmith

Re: [PATCH xfree86 v2] systemd-logind.c: don't parse VT settings for non-seat0 X servers

2015-12-04 Thread Hans de Goede
Hi, On 03-12-15 11:05, Laércio de Sousa wrote: Since non-seat0 X servers no longer touch VTs, I believe these settings are unnecessary. Signed-off-by: Laércio de Sousa Thanks, you're right keeptty is not used when !ServerIsNotSeat0(): Reviewed-by:

Re: [PATCH v3 06/10] kdrive: add options to set default XKB properties

2015-12-04 Thread Laércio de Sousa
2015-04-05 3:22 GMT-03:00 Jeremy Huddleston Sequoia : > While you're at it, would you mind updating Xephyr to support matching the > parent server's keyboard layout (similar to what Xnest does, cf: > 83fef4235db86343477b4ec9858c6ba35e1aa7d9)? I never got around to doing >

[PATCH v2] xserver: Fix configure.ac check for libsystemd/-daemon

2015-12-04 Thread Bob Ham
The configure script looks for the libsystemd-daemon pkg-config module. If the configure script finds it, the script will add libsystemd-daemon to a list of modules which are used to consolidate CFLAGS and LIBS. The check for libsystemd-daemon was altered to fallback to libsystemd if

Re: [PATCH v3 06/10] kdrive: add options to set default XKB properties

2015-12-04 Thread Daniel Stone
Hi, On 4 December 2015 at 11:37, Laércio de Sousa wrote: > 2015-04-05 3:22 GMT-03:00 Jeremy Huddleston Sequoia : >> While you're at it, would you mind updating Xephyr to support matching the >> parent server's keyboard layout (similar

Re: Deliver input events only to window owner

2015-12-04 Thread Keith Packard
Peter Hutterer writes: > the main problem is: I don't know what we'll break with this. I really don't > know who uses raw keyboard events, or if anyone does. We'll clearly have to leave people with the option to re-enable whatever we disable for the sake of security.

Re: [PATCH] xwayland: Group multiple cursor buffers per shm pool

2015-12-04 Thread Kristian Høgsberg
On Tue, Dec 1, 2015 at 7:48 AM, Rui Matos wrote: > Each shm pool implies a file descriptor which means that currently, we > can quickly exhaust the available FDs since we create a shm pool per > cursor buffer. Instead, this patch creates shm pools big enough to > contain