Fix inotify shutdown

We were incorrectly passing NULL for a DBusList when the usage expected
is a pointer to a NULL DBusList pointer.  Also during dbus_shutdown
we need to actually close the inotify fd, and remove our watch.

Move the shutdown handler out of bus.c and into inotify where we
can do all of this cleanly.
(cherry picked from commit 90fe96b1875350f86a4a773d4a0a22009950dd4d)

linux cmake build system related compile fix

Fix compilation in --disable-selinux case

_dbus_change_to_daemon_user moved into selinux.c for the --with-selinux
(and audit) case because that's where all of the relevant libcap headers
were being used.  However in the --disable-selinux case this didn't
compile and wasn't very clean.

If we don't have libaudit, use the legacy direct setgid/setuid bits
we had before in dbus-sysdeps-util-unix.c.

Merge branch 'my-dbus-1.2'

Conflicts:
configure.in

Release 1.2.18

Use monotonic clock for _dbus_get_current_time() if it's available.

_dbus_get_current_time() is used for timeouts, but uses gettimeofday(), which
relies on the wall clock time, which can change. If the time is changed forwards
or backwards, the timeouts are no longer valid, so the monotonic clock must be used.

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

Signed-off-by: Colin Walters <walters@verbum.org>

win32 fix: provide separate def file for cmake until there is full splitlib support for cmake

win32 fix for commit 8a9880ffd2b81df38bb0e3492bda7a9636ac0280 - Clean up inotify watch handling

Don't crash when reloading if we haven't loaded user database yet

The user database is populated on-demand, but the cache dropping
code assumed it had been initialized.  Simply check for NULL.

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

Merge branch 'my-dbus-1.2'

Move system-activation.txt into non-conditional EXTRA_DIST

It doesn't depend on the XML doc generation.

Clean up inotify watch handling

Substantially based on a patch by Matthias Clasen <mclasen@redhat.com>
kqueue implementation by Joe Marcus Clarke <marcus@freebsd.org>

Previously, when we detected a configuration change (which included
the set of config directories to monitor for changes), we would
simply drop all watches, then readd them.

The problem with this is that it introduced a race condition where
we might not be watching one of the config directories for changes.

Rather than dropping and readding, change the OS-dependent monitoring
API to simply take a new set of directories to monitor.  Implicit
in this is that the OS-specific layer needs to keep track of the
previously monitored set.

_dbus_change_to_daemon_user() is platform related  and commit 0a3905d7f3b2ff43b09479863775939f9c8acad4 broke windows builds

reverted 2aea64fbb69287f52bb9910e89b6732854f47be8 because it breaks msvc2008 builds

Update messagebus.in init script to start earlier

Patch from Fedora.

[doc] diagram.[svg,png] to EXTRA_DIST

These files are part of the docs and should appear in tarballs.

Merge branch 'my-dbus-1.2'

Conflicts:
bus/Makefile.am
dbus/dbus-sysdeps-util-unix.c
dbus/dbus-transport.c
test/test-service.c

Don't drop pending activations when reloading configuration

The reload handling for activation simply dropped all knowledge
of pending activations, which was clearly wrong.  Refactor things
so that reload only reloads directories, server address etc.

Based on a patch originally from Matthias Clasen <mclasen@redhat.com>

Switch to libcap-ng, avoid linking libdbus against libcap[-ng]

(Commit message written by Colin Walters <walters@verbum.org>)

A current Fedora goal is to convert projects to libcap-ng which
more easily allows dropping Linux capabilities.  For software
which also links to libdbus, it's problematic to link against
libcap as well.

Though really, libdbus should have never linked against libcap
in the first place, which is another thing this patch changes
by moving the libcap-using bits out of dbus/ and into bus/.

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

Add Will Thompson and Simon McVittie to reviewers, add emails to all

The reviewer list was sorely lacking actual email addresses; fix this.
Also add Will and Simon.

Ignore exit code zero from activated services

A variety of system components have migrated from legacy init into DBus
service activation.  Many of these system components "daemonize", which
involves forking.  The DBus activation system treated an exit as an
activation failure, assuming that the child process which grabbed the
DBus name didn't run first.

While we're in here, also differentiate in this code path between the
servicehelper (system) versus direct activation (session) paths.  In
the session activation path our error message mentioned a helper
process which was confusing, since none was involved.

Based on a patch and debugging research from Ray Strode <rstrode@redhat.com>

Print all-printable-ASCII byte arrays as strings

