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

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

Keep unix autolaunch functions in sync.

Added documentation for autolaunch transport.

Fixed case when no scope attribute is used.

Extended autolaunch protocol with scope attribute.

Add return_if_fail watch != NULL in public DBusWatch API

Reviewed-by: Havoc Pennington <hp@pobox.com>
Signed-off-by: Will Thompson <will.thompson@collabora.co.uk>

Include cmake subdirectory in source distribution.

Fix typo in creating temp file.

Fix use of _dbus_make_file_world_readable.

Explicitly require the daemon to respect destination=''

Unix compile fix.

post release version bump

fix some "unused result" warnings in dbus-launch and dbus-spawn

Needed to avoid -Werror

Change failure of setsockopt SO_REUSEADDR to be a warning, not set an error

It was previously setting an error but then continuing, which is
not allowed since errors can't be "piled up"

build-sys: bump version number for the 1.4.0 release

systemd: enable the dbus service unconditionally

Since D-Bus is a core part of the OS don't make this service optional.

autoconf: update lt version info in preparation for 0.4.2

NEWS: update for 1.4.0

systemd: update sd-daemon.[ch] from upstream systemd

Merge branch 'dbus-1.2'

Conflicts:
dbus/dbus-sysdeps-unix.c

Make dbus-uuidgen atomic

A Red Hat QA engineer hit in practice a race condition in dbus-uuidgen
where it could leave an empty file.

dbus-uuidgen (_dbus_create_uuid_file_exclusively) formerly created an
empty file in the path to the uuid, then filled it in.  At some point,
the internal libdbus _dbus_string_save_to_file became atomic on Unix
at least (doing the save to temp file, fsync(), rename() dance).

So _dbus_create_uuid_file_exclusively doesn't need to create the file
beforehand anymore.  However, it *does* need the file to be
world-readable, unlike all other consumers of
_dbus_string_save_to_file.  So add a "world_readable" argument.

sd-daemon: update from upstream

This updates the sd-daemon.[ch] copy from upstream, which makes sure
thet the code compiles even if SOCK_CLOEXEC is not supported.

Cleanup: removed unused code; fixed Winsock error handling; fixed code indentation.

Fixed wince build problem reported by aheinecke at intevation dot de.

Fixed bug not creating bus/messagebus-config.

Delete stale pid file on cygwin.

Avoid -Werror problems on cygwin.

Applied patches from cygwin port.

fix socket descriptor leak in _dbus_connect_tcp_socket_with_nonce

Wrap OutputDebugString for Windows CE builds, and add cast for MSVC.

Add unit tests for dbus_message_get_path_decomposed

Assert in _dbus_decompose_path if path is NULL or len is wrong

systemd: update systemd glue code

systemd: remove dbus pid file before starting dbus

man: add documentation for four new switches to dbus-daemon

This adds explanations for --address and --systemd-activation which have
recently been added to the daemon. Also adds coumentation for --nofork
and --introspect which has been previously forgotten.

systemd: add a nice description string to the .socket file

systemd: install systemd socket and service file by default

build-sys: update pkg-config.m4

activation: optionally, use systemd for system bus activation

Fix use of $servicename in status

See https://bugzilla.redhat.com/show_bug.cgi?id=521797

We have two init scripts, and commit 05283235632bc8891aaa3047563cb4
patched the "wrong" (i.e. slackware) one.  Patch one that ships
with real operating systems too.

bus: add --address switch

This is allows overriding of the listening address on the command line,
which is particularly useful for systemd socket-based activation.

systemd: add systemd-style socket-activation

This is a pretty straightforward implementation of systemd-style socket
activation using the reference implementation of the
$LISTEN_FDS/$LISTEN_PID env var parsing as supplied by systemd upstream.

sysdeps-unix: enable address reuse on TCP sockets

This helps to avoid 'Address already in use' problems if server has been shut
down, and then restarted while sockets are still active on its port.

Signed-off-by: Steffen Sledz <sledz@dresearch.de>

Fix the reentrancy issue reported on bug 17754.

