Open the X11 socket with close-on-exec flag

This saves the X11 connection from leaking into children processes.
On Linux, this is fully thread-safe using SOCK_CLOEXEC. On other
systems, there is a small race condition.

Signed-off-by: Julien Danjou <julien@danjou.info>

Support xcb_discard_reply

This function is useful for dynamic language garbage collectors. Frequently
a GC cycle may run before you want to block wainting for a reply.

This function is also marginally useful for libxcb apps that issue
speculative requests (eg. xlsclients).

Reviewed-by: Jamey Sharp <jamey@minilop.net>
Tested-by: Eamon Walsh <efw@eamonwalsh.com>
Signed-off-by: Peter Harris <pharris@opentext.com>

xcb_connect_to_fd: fix descriptor leak on memory error path

Signed-off-by: Julien Danjou <julien@danjou.info>

configure.ac: Fix a typo on the last commit.

Signed-off-by: Jeremy Huddleston <jeremyhu@freedesktop.org>

darwin: xnu doesn't support poll on ttys on the master side.

<rdar://problem/7360546>

Signed-off-by: Jeremy Huddleston <jeremyhu@freedesktop.org>

Tutorial code fix

Accessed elements of names[] after freeing them in the first example.

Signed-off-by: Julien Danjou <julien@danjou.info>

DRI2 extension requires xcb-proto 1.6 or newer

Signed-off-by: Rémi Cardona <remi@gentoo.org>
Signed-off-by: Julien Danjou <julien@danjou.info>

build: simplify extension building

Signed-off-by: Julien Danjou <julien@danjou.info>

Release libxcb 1.5

Signed-off-by: Julien Danjou <julien@danjou.info>

setsockopt(SO_KEEPALIVE) on TCP display connections.

This matches xtrans behaviour in SocketINETConnect, and makes it so apps
don't hang forever if their display dies.

Signed-off-by: Adam Jackson <ajax@redhat.com>

Add DRI2 support. (v2)

v2: Build fix from jcristau.

Fix typo in the tutorial.

Don't build docs if 'dot' is not found Signed-off-by: Paulo R. Zanoni <pzanoni@mandriva.com>

Fix check dependency

Bugzilla #21992

make -j check fails because the check-local rule gets executed before
the tests actually ran, so CheckLog*.xml doesn't exist.

Signed-off-by: Julien Danjou <julien@danjou.info>

Cygwin build fix: Add -no-undefined to libtool flags

-no-undefined is needed to tell libtool a shared library can be built
on platforms which require all references to be statisfied at link time.

Signed-off-by: Jon TURNEY <jon.turney@dronecode.org.uk>
Signed-off-by: Julien Danjou <julien@danjou.info>

Release libxcb 1.4

Add majorCode, minorCode and resourceID fields to X generic error

Fix precedence bug: wrong length for big-requests preceded by sync.

Also replace excessively clever use of bitwise OR with equivalent
addition.

Reported-by: Geoffrey Li <geoffrey@seitopos.com>
Signed-off-by: Jamey Sharp <jamey@minilop.net>
Signed-off-by: Josh Triplett <josh@joshtriplett.org>

Fix libxcb-randr version info

The SONAME shouldn't have been bumped in 1.3, only new symbols were
added.

Signed-off-by: Julien Danjou <julien@danjou.info>

Release libxcb 1.3

Disable Nagle on TCP socket

Signed-off-by: Julien Danjou <julien@danjou.info>

Store xcbproto version libxcb was compiled with

Signed-off-by: Julien Danjou <julien@danjou.info>

depends on recent xcb-proto and bump version of randr

Signed-off-by: Julien Danjou <julien@danjou.info>

