test-privserver-client: wait for server to die between iterations

This fixes a race condition: the server exits while the client continues
to the next iteration. If the server wins, the test passes. If the client
wins, it sends a message to the dying service, never gets a reply, and the
test fails.

My branch to refactor the main loop for fd.o #23194 seems to make the
client more likely to win this race, resulting in intermittent test
failures.

This is an instance of the general problem described by fd.o #11454.

test_server_shutdown: disconnect the server before causing it to be freed

Otherwise we die with an assertion failure.

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33277

test_server_setup: allocate two server-data blobs, for watches and timeouts

This is similar to how ConnectionData works. Without this change, we
deserve to segfault: when the first set of callbacks (either watches or
timeouts) is cleaned up, we unref the server and loop, and free sd;
when the second set of callbacks is cleaned up, we use-after-free sd,
the server and the loop, then double-free sd.

However, due to fd.o #33277 we don't even get that far, because we've
already died with an assertion failure.

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33277

test_server_setup: on OOM while setting up watch/timeout, don't leak sd

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33128
Reviwed-by: Colin Walters <walters@verbum.org>

_dbus_read_credentials_socket: support OpenBSD

OpenBSD SO_PEERCRED is nearly the same as Linux SO_PEERCRED, but the
struct is named differently. Sigh.

Based on a patch from 'ajacoutot'.

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=32542

fix bad DBusMessageIter doc example

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=11190
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

systemd: ensure that the system bus is unlikely to be killed on OOM

_dbus_listen_tcp_socket: unify cleanup for EADDRINUSE and other failures

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=29881

_dbus_listen_tcp_socket: free listen_fd in the error case

Origin: vendor, Maemo
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=29881
Bug-NB: NB#180486
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Fix failure in configure test for abstract sockets

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=29895
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

clean up credentials on OOM in process_config_first_time_only

Origin: vendor, Maemo
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33128
Bug-NB: NB#180486
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

dbus-cleanup-sockets: free list of socket entries

Origin: vendor, Maemo
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33128
Bug-NB: NB#180486
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

remove obviously-redundant code from list_concat_new

Origin: vendor, Maemo
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33128
Bug-NB: NB#180486
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

handle failure to register inotify shutdown function

Origin: vendor, Maemo
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33128
Bug-NB: NB#180486
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

bus_matchmaker_new: free matchmaker itself, on OOM

Origin: vendor, Maemo
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33128
Bug-NB: NB#180486
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

dbus-nonce: check results of _dbus_string_init for OOM

Origin: vendor, Maemo
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33128
Bug-NB: NB#180486
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Merge branch 'tmp'

Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=32854

Remove vestigial support for $INTLLIBS

We no longer call AM_GLIB_GNU_GETTEXT or use gettext directly.

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=25306
Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

Add autolaunch test to .gitignore

cmake/cross-compile.sh: use mktemp(1) to avoid a symlink attack in /tmp

Fix the check for -Wfloat-equal

configure.in: use AC_TRY_COMPILE to avoid a symlink attack in /tmp during compilation

Free session file early in dbus-launch

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=29881
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Verify that getsockname succeeded in _dbus_listen_tcp_socket

Bug: https://bugs.freedesktop.org//show_bug.cgi?id=29881
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Handle failure to allocate error message in _read_subprocess_line_argv

Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=29881

Check return value of XGetWindowProperty in x11_get_address

Reviewed-by: Thiago Macieira <thiago@kde.org>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=29881

Annotate dbus_malloc*() functions with __attribute__ malloc and __attribute__ alloc_size

This improves optimization as well catches buffer overflows at compile time.

code like:

foo = dbus_malloc(2);
strcpy(foo, "epicfail");

will emit:

"warning ... will always overflow the destination buffer..."

Signed-off-by: Cristian Rodríguez <cristian.rodriguez@opensuse.org>
Reviewed-by: Thiago Macieira <thiago@kde.org>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=32710

connection: correct HAVE_DECL_MSG_NOSIGNAL guard