Patch based on patch by Havoc Pennington, with the references that
this is temporary removed.

        Patch based on one from Olivier Hochreutiner <olivier.hochreutiner
        gmail.com>

        * dbus/dbus-connection.c (protected_change_timeout): remove the
        elaborate nonworking hack to try to drop locks and just keep the
        locks; this isn't right either, but at least is correct, though
        it puts restrictions on apps.

        * dbus/dbus-connection.c (protected_change_watch): make the same
        change as for timeouts

        * dbus/dbus-connection.c (dbus_connection_set_timeout_functions):
        don't drop the lock here; add documentation of the problem to API
        docs
        (dbus_connection_set_watch_functions): same

        * dbus/dbus-connection.c (dbus_connection_get_data)
        (dbus_connection_set_data): introduce a separate slot_mutex
        protecting connection->slot_list so these two functions can be
        called inside watch and timeout functions. Not sure this
        is going to be a good idea.

        * dbus/dbus-connection.c (dbus_connection_unref)
        (dbus_connection_ref): avoid using connection lock in ref/unref
        so these can also be used in watch and timeout functions

build-sys: open 1.3.2 development cycle

build-sys: add make target for updating AUTHORS file

build-sys: prepare release 1.3.1

pkg-config: add a couple of additional variables to .pc

We already export the session services dir in the .pc file, add the
system services dir too, as well as the interfaces dir, so that packages
can drop their files into the right place.

Eventually we need to move the system-services dir to a place that is
relocatable to /lib/dbus, and this change makes the location queriable
for other projects, as a first step towards that goal.

Merge branch 'dbus-1.2'

Implement doctype check in the trivial parser

This is needed for the libxml parser backend.

Revert "Bug 17754 - Fixed race condition in protected_change_timeout."

This reverts commit b82ff17d7dd9689ab664e4a6c2a86c6f3dbfba69 and
9280395330c833dd98d0811d31349c26a84b2f6a.

Conflicts:

dbus/dbus-connection.c

Clarify GetConnectionUnixUser documentation.

(This makes it match the freshly-documented GetConnectionUnixProcessID.)

Document ListQueuedOwners in the spec

Document GetConnectionUnixProcessID in the spec

Fix a broken <xref/>

Revert "Bug 28460 - Refactored dbus configuration access."

This reverts commit 6f9077ee870ad02119facf83d1293301b4535c3b, which
broke the build on Unix (see previous patch) and introduced a bunch of
warnings.

Fix dbus-sysdeps.h on amd64, again.

HAVE_STDINT_H is defined in config.h, but that's not much use if
config.h isn't included. The new file dbus/dbus-config.c includes this
header without including config.h first, so fails to build.

Since dbus-sysdeps.h is internal, we can include config.h with impunity,
rather than relying on our callers to do so. Also, there's no need to
include stdint.h twice.

Bug 28460 - Refactored dbus configuration access.

Libdbus uses several config variables. On unix these settings are read from
environment variables by using _dbus_getenv.

On other platforms like wince there are no environment variables available and
_dbus_getenv needs an emulation for those plattforms (see
dbus/dbus-sysdeps-wince-glue.c)

To cleanup this emulation the appended patch adds a config api by adding
_dbus_config_... functions.

Also having all client config related functions listed in one header file
provides a good overview about which config attributes  are available.

The default implementation retrieves the config values from environment
variables. For other os this could be easily extended or replaced by.

Use dbus provided wait function.

Bug 17754 - Fixed race condition in protected_change_timeout.

Corrected thread problem causing some calls to hang for 25s

Since the connection lock is released for a short while in
_dbus_connection_acquire_io_path there can already be a method return
received by another thread. The fix is to do an extra check after the
I/O path has been aquired both.

Approved-by: Thiago Macieira

Do not set DBUS_SESSION_BUS_DEFAULT_ADDRESS unconditional in code, it is already defined in build system.

Keep cmake default session and system bus address setup in sync with autotools.

Fix x64 build after WinCE intptr_t usage changes.

Corrected thread problem causing some calls to hang for 25s

Since the connection lock is released for a short while in
_dbus_connection_acquire_io_path there can already be a method return
received by another thread. The fix is to do an extra check after the
I/O path has been aquired both.