In practice, ay seems to be used mostly for binary data (in which case,
hex output is fine) or for Unix file paths (because they may be
non-UTF-8) and similar human-readable strings. So let's print the latter
similarly to strings.

Print byte arrays as nicely-formatted hex.

Add an accessor for the loader's corruption reason

Include reason when reporting corrupt messages

It would have been much easier to diagnose fd.o#19723 if the error
message had said more than just "Message is corrupted".

Forbid zero serial numbers

Make array-printing code easier to follow

Previously dbus_message_iter_get_arg_type() was called twice: once in
the loop condition to update 'current_type', and once to check if the
loop will run again. This patch moves updating current_type to the end
of the loop body.

Correct timeout handling

The timeout handling code subtracts the elapsed time from the timeout
each time a message is received, which drastically reduces the timeout
in circumstances such as service activation.

Correct so that the timeout is never modified, and the elapsed time
instead subtracted where necessary.

Signed-off-by: James Westby <jw+debian@jameswestby.net>
Signed-off-by: Scott James Remnant <scott@ubuntu.com>

Allow to override test tool path for cross compilation setup.

Bug 23502 - corrected wrong verbose-output

Invoke AC_PROG_MKDIR_P to make sure MKDIR_P is defined.

Only list those symbols that can be exported (new toolchain barfs on undefined symbols in export list).

Include errno.h.

Fix buffer underrun and clarify condition.

Add Will Thompson and Simon McVittie to reviewers, add emails to all

The reviewer list was sorely lacking actual email addresses; fix this.
Also add Will and Simon.

fixes bug where sometimes objects were not unregistered from dbus
although their app had exited. As it turns out it was a missing
implementation of the method _dbus_fd_set_close_on_exec() for windows.
It's been tested with msvc2005, msvc2008, and mingw, and solves
all the unregistering issues on many kde apps.

Rename DBusConnection *disconnected param to connection

Group match rules by their interface.

In my informal studies of "normal" sets of match rules, only checking
match rules with the appropriate interface for the message reduces the
number that need to be checked by almost 100x on average (ranging from
halving for messages from the bus daemon, to a >200x reduction in many
cases). This reduces the overhead added to dispatching each message by
having lots of irrelevant match rules.

Extract rule_list_remove_by_connection

Extract freeing a DBusList<BusMatchRule>

Don't bother re-matching features we've checked.

This is currently not a big deal, but will make more of a difference
once the set of match rules is partitioned by more features than just
the message type.

Index match rules by message type

This avoids scanning all the signal matches while dispatching method
calls and returns, which are rarely matched against.

Add a constant for the number of message types

added tranport address format to dbus specification

added initial tcp transport section

nonce tcp support requires _dbus_string_equal_len also when build without tests

fixed install root when using msvc ide

let dbus_verbose print file name, line number and function name if available; this eases debugging.

Bug 25697 - Fix memory leak in policy reload

Signed-off-by: Colin Walters <walters@verbum.org>

removed local definition of _DBUS_GNUC_PRINTF, it is already in dbus-macro.h

DBusPipe related unix compile fix

added pipe related files real

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

windows does not support fork, skip it

moved out DBusPipe implementation into separate files

moved out DBusPipe implementation from into separate files

Ignore exit code zero from activated services

A variety of system components have migrated from legacy init into DBus
service activation.  Many of these system components "daemonize", which
involves forking.  The DBus activation system treated an exit as an
activation failure, assuming that the child process which grabbed the
DBus name didn't run first.

While we're in here, also differentiate in this code path between the
servicehelper (system) versus direct activation (session) paths.  In
the session activation path our error message mentioned a helper
process which was confusing, since none was involved.

Based on a patch and debugging research from Ray Strode <rstrode@redhat.com>

added new variable DBUS_BUILD_TIMESTAMP

fixed memory leak

most patches are obsolate

fixed cmake version macro and string handling

added cmake option for using nonce-tcp protocol as default; set tcp as default protocol to be compatible with earlier dbus releases

fixed crashes accessing not initiated noncefile member struct

cleanup nonce socket only when really used

uses tcp based test connections for now

dbus-daemon needs more exports

readded test files required for windows

removed unsupported symbols for msvc

Drop unused complex function

Drop _dbus_user_at_console() from dbus-sysdeps-util-win.c. It is
called only from dbus-userdb-util.c which isn't compiled on Windows.

Set the DBusError

Put the correct error code and a relevant message into the DBusError
in _dbus_unix_user_is_at_console().

Don't fake network errno values on Windows