Helpfully, AC_CHECK_DECLS (which configure.in uses to define
HAVE_DECL_MSG_NOSIGNAL) behaves differently to every similar AC_CHECK_*
macro and, rather than producing #undef HAVE_DECL_MSG_NOSIGNAL if the
given symbol is not found, instead produces #define
HAVE_DECL_MSG_NOSIGNAL 0.

Mike McQuaid's patch fixes the uses of this constant in sysdeps-unix;
with this patch, all code is guarded consistently and correctly.

Fix error condition in _dbus_pipe_close

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=32264
Reviewed-by: Colin Walters <walters@verbum.org>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Make _dbus_system_logv actually exit with DBUS_SYSTEM_LOG_FATAL

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=32262
Bug-NB: NB#180486
Reviewed-by: Thiago Macieira <thiago@kde.org>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Windows fix: MSVC 2010 has errno.h.

Avoid live lock in Windows (CE) under some situations due to unfair condition variables.

Use absolute session service dir path in bus-test on windows.

Keep in sync with autotools.

sysdeps-unix: use #if for HAVE_DECL_MSG_NOSIGNAL

Signed-off-by: Will Thompson <will.thompson@collabora.co.uk>

Mention maintainer-upload-docs in the release process

Bump development version to 1.4.3.

The last release should have been 1.4.2; sorry.

Fix configure doc: libxml may be named libxml2 on some systems.

Keep in sync with automake.

Dist additional README's too.

Post-release version bump to 1.4.2

Bump LT_REVISION for this new release.

NEWS for 1.4.1

CVE 2010-4352: Reject deeply nested variants

Add DBUS_INVALID_NESTED_TOO_DEEPLY validity problem and a test that
should generate it.

Previously, we rejected deep nesting in the signature, but
variants allow dynamic message nesting, conditional only
on the depth of the message body.

The nesting limit is 64, which was also the limit in static
signatures.  Empirically, dynamic nesting depth observed on my
Fedora 14 system doesn't exceed 2; 64 is really a huge limit.

https://bugs.freedesktop.org/show_bug.cgi?id=32321

Signed-Off-By: Colin Walters <walters@verbum.org>
Signed-off-by: Will Thompson <will.thompson@collabora.co.uk>

policy tests: Use bin rather than nogroup, wheel, or root

This group seems to exist on all of Debian, Fedora and Mac OS X. (I
mistakenly believed that nogroup exists on Fedora.)

sysdeps-unix: check for HAVE_DECL_MSG_NOSIGNAL

MSG_NOSIGNAL could theoretically be an enum member or something rather
than an #define, so it's better to check for the flag defined by the
configure check (as is done in dbus-connection.c already).

Reviewed-by: Colin Walters <walters@verbum.org>
Reviewed-by: Scott James Remnant <scott@netsplit.com>

sysdeps-unix: use MSG_NOSIGNAL when sending creds

Since SIGPIPE is no longer touched by default when MSG_NOSIGNAL is
available, it's extra-critical that all socket writes actually pass
that flag.

Signed-off-by: Will Thompson <will.thompson@collabora.co.uk>

Revert "test-autolaunch is only known to work in launchd environments."

This reverts commit 4626b405609900812125a407ef5fdd31c7b89868.
test-autolaunch works fine in non-launchd environments (and non-X11
environments, based on a quick test passing enable_x11=no to configure).
On the contrary: this commit *broke* the build on non-launchd
environments, because test/name-test/run-test.sh still tried to run this
test even if it hadn't been built.

Ignore generated bus/org.freedesktop.dbus-session.plist

policy tests: Use 'nogroup' rather than 'wheel' or 'root'

The excellently-titled commit 197bef8 “Fix test failures on OSX.” broke
the tests on Linux, since there's no wheel group on this side of the
tracks. So here's a group everyone should enjoy.

(If anyone comes along and tells me that DragonflyBSD doesn't have
'nogroup' …)

Handle case when autolaunched daemon address is already pulished on windows.

Windows fix: In install path scope case add hashed install path to mutex/shm name.

Fix malformed specification XML.

bf532d6 and 28a2d08 each introduced errors in the specification XML that
left it malformed and unbuildable.

Be more verbose when publishing or requesting session bus address on Windows.