Approved-by: Thiago Macieira

Fixed compiler warnings.

Minor include file fix.

When including dbus/dbus-errors.h to get error related functions,
the related error constants should be available too.

Keep in sync with autotools.

Keep in sync with autotools.

Merge branch 'dbus-1.2'

Conflicts:
bus/main.c
configure.in

kqueue set_watched_dirs: fix termination condition

num_fds is the number of elements of dirs currently in use. This bug
meant that encountering a previously un-watched directory would cause j
to increment forever, and so dirs[j] would eventually segfault.

(I've checked the corresponding code for inotify, and it's correct. I
wonder if some of the duplication could be eliminated.)

Thanks to Pablo Martí Gamboa <pmarti@warp.es> for reporting this issue!

Fix building on Unix

Fix memory allocation when reusing message from cache

Signed-off-by: Thiago Macieira <thiago@kde.org>

Add an extra parameter to the PropertiesChanged() signal

It was pointed out on the mailing list that it would be useful to know
that a given property has changed without conveying its value. Because
without this parameter a true_no_value property could change, however
there is no way for a client-side proxy to know _what_ property it was
(only that some property changed).

With the parameter, however, a client-side proxy can reliably discard
a cached property value.

Also rename the "true_no_value" to "invalidates" as the spec is now
using this language.

Also allow using the annotation in the enclosed interface name.

Also rename the annotation name so it uses Property in its name
instead of Properties. This is to be more consistent with the existing
org.freedesktop.DBus.Method.NoReply annotation which uses Method, not
Methods.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Add an optional PropertiesChanged() signal

Some notes about this new signal

 - The PropertiesChanged() signal is optional. An application can
   convey support for this signal by either including or excluding it
   from the returned introspection data much like apps not supporting
   (or predating) the GetAll() method does not include GetAll() in the
   introspection data.

 - An object can use PropertiesChanged() but opt out of using it for
   one or more properties by using the

     org.freedesktop.DBus.Properties.EmitsChangedSignal

   annotation on the properties in question

 - Applications can start using this new signal without breaking
   compatibility with clients relying on existing D-Bus API.

The intent of the patch is simply to standardize existing behavior

 - EggDBus has a very similar signal called EggDBusPropertiesChanged()
   (also on the org.freedesktop.DBus.Properties interface)

 - NetworkManager has a PropertiesChanged() signal on each different
   interface (e.g. not org.fd.D.P) that it implements

 - GDBus, an implementation of the D-Bus protocol in GLib, already
   implements this signal

Signed-off-by: David Zeuthen <davidz@redhat.com>

Fix _IOLBF for Windows CE and also define _IONBF if needed.

Fixed assert raised with msvc 2008 on windows.

Fix infinite recursion on Windows CE.

Fix GetModuleFileNameA and RegQueryValueExA for Windows CE.

Add clarification to spec about UpdateActivationEnvironment

UpdateActivationEnvironment takes a a{ss}.  This means only
valid UTF-8 can be used.  Environment variables are normally
ascii, but in theory have no specific encoding to them.  This
means that certain valid environment variables can't be sent
to the bus for updating its activation environment.

This commit just adds a note to the spec explaining this
restriction.

Move definition of _WIN32_WCE to configure.in (cmake defines it already).

fix {u}intptr_t usage on wince with msvc

Send system log messages to debug port for now.

Added missing implementation for _dbus_system_logv().

Cleanup debug messages.

Defensively protect against bogus argv argument on W32CE.

Disabled launch helper activation tests on windows.

Launch helper application isn't supported on windows yet.

Be more silent with porting problems.

Don't truncate pointers on Windows x64 platform

Change to assume executable extension when none is given (except on wince)

Pass the environment to CreateProcessA correctly and be more defensive

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

Minor name spelling fix.

Fix spelling error

This broke 'make install' since dbus-daemon-launch-helper didn't get
installed.

Signed-off-by: David Zeuthen <davidz@redhat.com>

[legacy init script] Fix the use of $servicename

Added cmake cross compile wrapper.