Marcus Brinkmann [Fri, 19 Mar 2010 11:51:12 +0000 (12:51 +0100)]
Do not use DBUS_WIN in a public header file, use _WIN32.
Marcus Brinkmann [Fri, 19 Mar 2010 11:37:21 +0000 (12:37 +0100)]
Define dbus_1_EXPORTS when appropriate.
Marcus Brinkmann [Fri, 19 Mar 2010 11:36:49 +0000 (12:36 +0100)]
Consistently include <config.h> in all C source files and never in header files.
Will Thompson [Thu, 4 Feb 2010 20:24:54 +0000 (20:24 +0000)]
Dispatch post-activation messages to anyone interested
Previously, if a method call activated a service, it would only be
delivered to that service, and not to other services with match rules
which should match. This patch replaces the improperly-duplicated
dispatch code in activation.c with a call back into the normal dispatch
code, fixing this bug (fd.o#26427).
(Additionally, were one to service-activate a service that doesn't
understand file descriptors with a message containing a file descriptor,
the previous code would send it anyway, and the service's dbus library
would blow up. This is also fixed here, since the normal dispatch code
checks this correctly.)
Will Thompson [Thu, 4 Feb 2010 20:12:28 +0000 (20:12 +0000)]
Move dispatching to destination to bus_dispatch_matches()
Romain Pokrzywka [Tue, 16 Mar 2010 23:14:55 +0000 (16:14 -0700)]
export _dbus_credentials_to_string_append for the tests
Romain Pokrzywka [Tue, 16 Mar 2010 23:13:00 +0000 (16:13 -0700)]
Update the windows implementation of logging functions
Following commit:
04cf3166002a86b9a22851be4e243c87b5b3048d for Unix.
Marcus Brinkmann [Tue, 16 Mar 2010 10:15:58 +0000 (11:15 +0100)]
Fix the make dist target.
Marcus Brinkmann [Tue, 16 Mar 2010 02:15:15 +0000 (03:15 +0100)]
Prepend HOMEDRIVE to HOMEPATH so cross-volume accesses work.
Marcus Brinkmann [Mon, 15 Mar 2010 14:52:49 +0000 (15:52 +0100)]
Check return value of MoveFileEx correctly.
Marcus Brinkmann [Mon, 15 Mar 2010 14:51:27 +0000 (15:51 +0100)]
Close process handle leak.
Marcus Brinkmann [Mon, 15 Mar 2010 14:49:31 +0000 (15:49 +0100)]
Use INVALID_HANDLE_VALUE, not NULL.
Marcus Brinkmann [Mon, 15 Mar 2010 14:43:32 +0000 (15:43 +0100)]
Fix possible alignment issue.
Colin Walters [Tue, 16 Mar 2010 20:44:48 +0000 (16:44 -0400)]
Merge branch 'dbus-1.2'
Colin Walters [Tue, 16 Mar 2010 17:20:36 +0000 (13:20 -0400)]
Don't send an reply for driver messages if one isn't requested
Basically honor the no_reply flag; a binding is likely to want to
set this so it doesn't have to process extra traffic for match
rules.
Colin Walters [Tue, 16 Mar 2010 20:04:15 +0000 (16:04 -0400)]
Merge branch 'dbus-1.2'
Conflicts:
bus/bus.c
configure.in
Colin Walters [Mon, 15 Mar 2010 22:31:30 +0000 (18:31 -0400)]
Make SELinux initialization failure fatal
https://bugzilla.redhat.com/show_bug.cgi?id=572769
Previously we'd just continue if AVC initialization failed, but
that didn't really work because then we'd later crash in
avc_has_perm.
Simply treat initialization failures as fatal, and hopefully we
can get data from the system log.
Colin Walters [Tue, 16 Mar 2010 11:19:04 +0000 (07:19 -0400)]
Add a prefix to our syslog messages
Previously we were simply logging as "dbus", and it was unclear
whether it was the system bus, or a session bus. And if the latter,
which user?
This patch adds a prefix to the log message with the bus type
and the userid.
Colin Walters [Tue, 16 Mar 2010 13:00:23 +0000 (09:00 -0400)]
Add _dbus_credentials_to_string_append
Convert the credentials to a string; useful for logging.
Colin Walters [Fri, 12 Mar 2010 16:30:00 +0000 (11:30 -0500)]
Add DBUS_SYSTEM_LOG_FATAL severity
This severity is useful for when we encounter a fatal problem; we
get a log message out, then exit.
Colin Walters [Mon, 15 Mar 2010 17:50:48 +0000 (13:50 -0400)]
Refactor _dbus_log_info, _dbus_log_security into _dbus_log_system
In preparation for a future patch which introduces a fatal logging level,
don't duplicate the API here.
Kimmo Hämäläinen [Tue, 16 Mar 2010 17:55:13 +0000 (13:55 -0400)]
[dbus-spawn] Use correct sign values for variables
read() returns a signed value, whereas the byte count is unsigned.
Signed-off-by: Colin Walters <walters@verbum.org>
Kimmo Hämäläinen [Tue, 16 Mar 2010 17:50:30 +0000 (13:50 -0400)]
[dbus] Add missing va_end in dbus-errors.c
Colin Walters [Tue, 16 Mar 2010 17:11:20 +0000 (13:11 -0400)]
[main] Handle OOM in reload watch
http://bugs.freedesktop.org/show_bug.cgi?id=12953
Wait for memory if we can't init the string.
Based on an original patch by Kimmo Hämäläinen <kimmo.hamalainen@nokia.com>
Simon McVittie [Tue, 16 Mar 2010 16:40:08 +0000 (12:40 -0400)]
[doc/dbus-specification.xml] Explicitly specify lower-case hex must be used
https://bugs.freedesktop.org/show_bug.cgi?id=9486
When describing DBUS_COOKIE_SHA1, specify that lower-case hex encoding
must be used.
Colin Walters [Tue, 16 Mar 2010 16:16:33 +0000 (12:16 -0400)]
Support inotify on older kernels
https://bugs.freedesktop.org/show_bug.cgi?id=23957
Previously we detected glibc support at compile time and used
it unconditionally; better to try it and fall back, this way
we continue to run on older kernels when compiled for newer ones.
Ralf Habacker [Thu, 4 Mar 2010 22:39:03 +0000 (23:39 +0100)]
Decorated public exported functions with DBUS_EXPORT macro, reviewed by Colin Walters.
Marcus Brinkmann [Mon, 15 Mar 2010 14:39:45 +0000 (15:39 +0100)]
Fix double-free in error case.
Signed-off-by: Thiago Macieira <thiago@kde.org>
Colin Walters [Fri, 12 Mar 2010 22:23:10 +0000 (17:23 -0500)]
[configure.in] Use AM_SILENT_RULES if available
By rough consensus, and to keep up with the cool kids.
Note to builders: you need to use V=1 to enable verbose mode.
Michael Meeks [Fri, 12 Mar 2010 16:50:27 +0000 (11:50 -0500)]
[DBusConnection] When handling a watch, return if another thread is doing I/O
http://lists.freedesktop.org/archives/dbus/2010-March/012337.html
Previously, the watch handler would block until the I/O path was available.
However, if another non-main thread was doing a synchronous call, this would
cause the main thread to block on that thread, a highly undesirable
thing because it's important for the main thread to remain responsive
for user interfaces.
Signed-off-by: Colin Walters <walters@verbum.org>
Signed-off-by: Thiago Macieira <thiago@kde.org>
Ralf Habacker [Mon, 8 Mar 2010 13:44:45 +0000 (14:44 +0100)]
removed obsolate file from windbus port
Ralf Habacker [Mon, 8 Mar 2010 14:56:56 +0000 (15:56 +0100)]
fixed bus-test sha1 connection address on unix with cmake build system
Ralf Habacker [Mon, 8 Mar 2010 14:55:46 +0000 (15:55 +0100)]
fixed backtrace printing on unix with cmake build system
Frederic Crozat [Wed, 3 Mar 2010 13:05:23 +0000 (14:05 +0100)]
Fix build when selinux is disabled but libaudit enabled
https://bugs.freedesktop.org/show_bug.cgi?id=26859
Ralf Habacker [Sun, 28 Feb 2010 20:32:05 +0000 (21:32 +0100)]
_dbus_string_toupper_ascii(): new function, reviewed by Colin Walters.
Ralf Habacker [Sat, 27 Feb 2010 08:09:04 +0000 (09:09 +0100)]
win32 fix: make dbus string content visible to debuggers, reviewed by Colin Walters.
Ralf Habacker [Sun, 28 Feb 2010 19:56:42 +0000 (20:56 +0100)]
_dbus_string_tolower_ascii(): new function, reviewed by Colin Walters.
Ralf Habacker [Fri, 26 Feb 2010 07:37:53 +0000 (08:37 +0100)]
linux compile fix
Marcus Brinkmann [Fri, 26 Feb 2010 01:34:54 +0000 (02:34 +0100)]
Generate Windows def files from def.in files.
Romain Pokrzywka [Tue, 23 Feb 2010 00:32:44 +0000 (16:32 -0800)]
fix potential crashes on windows with MSVC, due to badly exported variables
Romain Pokrzywka [Sat, 13 Feb 2010 01:59:14 +0000 (19:59 -0600)]
--warning
Romain Pokrzywka [Tue, 9 Feb 2010 03:16:05 +0000 (21:16 -0600)]
small fixes for DBUS_USE_OUTPUT_DEBUG_STRING code
Ralf Habacker [Thu, 25 Feb 2010 17:12:47 +0000 (18:12 +0100)]
fixed compile break introduced in commit
e1c31c73074513d96fa22b5c0355107c42720597
Colin Walters [Mon, 22 Feb 2010 14:48:45 +0000 (09:48 -0500)]
Merge branch 'dbus-1.2'
Conflicts:
bus/bus.c
bus/selinux.c
configure.in
Colin Walters [Fri, 19 Feb 2010 21:34:47 +0000 (16:34 -0500)]
[dbus-string] Sync up UNICODE_VALID with glib, add documentation
See https://bugzilla.gnome.org/show_bug.cgi?id=107427 for rationale
behind the first change.
The documentation was derived from an IRC conversation with
Behdad Esfahbod.
Colin Walters [Fri, 19 Feb 2010 17:39:56 +0000 (12:39 -0500)]
Finish moving file-related functions from dbus-sysdeps-unix to dbus-file-unix
Colin Walters [Thu, 18 Feb 2010 20:33:28 +0000 (15:33 -0500)]
Monitor service directories for changes
It's not expected to have to manually SIGHUP the bus after installing
a new .service file. Since our directory monitoring is already set
up to queue a full reload which includes service activation, simply
monitor the servicedirs too.
https://bugs.freedesktop.org/show_bug.cgi?id=23846
Brian Cameron [Thu, 18 Feb 2010 15:38:42 +0000 (10:38 -0500)]
Fix dummy file monitoring backend compilation
https://bugs.freedesktop.org/show_bug.cgi?id=26421
Colin Walters [Thu, 18 Feb 2010 15:09:20 +0000 (10:09 -0500)]
Revert "_dbus_string_to_lower(): new function"
This reverts commit
39aa8090241b2588eeb2df05880d71abaf8c526e.
Ralf Habacker [Fri, 12 Feb 2010 07:47:33 +0000 (08:47 +0100)]
_dbus_string_to_lower(): new function
Ralf Habacker [Fri, 12 Feb 2010 07:46:10 +0000 (08:46 +0100)]
Merge branch 'master' of ssh://git.freedesktop.org/git/dbus/dbus
Marcus Brinkmann [Thu, 11 Feb 2010 16:04:10 +0000 (17:04 +0100)]
Enable dbus-launch for Windows target.
Ralf Habacker [Wed, 10 Feb 2010 12:03:19 +0000 (13:03 +0100)]
added some docs for windows and updated the documentation index
Ralf Habacker [Wed, 10 Feb 2010 08:26:52 +0000 (09:26 +0100)]
fixed xml doc generating for cmake build system - supported docbook generator is currently kde's meinproc4, xmlto may also work but need still some checks
Ralf Habacker [Tue, 9 Feb 2010 18:44:12 +0000 (19:44 +0100)]
Merge branch 'master' of ssh://git.freedesktop.org/git/dbus/dbus
Ralf Habacker [Tue, 9 Feb 2010 07:37:50 +0000 (08:37 +0100)]
fixed bug on win32 not been able to use session bus with --config-file option
Internal setup of session bus type was bound to the presence of the --session
command line parameter which prevents to use the --config-file parameter for
session bus setup.
Cyril Brulebois [Mon, 8 Feb 2010 17:21:35 +0000 (12:21 -0500)]
Fix compilation of kqueue file monitoring on FreeBSD
Romain Pokrzywka [Fri, 5 Feb 2010 08:54:16 +0000 (09:54 +0100)]
variable name fix (DBUS_DAEMON_NAME instead of DAEMON_NAME)
Romain Pokrzywka [Fri, 5 Feb 2010 00:36:57 +0000 (16:36 -0800)]
apply remaining unapplied parts of
eeedba5b2f5b008819619764943caddd52441adf in dbus4win
Romain Pokrzywka [Fri, 5 Feb 2010 00:13:14 +0000 (16:13 -0800)]
not needed anymore. Builds fine without for both msvc and mingw
Romain Pokrzywka [Thu, 4 Feb 2010 23:20:40 +0000 (15:20 -0800)]
add new symbols from 1.3 to the exports
Romain Pokrzywka [Thu, 4 Feb 2010 21:49:39 +0000 (13:49 -0800)]
move dbus-1.def.cmake to the cmake subdir, as it's cmake specific
Romain Pokrzywka [Thu, 4 Feb 2010 01:31:50 +0000 (17:31 -0800)]
add the LIBRARY header in the .def file, but for mingw only, msvc doesn't support it anymore. It's still needed when creating a .lib from a mingw-built dll
Romain Pokrzywka [Wed, 3 Feb 2010 20:27:47 +0000 (12:27 -0800)]
remove DBUS_DAEMON_EXECUTABLE_NAME, we already have DBUS_DAEMON_NAME, and use the specified name for the executable when building with cmake
Romain Pokrzywka [Wed, 3 Feb 2010 19:32:37 +0000 (11:32 -0800)]
remove the debug postfix for the executables, it only makes sense for the lib
Romain Pokrzywka [Wed, 3 Feb 2010 01:48:55 +0000 (17:48 -0800)]
and another .def.in for the verbose mode handling
Romain Pokrzywka [Wed, 3 Feb 2010 01:31:43 +0000 (17:31 -0800)]
commented bits cleanup
Romain Pokrzywka [Wed, 3 Feb 2010 01:18:17 +0000 (17:18 -0800)]
disable cmake options that don't apply (for cmake-gui)
Romain Pokrzywka [Tue, 2 Feb 2010 23:24:48 +0000 (15:24 -0800)]
fixed the TODO, the if block didn't make sense indeed. Also make it clearer which dirs in the summary are installation dirs
Romain Pokrzywka [Tue, 2 Feb 2010 23:12:30 +0000 (15:12 -0800)]
fixed and simplified install prefix handling with cmake
Romain Pokrzywka [Tue, 2 Feb 2010 21:52:58 +0000 (13:52 -0800)]
dbus-spawn.h is internal, it shouldn't be installed
Romain Pokrzywka [Tue, 2 Feb 2010 20:31:31 +0000 (12:31 -0800)]
now fix the build for the tests; turns out we need another set of exports, since some tests are compiled in the client lib and some are in the internal lib...
Romain Pokrzywka [Tue, 2 Feb 2010 19:13:26 +0000 (11:13 -0800)]
refactor the cmake build to match with autotools: only export client symbols in dbus-1 and use and internal library for the rest. Currently this library is statically linked to the apps but it can be made dynamic if wanted
Romain Pokrzywka [Tue, 2 Feb 2010 00:24:34 +0000 (16:24 -0800)]
oops, this symbol belongs to the client exports
Romain Pokrzywka [Sat, 30 Jan 2010 03:18:37 +0000 (19:18 -0800)]
split the symbols into separate files matching the client/shared/util sections of the code, and generate the final .def file dynamically
Romain Pokrzywka [Sat, 30 Jan 2010 01:49:37 +0000 (17:49 -0800)]
make the def handling simpler with cmake, we don't need to specify the library name anyway
Ralf Habacker [Sat, 6 Feb 2010 23:32:13 +0000 (00:32 +0100)]
win32 fix: _dbus_get_install_root(): there are more places to fix case insensitive pattern search
Ralf Habacker [Sat, 6 Feb 2010 23:29:10 +0000 (00:29 +0100)]
win32 fix: _dbus_get_install_root(): to catch all cases perform case insensitive pattern search
Colin Walters [Wed, 3 Feb 2010 17:13:38 +0000 (12:13 -0500)]
Release 1.2.20
Colin Walters [Tue, 2 Feb 2010 17:37:17 +0000 (12:37 -0500)]
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)
Ralf Habacker [Wed, 3 Feb 2010 13:14:03 +0000 (14:14 +0100)]
linux cmake build system related compile fix
Colin Walters [Wed, 3 Feb 2010 07:36:38 +0000 (08:36 +0100)]
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.
Colin Walters [Tue, 2 Feb 2010 19:57:47 +0000 (14:57 -0500)]
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.
Colin Walters [Tue, 2 Feb 2010 17:37:17 +0000 (12:37 -0500)]
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.
Colin Walters [Tue, 2 Feb 2010 16:00:09 +0000 (11:00 -0500)]
Merge branch 'my-dbus-1.2'
Conflicts:
configure.in
Colin Walters [Tue, 2 Feb 2010 15:31:28 +0000 (10:31 -0500)]
Release 1.2.18
Tom Hughes [Sun, 13 Dec 2009 21:30:09 +0000 (13:30 -0800)]
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>
Ralf Habacker [Tue, 2 Feb 2010 13:26:50 +0000 (14:26 +0100)]
win32 fix: provide separate def file for cmake until there is full splitlib support for cmake
Ralf Habacker [Tue, 2 Feb 2010 13:14:20 +0000 (14:14 +0100)]
win32 fix for commit
8a9880ffd2b81df38bb0e3492bda7a9636ac0280 - Clean up inotify watch handling
Colin Walters [Mon, 1 Feb 2010 22:38:25 +0000 (17:38 -0500)]
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
Colin Walters [Mon, 1 Feb 2010 22:27:26 +0000 (17:27 -0500)]
Merge branch 'my-dbus-1.2'
Colin Walters [Mon, 1 Feb 2010 21:45:48 +0000 (16:45 -0500)]
Move system-activation.txt into non-conditional EXTRA_DIST
It doesn't depend on the XML doc generation.
Colin Walters [Thu, 28 Jan 2010 21:26:39 +0000 (16:26 -0500)]
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.
Ralf Habacker [Fri, 29 Jan 2010 22:57:07 +0000 (23:57 +0100)]
_dbus_change_to_daemon_user() is platform related and commit
0a3905d7f3b2ff43b09479863775939f9c8acad4 broke windows builds
Ralf Habacker [Fri, 29 Jan 2010 22:48:34 +0000 (23:48 +0100)]
reverted
2aea64fbb69287f52bb9910e89b6732854f47be8 because it breaks msvc2008 builds
Colin Walters [Thu, 28 Jan 2010 22:31:36 +0000 (17:31 -0500)]
Update messagebus.in init script to start earlier
Patch from Fedora.
Colin Walters [Thu, 28 Jan 2010 22:28:25 +0000 (17:28 -0500)]
[doc] diagram.[svg,png] to EXTRA_DIST
These files are part of the docs and should appear in tarballs.
Colin Walters [Thu, 28 Jan 2010 22:09:15 +0000 (17:09 -0500)]
Merge branch 'my-dbus-1.2'
Conflicts:
bus/Makefile.am
dbus/dbus-sysdeps-util-unix.c
dbus/dbus-transport.c
test/test-service.c
Colin Walters [Thu, 28 Jan 2010 20:04:14 +0000 (15:04 -0500)]
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>
Colin Walters [Thu, 28 Jan 2010 00:38:44 +0000 (19:38 -0500)]
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