summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* DBusMessage: add support for custom marshalingbaserock/dbus-1.10.6-capi-patchesAleksandar Kanchev2016-02-224-44/+132
| | | | | | | Add functions to support querying and manipulating the message body and signature. This is useful for code generators, which can generate custom marshaling functions based on a given IDL. Those functions tend to be optimized and faster than the generic iterator based marshaling.
* Connection block pending callPedro Alvarez2016-02-221-1/+1
|
* Block acquire io path on sendPedro Alvarez2016-02-221-2/+1
|
* Add dbus_connection_send_with_reply_set_notify function for proper thread ↵Jürgen Gehring2016-02-222-0/+165
| | | | handling
* 1.10.6dbus-1.10.6Simon McVittie2015-12-012-3/+5
|
* NEWS for 1.10 branchSimon McVittie2015-11-231-0/+13
|
* uid-permissions test: don't assert that root can UpdateActivationEnvironmentSimon McVittie2015-11-231-4/+79
| | | | | | | | | | | | | Since 1.10.4 this is hard-coded to be disallowed when an activation helper is used. That would be a security flaw waiting to happen, and makes little sense anyway, because the activation helper sanitises its environment. Use BecomeMonitor() instead, as our way to assert that root and messagebus are privileged. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=93036 Reviewed-by: Iain Lane
* name-test: don't run these tests with autotools if targeting Windows.Simon McVittie2015-11-171-3/+8
| | | | | | | | | | | The wrapper shell script that sets up their environment is nowhere near being portable. In particular, it uses dbus-run-session, which is Unix-specific. [rh: Add autotools scope in commit title] Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92899 Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Rewieved-by: Ralf Habacker <ralf.habacker@freenet.de>
* Windows _dbus_get_autolaunch_address: don't leak shm_nameSimon McVittie2015-11-181-0/+1
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92899 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
* 1.10.5Simon McVittie2015-11-172-1/+4
|
* dbus 1.10.4dbus-1.10.4Simon McVittie2015-11-172-3/+5
|
* NEWSSimon McVittie2015-11-171-1/+30
|
* bus-driver: Support returning org.freedesktop.DBus UID and PIDJan Alexander Steffens (heftig)2015-11-174-56/+127
| | | | | | | | | | | Attempting to call SetEnvironment on systemd causes it to inquire about the caller's connection UID and PID. If this check fails, the call is rejected. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92857 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk> [smcv: go back to DBUS_ERROR_UNIX_PROCESS_ID_UNKNOWN as the error code for failure to determine the pid]
* test/sd-activation: Test UpdateActivationEnvironment forwardingJan Alexander Steffens (heftig)2015-11-171-1/+211
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92857 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* bus_driver_handle_update_activation_environment: Forward to systemdJan Alexander Steffens (heftig)2015-11-171-3/+152
| | | | | | | | | | | | | If we use systemd activation, forward all UpdateActivationEnvironment requests to org.freedesktop.systemd1.Manager.SetEnvironment, in order to ensure variables needed by D-Bus services are available when these services are launched by systemd. Since UpdateActivationEnvironment is not available on the system bus, this only applies to user buses. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92857 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* bus_driver_handle_update_activation_environment: Error on system busesJan Alexander Steffens (heftig)2015-11-171-0/+11
| | | | | | | | | The default policy already disallows calls on system buses. Since any bus with a service helper cleans the environment anyway, there's no point in allowing this to be called. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92857 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* fdpass test: assert that TCP connections don't say they can pass fdsSimon McVittie2015-11-171-18/+58
| | | | | | | | | | | As well as slightly increasing our test coverage, this ensures that at least one test-case in this test is not skipped, working around CMake's lack of support for the Automake-style exit code produced by GTest executables (they exit 77 if everything was skipped). Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92887 Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
* Fix recursive loop in backtrace generator on windows.Ralf Habacker2015-11-161-1/+1
| | | | | | | | Backtrace generator called _dbus_warn(), which calls backtrace generator recursively with DBUS_FATAL_WARNINGS=1. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Do not fail with fatal message skipping GetConnectionUnixProcessID test-bus ↵Ralf Habacker2015-11-131-1/+1
| | | | | | | test on windows. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Fix test-bus test for GetConnectionUnixUser driver method on windows.Ralf Habacker2015-11-131-4/+9
| | | | | | | | Because GetConnectionUnixUser is not supported on windows it fails with DBUS_ERROR_FAIL. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Skip launch helper activation tests on windows silently.Ralf Habacker2015-11-121-3/+1
| | | | | | | | | The system bus is unsupported there and Windows does not have any concept of setuid binaries, so it can't ever actually work. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Fix memory leak in _dbus_win_set_error_from_win_error().Ralf Habacker2015-11-111-6/+1
| | | | | | | | There is no need to make a local (leaked) copy of the message, because dbus_set_error() already makes a copy of its parameters. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Fix memory leaks in bus_activation_service_reload_test() in case of errors.Ralf Habacker2015-11-111-8/+9
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Revert "Fix memory leaks in bus_activation_service_reload_test() in case of ↵Ralf Habacker2015-11-111-9/+8
| | | | | | errors." This reverts commit d8569ac647da4d62c7a251d8e809f2d545a77837.
* Fix memory leaks in bus_activation_service_reload_test() in case of errors.Ralf Habacker2015-11-111-8/+9
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: https://bugs.freedesktop.org/show_bug.cgi?id=92721
* Fix test-bus segfault_service_no_auto_start test on windows.Ralf Habacker2015-11-111-11/+16
| | | | | | | | | | | Windows returns unhandled exceptions from a running child by specific exit codes and not by signals as on UNIX. Therefore we use DBUS_ERROR_SPAWN_CHILD_EXITED for propagating unhandled exceptions to the parent too. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* update NEWSSimon McVittie2015-11-061-1/+14
|
* Fix test cases running client and server dispatch design issue.Ralf Habacker2015-11-061-5/+34
| | | | | | | | | | DBus test cases running the server *and* client loop in the same process assumed that all messages send from the server has to be received in one client dispatch, which is not the case in all environments. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Fix bug unrefing connection too early in check_hello_message().Ralf Habacker2015-11-061-2/+2
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Compile fix on Windows.Ralf Habacker2015-11-041-1/+1
| | | | This commit fixes a regression introduced with commit 04b8a7a772cfa9ae8ea6ce452d1fb7f23e25fd3f.
* Fix warning: variable 'ret' set but not used [-Wunused-but-set-variable].Ralf Habacker2015-11-031-4/+7
| | | | | | | | Only set valid child status in case exit code has been gotten from child process, otherwise signal failure through thread return value. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Test system bus config files on Unix onlyRalf Habacker2015-11-038-33/+94
| | | | | | | | | | | | | | | | | | Previously, we didn't consistently test parsing of every file in valid-config-files-system/ everywhere that we tested valid-config-files/. We now test it on Unix. The system bus is not supported on Windows, so we do not test valid-config-files-system/ there. valid-config-files/many-rules.conf contains <user> and <group> rules which are not applicable to Windows. Copy the original many-rules.conf to valid-config-files-system/ so that it will be tested on Unix, and remove the non-portable rules from valid-config-files/many-rules.conf. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk> [rh:base patch came from Simon]
* Don't use _dbus_warn() for intentionally-skipped testsSimon McVittie2015-11-021-2/+5
| | | | | | | | The tests are run with _dbus_warn() fatal, so if a particular test is not applicable on the current platform, we shouldn't call it. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
* test_default_session_servicedirs: use the intended data directorySimon McVittie2015-11-021-13/+22
| | | | | | | | | | | | | | | If D-Bus was configured for /usr/local and built in Z:/build, the previous code would use Z:/build/dbus/.libs/usr/local/share/dbus-1/services whereas the intention was to replace the configured prefix /usr/local with the detected location, more like Z:/build/dbus/.libs/share/dbus-1/services Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
* test_default_session_servicedirs: simplify to a single exit code-pathSimon McVittie2015-11-021-25/+20
| | | | | | | | A similar simplification was already done on master as part of commit f830e14, Bug #83539. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
* Fix warning: ISO C90 forbids mixed declarations and code ↵Ralf Habacker2015-11-021-1/+2
| | | | | | | | | [-Wdeclaration-after-statement]. Includes minor indention fix. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Fix typo and missing eol on debug message.Ralf Habacker2015-11-021-1/+1
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92721 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* refs test: reduce number of repeats under WineSimon McVittie2015-10-291-32/+68
| | | | | | | | | | | | Under Wine, the API calls we use to do this are implemented via IPC to wineserver, which makes it unreasonably slow to try to brute-force bugs by having many threads stress-test refcounting. Do a few repetitions just to verify that refcounting basically works, but don't do the full stress-test. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92538 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de> [rh: add WINESERVERSOCKET related hint]
* When running TAP tests, filter out trailing \r from Windows .exeSimon McVittie2015-10-291-1/+21
| | | | | | | | If we're running Windows executables using Wine, then tap-driver.sh won't accept "1..4\r\n" as TAP syntax. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92538 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
* Fix crash running test-syslog on windows/wine.Ralf Habacker2015-10-271-3/+4
| | | | | | | Use seperate buffer in _dbus_system_logv() to avoid crash in vsprintf(). Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92538 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* Display autolaunch scope on verbose print of daemon found message on windows.Ralf Habacker2015-10-271-2/+2
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92080 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* 1.10.3Simon McVittie2015-10-262-1/+6
|
* 1.10.2dbus-1.10.2Simon McVittie2015-10-262-3/+5
|
* Add a regression test for invalid BecomeMonitor callsSimon McVittie2015-10-261-0/+151
| | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92298
* NEWSSimon McVittie2015-10-261-0/+7
|
* user dbus.socket: Set DBUS_SESSION_BUS_ADDRESSJan Alexander Steffens (heftig)2015-10-262-0/+7
| | | | | | | | | | | | As discussed in <https://github.com/systemd/systemd/issues/1600>. See also <https://bugs.archlinux.org/task/46721>, <https://bugzilla.gnome.org/show_bug.cgi?id=756420> Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92612 [smcv: use AC_PATH_PROG to find systemctl; ignore systemctl failure] Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Reviewed-by: David Herrmann <dh.herrmann@gmail.com> [smcv: add links to earlier bug reports elsewhere]
* When running dbus-daemon --session in tests, override listen addressSimon McVittie2015-10-261-12/+32
| | | | | | | | | | | Otherwise, we can't reliably run tests for Windows, because the default listening address on Windows is "autolaunch:" which is global to a machine, resulting in testing an installed dbus-daemon instead of the one we intended to test. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92538 Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
* Disable activation tests on Windows buildsSimon McVittie2015-10-222-1/+20
| | | | | | | | These rely on the --systemd-activation code path, which is not compiled for Windows. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92538 Reviewed-by: Ralf Habacker <ralf.habacker@freenet.de>
* Keep cmake build system in sync with autotools (add test-monitor).Ralf Habacker2015-10-211-0/+1
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=92298 Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* NEWSSimon McVittie2015-10-191-0/+5
|