Add documentation for autolaunch meta transport on Windows.

Reviewed-by: Romain Pokrzywka

Add launchd section to specification.

Implements user limited session bus on Windows.

This feature is enabled by using the session bus address
"autolaunch:scope=*user" for the client and the server.

Reviewed-by: Romain Pokrzywka

Fixed bug not freeing mutex handle on Windows.

Reviewed-by: Romain Pokrzywka

Variables and functions name cleanup on Windows.

Reviewed-by: Romain Pokrzywka

10.4 is old so set more sensible launchd defaults.

Fix bus-test failure with cmake on unix.

SHA1 connection tests needs an non debug-pipe transport for the activation bus.

Enable launchd.

This patch enables support for Mac OS X's launch daemon
for startup as well as sharing of the DBus session bus
environment.  It includes a LaunchAgent plist for automatic
start of the session bus.

Look up DISPLAY from launchd if not initialized.

Add launchd implementation.

Fixed wrong libtoolize presence check.

libtoolize is called glibtoolize on OSX so check for it too.

Fix test failures on OSX.

test-autolaunch is only known to work in launchd environments.

Keep in sync with automake.

Added a test-autolaunch.

* test/name-test/test-autolaunch.c: New file,
unsets DBUS_SESSION_BUS_ADDRESS so we should
fall back to autolaunch:.
* test/name-test/run-test.sh: Run it.
* test/name-test/Makefile.am: Build it.

Fix PROFILE_TIMED_FORMAT printf warning on OSX.

Added revision history.

Documentated autolaunch implementation for X Windowing system.

Service dir related bug fix on windows.

The code for accessing services requires absolute pathes, which are based
on DBUS_DATADIR. DBUS_DATADIR on windows is defined relative. This patch
makes sure that those pathes are absolute.

Restore the close_on_exec flag on windows.

Merge branch 'documentation-build-system'

Dist Readme.win instead of Readme.windbus

Cleaned up windows related README's.

Fixed reference to cmake configure options.

keep version in sync with automake

Merge branch 'master' of ssh://git.freedesktop.org/git/dbus/dbus

Revert "Added documentation for autolaunch transport."

This reverts commit 2e61875728deca49a96e2db52275f3a5e24bb59b.

Tentative workaround for the random hangs on windows.

The problem seems to be a race condition with winsock's
internal threads for the non-blocking mode of the sockets,
but I haven't had time to try a standalone test case yet
to confirm it. Anyway, I found a workaround that fixes it
in all cases, so it's good enough for now.

tentative workaround for the random hangs on windows

Only use signature directly inside _dbus_verbose

Don't pretend to free lock if it was never allocated

Always get current time in _dbus_connection_block_pending_call

Take care to free windows_sid in cache_peer_loginfo_string

Free envvar and args in pass_info in the oom case

Move uploading docs into build system.

This uploads a doc tarball, and its contents, to the appropriate
location on dbus.freedesktop.org. It also uploads the DTDs to the
appropriate location on specifications.freedesktop.org.

I believe this uploads the same files as the old update-dbus-docs.sh
script did.

Integrate building HTML-ified manpages with the build system

Report correct result of doxygen/xml doc checks

Previously the configure output always said these were enabled during
the 'Checking for...' stage, even if they weren't. The summary at the
end of configure was correct, though.

Move manpages to doc/

This will make integrating the building of HTML versions of these
manpages into the build system way easier, at the cost of keeping
manpages in a different directory to the source for the program they
describe. I think this is an acceptable trade-off.

Build Doxygen documentation for `make all`.

This depends on GNU Make for the wildcard dependency on all the source
files in dbus/. If anyone objects very strongly to this, I'd welcome
suggestions of a more portable way to do this.

Update a load of .gitignores

Silence silent build a bit more.

Squash warnings and spurious output from autogen.sh

Use Automake 1.11, if available, for silent rules

Automake 1.11 adds support for silent rules, which are conditionally
enabled in configure.in if available.

Also, previously it was impossible to override the version chosen by
this script; this patch makes that possible.

When launching dbus-daemon be more verbose in error case.

Keep unix autolaunch functions in sync.