Don't pretend there exists errno values for some network errors in the
Microsoft C library. Just use the WinSock error values.

Handle also WinSock errors in _dbus_error_from_errno

Handle those WinSock errors that match the errno values handled. Don't
bother handling those errors that are mapped to DBUS_ERROR_FAILED in
the switch as that is the default return value anyway.

Windows fixes

Replace dbus_daemon_init() by dbus_publish_session_bus_address(),
publishing the full address. Omit username from mutexes (not necessary
as mutex names are local to session). Don't exit if publishing the
address failed (allow multiple session bus instances per
session). Based on 00ee92ae314 by Tor Lillqvist.

Cherry-picked from commit 23945513e9a4da61d286ebfbce3897aa061ddbfe in
the dbus4win repository by tml@iki.fi. Remove claim of that commit not
being merged from README.dbus4win.

Avoid warnings on Windows

Remove unused functions, or put in #if 0 if potentially useful. Make
internal functions used just in one file static. Use -Werror after all
also on Windows. Construct the installation root from the location of
the dbus DLL, not from the location of the program .exe of the
process.

used dbus/dbus-1.def for cmake too

added missing symbols for bus-test

updated version

Drop terminating slash in _dbus_get_tmpdir

The callers will append strings starting with a slash anyway.

Use nonce-tcp default session bus on Windows

Check for dirent.h

DBUS_VERSION is always three numbers

No *-rc* possible. So drop strange crap manipulating it for
BUILD_FILEVERSION. Drop duplicate AC_SUBST.

Actually do install the .def file on Windows

Use dbus-1.def on Windows

Drop internal symbols, add missing symbols

Add a dbus/dbus-1.def file

From cmake/dbus/dbus-1.def.cmake

Don't use DBUS_DIR_SEPARATOR

I didn't cherry-pick the commit from dbus4win that introduced
DBUS_DIR_SEPARATOR, as I think it just uglifies the code with little
gain. (Forward) slashes work fine on Windows.

Don't use -fPIC and -fPIE on Windows

The MinGW compiler recognizes them but emits a warning "-fPIC ignored
for target (all code is position independent)"

Add file describing the merge of the dbus4win code

Tell where the file comes from and that it might not be reliable

Rename README.win to README.windbus to match its origin

Also add it to EXTRA_DIST.

Look for config files also in the normal Unix-style location (cherry picked from commit 445d1f780fb916f9087c0d95e7594ee5a425d43f)

Use multi-byte string functions on Windows

Don't walk through char arrays that contain strings in the system
codepage char by char looking for '\\'. There are double-byte
characters in East Asian codepages where the second byte is a '\\'.
(cherry picked from commit 61316262da466993abbcba010c6bac90bb0b1d43)

Don't bother with -Werror on Windows for now

We have so many warnings

with msvc, replace va_copy by assigning the va_lists (cherry picked from commit 42d613fc9284afaa9543549be26f32e503a58348)

Fixes to the nonce code

Cherry-picked from commit f9bc0f4bd036f6ede5d9850fb0f8587178c85f44 in
the dbus4win repository, slightly massaged to apply by tml@iki.fi.

Fix IPv6 setup (the default on Vista)

Ensure that the right struct is passed.

Cherry-picked from commit 172d60d100d04e301e51ff475ada02007f1709dc in
the dbus4win repository by tml@iki.fi.

Add api dox for nonce-tcp

Cherry-picked from commit e7a070db22ed4a84dc04a062255356c9f6c4c9c5 in
the dbus4win repository, trailing whitespace issues corrected by
tml@iki.fi.

Fixes to the nonce code

On Unix, we create a 700 subdir in /tmp only readable by the user, on
Windows we use the user-specific tmpdir and create the noncefile
directly.

Add NonceFile to abstract this behaviour and to delete the noncefile
and possibly tmpdir on shutdown.

Cherry-picked from commit 885f16b90c4f769ae29f432d0ed2a63bb2e4dab8 in
the dbus4win repository. Fixed to apply and correct whitespace issues
by tml@iki.fi.

Add nonce-tcp section to the specification (draft) (cherry picked from commit 7015b2db6c8744c88da9cb0fab8d5e0fcfc0220b)

Improve error handling in nonce code

Replace errno (from assuan code) by DBusError. Errors are not passed
through dbus_accept_with_nonce atm, as the original _dbus_accept() has
no error arg either.

Cherry-picked from commit 8082e92bb3a6304362e730f8b6e324860e1bb26c in
the dbus4win repository, edited to apply and fix whitespace issues by
tml@iki.fi.