Fix XDM-AUTHORIZATION-1 (bug #14202)

With this patch, we know use correctly the socket address or peer
address for authentication purpose.

Signed-off-by: Julien Danjou <julien@danjou.info>

util: remove useless strlen calls from decnet opening

Signed-off-by: Julien Danjou <julien@danjou.info>

util: merge common code for xcb_connect

Many code was duplicated between xcb_connect_to_display_with_auth_info
and xcb_connect(). We merge both, since the difference is just about the
xcb_auth_info_t pointer being supplied, or not.

Signed-off-by: Julien Danjou <julien@danjou.info>

util: open_abstract gets filelen as parameters

That saves us from a couple of strlen() calls.

Signed-off-by: Julien Danjou <julien@danjou.info>

auth: use snprintf() return value

That save us from a strlen().

Signed-off-by: Julien Danjou <julien@danjou.info>

auth: precompute authnameslen

Signed-off-by: Julien Danjou <julien@danjou.info>

darwin: Don't use poll() on versions of darwin before darwin10

Local socket connections do not work on hurd-i386

Local socket connections currently do not work on hurd-i386 because
xcb_auth calls getpeername() on the client socket, but hurd-i386 does
not implement anything in that case (I actually wonder what reasonable
value could be returned).  In such case the xcb code does not actually
need the peer name anyway.

Signed-off-by: Julien Danjou <julien@danjou.info>

use poll() instead of select() when available

Signed-off-by: Julien Danjou <julien@danjou.info>

kludgily hand-merged xid fixes

Signed-off-by: Bart Massey <bart@cs.pdx.edu>
Signed-off-by: Julien Danjou <julien@danjou.info>

Fix do_append() arguments

Signed-off-by: Julien Danjou <julien@danjou.info>

Copy full IPv4 mapping (Bug #20665)

Signed-off-by: Julien Danjou <julien@danjou.info>

Avoid name collisions between xidtype and enum.

These changes are necessary to build with latest xcb/proto.

Signed-off-by: Peter Harris <pharris@opentext.com>

Revert "Don't use enums in generated C code"

This commit broke xcb/util.

This reverts commit 9984b72888108a038d6b3f7dee374d17e26ef9e2.

Signed-off-by: Peter Harris <pharris@opentext.com>

Don't use enums in generated C code - use integer constants instead.

Signed-off-by: Peter Harris <pharris@opentext.com>

Release libxcb 1.2

Signed-off-by: Julien Danjou <julien@danjou.info>

Stop providing autogenerated files in tarball

Signed-off-by: Julien Danjou <julien@danjou.info>

Release libxcb 1.1.93

Signed-off-by: Julien Danjou <julien@danjou.info>

Shutup compiler warning about unused variable...

Apple: Apple launchd cleanup
Added --with-launchd option instead of just using __APPLE__
Fixed opening launchd fd when displayname=NULL

Apple: Enable support for launchd DISPLAY socket

Treat XIDs the same as other cardinal values.

This fixes a bug where c_client.py wasn't generating *_end functions,
but expected them to exist in order to find the subsequent list's start.

Signed-off-by: Peter Harris <peter.harris@hummingbird.com>

Release libxcb 1.1.92

Signed-off-by: Julien Danjou <julien@danjou.info>

Support handing off socket write permission to external code.

Libraries like Xlib, some XCB language bindings, and potentially others
have a common problem: they want to share the X connection with XCB. This
requires coordination of request sequence numbers.  Previously, XCB had an
Xlib-specific lock, and allowed Xlib to block XCB from making requests.
Now we've replaced that lock with a handoff mechanism, xcb_take_socket,
allowing external code to ask XCB for permission to take over the write
side of the socket and send raw data with xcb_writev.  The caller of
xcb_take_socket must supply a callback which XCB can call when it wants
the write side of the socket back to make a request.  This callback
synchronizes with the external socket owner, flushes any output queues if
appropriate, and then returns the sequence number of the last request sent
over the socket.

Commit by Josh Triplett and Jamey Sharp.
Handoff mechanism inspired by Keith Packard.

Track 64-bit sequence numbers internally.

External APIs that used 32-bit sequence numbers continue to do so.

Commit by Josh Triplett and Jamey Sharp.

Use sequence number ranges in pending replies

This allows optimizing adjacent pending replies with the same flags, and
will help support default flags for a range of future requests.

Commit by Josh Triplett and Jamey Sharp.

Inline _xcb_lock_io, _xcb_unlock_io, and _xcb_wait_io.

These functions are once again a single pthread call, so just make that
call directly.

Remove libxcb-xlib and xcbxlib.h.

build: fix configure.ac AC_DEFINE

Rather use AC_DEFINE_UNQUOTED and only once.

Signed-off-by: Julien Danjou <julien@danjou.info>

allow compile-time setting for XCB queue buffer size

Signed-off-by: Julien Danjou <julien@danjou.info>

Remove duplicate XCB_EXTENSION calls for Composite extension

fix tiny memory leak in read_packet (leak only happens when malloc returns NULL so it's very rare)

Signed-off-by: Julien Danjou <julien@danjou.info>

Added generation of extern "C" for compatibility with C++

The auto-generated header files now include an extern "C"
declaration for compatibility with C++.

Signed-off-by: Julien Danjou <julien@danjou.info>

Add support for the abstract socket namespace under Linux

Based on same in Xtrans.

Signed-off-by: Julien Danjou <julien@danjou.info>

Fix some fd leaks in _xcb_open_*()

Signed-off-by: Julien Danjou <julien@danjou.info>

fixed overly aggressive warning about fixed field following variable

added small fix to support trailing fixed fields; also warning for non-pad fixed fields

factorize m4 macros and add one to set X extensions

Signed-off-by: Julien Danjou <julien@danjou.info>

Initialize all fields of addrinfo

Signed-off-by: Julien Danjou <julien@danjou.info>

Use ifdef instead of if for defined value

Signed-off-by: Julien Danjou <julien@danjou.info>

Set namelen unsigned

Signed-off-by: Julien Danjou <julien@danjou.info>

Rename index to idx to avoid shadowing

Signed-off-by: Julien Danjou <julien@danjou.info>

Use a signed size in read_block()

Signed-off-by: Julien Danjou <julien@danjou.info>

Use unsigned to compare and rename sync

- i must be unsigned to be compare in the loop
- sync shadow global sync() function

Signed-off-by: Julien Danjou <julien@danjou.info>

Fix htonl() arg & convert sizeof() to signed

Signed-off-by: Julien Danjou <julien@danjou.info>

initialize global_id to 0

Signed-off-by: Julien Danjou <julien@danjou.info>

Bump to 1.1.91.

Make EXTHEADERS, EXTSOURCES, EXTENSION_XML unconditional of configure flags.

yay, make distcheck works now even when some extensions are disabled.

Acked-by: Peter Hutterer <peter.hutterer@who-t.net>

fix FreeBSD support

The GNU/kFreeBSD (and BSDs in general) have a different
layout of struct sockaddr, sockaddr_in, sockaddr_un ...

The first member do not have to be "sa_family",
they also have "sa_len" field.

Signed-off-by: Julien Danjou <julien@danjou.info>

Bump to 1.1.90.

Fix variable declaration formatting

Add xcb_ge_event_t and handling for long events.

GenericEvent can be more than 32 bytes long. Ensure that the required data is
pulled off the wire and tack it onto the event.

Due to the structure of the xcb_generic_event_t, the data is appended AFTER
the full_sequence field.

Fix libxcb/src compile with srcdir != builddir.

Replace a stray c-client.xsl in the libxcb SOURCES.  Fixes make distcheck.

Use the python install path from xcb-xproto.pc to locate the xcbgen package.

Add mention of PYTHONPATH if xcbgen cannot be found.

Add Python parser C language-dependent part.

Add SELinux extension support, disabled by default.

Add XInput extension support, disabled by default.

Inputs to AC_CONFIG_FILES are automatically distributed.
Hence, it is not necessary to explicitly add them to EXTRA_DIST.

Generated the configure.ac and Makefile.am's in libxcb with the idea of
making each extension library individually selectable for build.

Signed-off-by: Eamon Walsh <ewalsh@tycho.nsa.gov>

Merge branch 'master' of git+ssh://git.freedesktop.org/git/xcb/libxcb

make IPv6 optional

Remove file that wasn't meant to be committed.

Merge branch 'master' of git+ssh://git.freedesktop.org/git/xcb/libxcb

Add comment noting the requirement to free replies when finished.

Add generated comment noting requirement to free replies after use.

Release libxcb 1.1

Revert "Generate error constants as XCB_BAD_*, similar to Xlib."

Since several extensions named their errors like "BadFoo", this patch
results in names like XCB_EXT_BAD_BAD_FOO, which is really awful. Those
extensions are already kind of awful, as they produce structure names
like xcb_ext_bad_foo_error_t, which is redundant.

A patch that removes "Bad" from the XML extension descriptions, while
maintaining API and ABI compatibility in XCB, is needed before this
patch can be released.

This reverts commit 158c9b6ba18b39f424bd524fceb66f3fec0d1616.

Don't hold the xlib-xcb lock while sleeping: that allows deadlock.

With this patch, `ico -threads 2` runs without deadlock.

Many thanks to Christoph Pfister <christophpfister@gmail.com> for
pointing out the problem, providing detailed analyses, explaining it to
me repeatedly until I understood what was going on, and proposing and
reviewing possible solutions.

Signed-off-by: Jamey Sharp <jamey@minilop.net>
Acked-by: Christoph Pfister <christophpfister@gmail.com>

Factor pthread_cond_wait(iolock) to _xcb_wait_io.

This parallels the _xcb_lock_io and _xcb_unlock_io factoring.

Don't abort() on locking assertions if LIBXCB_ALLOW_SLOPPY_LOCK is set.

But do still print a full backtrace, on platforms where that's
supported.

This commit follows the spirit of Novell's libxcb-sloppy-lock.diff.

I strongly opposed proposals like this one for a long time. Originally I
had a very good reason: libX11, when compiled to use XCB, would crash
soon after a locking correctness violation, so it was better to have an
informative assert failure than a mystifying crash soon after.

It took some time for me to realize that I'd changed the libX11
implementation (for unrelated reasons) so that it could survive most
invalid locking situations, as long as it wasn't actually being used
from multiple threads concurrently.

The other thing that has changed is that most of the code with incorrect
locking has now been fixed. The value of the assert is accordingly
lower.

However, remaining broken callers do need to be fixed. That's why libXCB
will still noisily print a stacktrace (if possible) on each assertion
failure, even when assert isn't actually invoked to abort() the program;
and that's why aborting is still default. This environment variable is
provided only for use as a temporary workaround for broken applications.

Signed-off-by: Jamey Sharp <jamey@minilop.net>
Acked-by: Josh Triplett <josh@freedesktop.org>

Allow unix:<screen> style display names again.

https://bugzilla.novell.com/show_bug.cgi?id=289007
This notion is used in a lot of scripts.

Send locking assertion backtraces to stderr.  Improve the heading on the backtrace.

Print backtraces in case an assert fails inside xlib/xcb.

As you know there are some nasty libs / apps doing locking
incorrectly. In order to improve the information given to the user
when he encounters such a situation (people don't run apps in gdb
normally) I created the patch attached.
It's very non-intrusive (and affects only xlib/xcb, Josh told me on
irc that it could be useful for other areas too, personally I don't
think that it's really needed at other places ...).

Some same outputs and the discussion of them:

    lxuser@pdln:/tmp$ ./main
    Got a backtrace:
    #0 /tmp/usr/lib/libxcb-xlib.so.0 [0xb7f9d728]
    #1 /tmp/usr/lib/libxcb-xlib.so.0(xcb_xlib_unlock+0x31) [0xb7f9d861]
    #2 ./test.so(function_a+0x11) [0xb7f9f3fd]
    #3 ./test.so(function_b+0x11) [0xb7f9f410]
    #4 ./main [0x80484a7]
    #5 /lib/libc.so.6(__libc_start_main+0xdc) [0xb7e60ebc]
    #6 ./main [0x80483f1]
    main: xcb_xlib.c:82: xcb_xlib_unlock: Assertion `c->xlib.lock' failed.
    Aborted

That's kinda the normal situation.

    lxuser@pdln:/tmp$ ./main
    Got a backtrace:
    #0 /tmp/usr/lib/libxcb-xlib.so.0 [0xb7f90728]
    #1 /tmp/usr/lib/libxcb-xlib.so.0(xcb_xlib_unlock+0x31) [0xb7f90861]
    #2 /tmp/test.so [0xb7f923cd]
    #3 /tmp/test.so(function_b+0x11) [0xb7f923e0]
    #4 ./main [0x80484ab]
    #5 /lib/libc.so.6(__libc_start_main+0xdc) [0xb7e53ebc]
    #6 ./main [0x80483f1]
    main: xcb_xlib.c:82: xcb_xlib_unlock: Assertion `c->xlib.lock' failed.
    Aborted

There are two possible reasons that the name doesn't appear in #2:
a) a hidden symbol or a symbol with statical linkage in a library
b) a symbol in an app not compiled with -rdynamic.
But in both cases you still know _where_ the caller is.

Note that in this example test.so was compiled with
-fomit-frame-pointer; this isn't an issue as _one_ (= the caller)
stack trace is still valid (as long as you don't have the insane idea
to compile xcb with -fo-f-p).

Another issue that may appear is "tail call elimination" (some entries
are mysteriously missing; this is quite ugly, but you still get enough
information so that you can do something useful with the issue e.g. by
disassembling the relevant parts with gdb).

Signed-off-by: Jamey Sharp <jamey@minilop.net>

xcb_poll_for_event: Return already-read events before read(2)ing again.

Merge branch 'master' of git://anongit.freedesktop.org/git/xcb/libxcb