summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Prepare version 1.4.10dbus-1.4.10Simon McVittie2011-06-012-3/+3
|
* NEWSSimon McVittie2011-05-251-0/+2
|
* bus_context_log: divert messages to stderr if we're not using syslogSimon McVittie2011-05-251-2/+13
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35358 Reviewed-by: Colin Walters <walters@verbum.org>
* process_config_first_time_only: initialize syslog as intended.Simon McVittie2011-05-253-3/+4
| | | | | | | | | | | | | | On Linux, we previously called openlog() (with different options!) while initializing SELinux; leave SELinux messages as LOG_USER|LOG_INFO in case anyone was relying on that, but let the rest of our log messages come out as LOG_DAEMON. Also enable LOG_PERROR (copy syslog messages to stderr) unconditionally; we can make this an autoconf check if anyone's pet Unix doesn't have LOG_PERROR. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35358 Reviewed-by: Colin Walters <walters@verbum.org>
* _dbus_init_system_log: remove FIXME on WindowsSimon McVittie2011-05-251-1/+1
| | | | | | | We don't need any initialization here, so there's nothing to fix. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35358 Reviewed-by: Colin Walters <walters@verbum.org>
* bus_context_check_security_policy: syslog if we hit the outgoing quotaSimon McVittie2011-05-251-10/+13
| | | | | | | | In practice, nothing copes with missing broadcast signals, so the least we can do is make the failure mode visible. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35358 Reviewed-by: Colin Walters <walters@verbum.org>
* bus_context_check_security_policy: for SELinux denials, share code to set errorsSimon McVittie2011-05-251-15/+6
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35358 Reviewed-by: Colin Walters <walters@verbum.org>
* bus_context_check_security_policy: factor out complain_about_messageSimon McVittie2011-05-251-98/+85
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35358 Reviewed-by: Colin Walters <walters@verbum.org>
* Check for X even if X11 autolaunching is disabledSimon McVittie2011-05-255-11/+24
| | | | | | | | | | | | | | | | | DBUS_ENABLE_X11_AUTOLAUNCH obviously requires DBUS_BUILD_X11. However, the converse is not true. If DBUS_BUILD_X11 is defined, dbus-launch will be able to connect to the X server to determine when the session ends; most distributors will want this, but it can be disabled with the standard Autoconf option --without-x. If DBUS_ENABLE_X11_AUTOLAUNCH is *also* defined, dbus-launch and libdbus will be willing to perform autolaunch. Again, most distributors will want this, but it can be disabled with --disable-x11-autolaunch. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=19997 Reviewed-by: Colin Walters <walters@verbum.org>
* test-autolaunch: don't expect autolaunching to work if X11 is disabledSimon McVittie2011-05-251-0/+19
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=19997 Reviewed-by: Colin Walters <walters@verbum.org>
* NEWSSimon McVittie2011-05-251-0/+2
|
* When checking for __sync_sub_and_fetch, don't underquote, to shut up recent ↵Simon McVittie2011-05-251-2/+2
| | | | | | | | | | | autoconf Without the correct number of levels of quoting, autoconf mistakenly believes we didn't use AC_LANG_SOURCE where required. (In fact, AC_LANG_PROGRAM calls AC_LANG_SOURCE.) Bug: https://bugs.freedesktop.org/show_bug.cgi?id=19681 Reviewed-by: Colin Walters <walters@verbum.org>
* When checking for va_copy, use AC_LANG_SOURCE to shut up recent autoconfSimon McVittie2011-05-251-4/+4
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=19681 Reviewed-by: Colin Walters <walters@verbum.org>
* Use TP_ADD_COMPILER_FLAG to simplify application of warning-like CFLAGSSimon McVittie2011-05-251-46/+14
| | | | | | | This also means we check for support for them. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=19681 Reviewed-by: Colin Walters <walters@verbum.org>
* Use TP_COMPILER_WARNINGS for all -Wfoo optionsSimon McVittie2011-05-251-84/+34
| | | | | | | | This consistently checks whether all these options actually work in the current version of gcc. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=19681 Reviewed-by: Colin Walters <walters@verbum.org>
* Import tp-compiler-flag.m4 and tp-compiler-warnings.m4 from telepathy-glibSimon McVittie2011-05-252-0/+92
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=19681 Reviewed-by: Colin Walters <walters@verbum.org>
* NEWSSimon McVittie2011-05-251-0/+6
|
* handle_server_data_anonymous_mech: remove unnecessary debug outputSimon McVittie2011-05-251-12/+0
| | | | | | | | | | | Doing a malloc and a hex-encoding pass just to produce a _dbus_verbose message (i.e. a message that, in practice, nobody will see) seems like overkill, and this block had incorrect error handling (not checking the result of _dbus_string_init) which upsets static analysis tools. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=29881 Bug-NB: NB#180486 Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
* dbus-launch: pass_info: always free strings on OOMSimon McVittie2011-05-251-7/+10
| | | | | | | | | This doesn't really do anything, because we're about to exit anyway, but it placates static analysis tools. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=29881 Bug-NB: NB#180486 Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
* dbus-send: remove minor dead codeSimon McVittie2011-05-251-3/+1
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33128 Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
* dbus_connection_dispatch: remove dead codeSimon McVittie2011-05-251-3/+0
| | | | | | | | | There's no way pending can be non-NULL here; if it was, we'd have jumped straight past this block (getting filters from the connection), because replies to pending calls don't go through filters. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33128 Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
* Spelling fix.Ralf Habacker2011-05-251-2/+2
|
* Definition is obsolate with commit e0fc90bdf2c60a4c77fffaf490d6f06b2bd6999dRalf Habacker2011-05-251-5/+0
|
* To avoid double dbus version definitions take values from configure.ac.Ralf Habacker2011-05-251-3/+7
|
* DBusNonceFile: don't always fail when use_subdir=TRUE, i.e. on UnixSimon McVittie2011-05-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | In shell-like pseudocode, the desired result is: if $use_subdir dir = $tmpdir/dbus_nonce-$random path = $dir/nonce mkdir $dir write file at $path else dir = $user_owned_place path = $dir/dbus_nonce-$random write file at $path However, /nonce was accidentally appended to $dir instead of $path, resulting in an attempt to mkdir /tmp/dbus_nonce-XXXX/nonce when dbus_nonce-XXXX hadn't been created yet. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=34569 Tested-by: David Zeuthen <davidz@redhat.com> Reviewed-by: David Zeuthen <davidz@redhat.com>
* Relax review criteria for the review cabal themselves, as discussed on-listSimon McVittie2011-05-251-0/+14
| | | | Colin agreed in principle and nobody actually objected, so here we go...
* Add davidz to the review cabalSimon McVittie2011-05-251-2/+1
| | | | | Acked-by: Will Thompson <will.thompson@collabora.co.uk> Acked-by: Colin Walters <walters@verbum.org>
* dbus_connection_can_send_type: clarify that invalid codes are allowed hereSimon McVittie2011-05-031-1/+1
| | | | | | | This is sufficiently unusual that it seems worth saying explicitly Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35182 Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
* NEWSSimon McVittie2011-04-291-0/+2
|
* minotaur: bail if asked to monitor >1 busWill Thompson2011-04-291-11/+35
| | | | | | | | | | A coworker was just tripped up by `dbus-monitor --session --system` only monitoring the system bus. This patch would have saved him reproducing a tricky bug several times! Bug: https://bugs.freedesktop.org/show_bug.cgi?id=26548 Reviewed-by: Colin Walters <walters@verbum.org> Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
* NEWS for 1.4Simon McVittie2011-04-271-0/+4
|
* machine-id: fall back to reading /etc/machine-id if D-Bus machine ID is not ↵Lennart Poettering2011-04-271-1/+12
| | | | | | | | | | | | | | | | | | | | available Newer systemd-based systems support a global machine ID with the same metrics as the D-Bus machine ID, but more powerful semantics (for example on ro media) in /etc/machine-id. If the D-Bus machine ID cannot be read, fall back to the systemd machine ID. This is a first step towards allowing D-Bus to be started up during early boot where /var is not available. [plus a whitespace fix -smcv] Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Reviewed-by: Colin Walters <walters@verbum.org> Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35228
* _dbus_check_fdleaks_enter, _dbus_check_fdleaks_leave: whitespaceSimon McVittie2011-04-271-12/+12
| | | | | Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35173 Reviewed-by: Colin Walters <walters@verbum.org>
* Don't report file descriptors as "leaked" if they were already openSimon McVittie2011-04-273-10/+90
| | | | | | | | This is necessary to run the regression tests under valgrind (if telling it to output to a dedicated fd), gdb, fakeroot etc. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35173 Reviewed-by: Colin Walters <walters@verbum.org>
* NEWSSimon McVittie2011-04-261-0/+2
|
* Merge remote-tracking branch 'origin/dbus-1.4' into dbus-1.4Simon McVittie2011-04-261-8/+60
|\
| * activation: Use _dbus_system_log for activation informationColin Walters2011-04-261-8/+60
| | | | | | | | | | | | | | Log when we are activating something (and whether it's via systemd) as well as when we fail to activate, and when one succeeds. https://bugs.freedesktop.org/show_bug.cgi?id=35705
* | NEWSSimon McVittie2011-04-261-0/+7
| |
* | Add dbus-1-uninstalled.pc.inSimon McVittie2011-04-262-0/+18
| | | | | | | | | | | | | | See: http://smcv.pseudorandom.co.uk/2008/09/pc-uninstalled/ Bug: https://bugs.freedesktop.org/show_bug.cgi?id=32827 Reviewed-by: Colin Walters <walters@verbum.org>
* | Add support for --disable-gc-sections for broken toolchainsSimon McVittie2011-04-261-38/+26
|/ | | | | | | | Also use AC_LINK_IFELSE rather than reinventing it as a shell function. This was the last user of ld_supports_flag, so, delete it. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=33466 Reviewed-by: Colin Walters <walters@verbum.org>
* Drop outdated list of options from README, mention configure --help insteadSimon McVittie2011-04-261-25/+2
|
* Release notes for the -fPIE changeSimon McVittie2011-04-261-1/+14
|
* Don't force use of -fPIE for the dbus-daemon if apparently supportedSimon McVittie2011-04-263-36/+10
| | | | | | | | | | | | | | | | | | | | | It's a minor security benefit, but not automatically beneficial (it enables ASLR, but breaks prelinking, some buggy toolchains, and some gdb versions). Distributions who know their infrastructure works well can enable it just as easily via ./configure CFLAGS="-fPIE" LDFLAGS="-pie" without extra support from us, and that's a generic solution applicable to many packages. Similarly, don't force libdbus and libdbus-internal to be PIC: libtool knows better than we do whether that's necessary/beneficial on a particular platform. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=16621 Bug: https://bugs.freedesktop.org/show_bug.cgi?id=27215 Bug-NB: NB#171940 Reviewed-by: Colin Walters <walters@verbum.org>
* Install dbus-daemon and dbus-daemon-launch-helper in the conventional waySimon McVittie2011-04-261-41/+27
| | | | | | | | | | | | | | Also remove some pointless indirection (extra_tests etc.), don't install the unused directory $(libexecdir)/dbus-1 (we actually install the launch helper directly into $(libexecdir)), and allow dbus_daemondir to be set on Windows rather than forcing dbus-daemon to be installed to $(bindir) there. dbus_daemon_execdir has to contain "exec" so that the dbus-daemon will be installed by "make install-exec" and not "make install-data". Bug: https://bugs.freedesktop.org/show_bug.cgi?id=14512 Reviewed-by: Colin Walters <walters@verbum.org>
* When uploading docs, use rsync -p to set permissionsSimon McVittie2011-04-121-1/+1
| | | | | | | | | | | | | | It turns out that rsync --chmod means "pretend the source files had already had this chmod operation applied to them", and not "chmod the destination files" like you'd expect. As a result, the -p (--perms) option is also needed, so that rsync will "preserve" the modified permissions. Otherwise, the docs will not be group-writeable as intended, and only the person who made the previous upload will be able to upload them next time. Bug: https://bugs.freedesktop.org/show_bug.cgi?id=36130 Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>
* development versionSimon McVittie2011-04-082-1/+6
|
* Prepare version 1.4.8dbus-1.4.8Simon McVittie2011-04-082-3/+13
|
* activation: Strip out code to compare by Exec=Colin Walters2011-04-081-18/+4
| | | | | | | | | | | | | | | | | | | | | | In commit: 075945f6 (John (J5) Palmieri 2005-07-14 20:44:15 +0000 some code was added to compare services by Exec key. The changelog is not pariticularly informative as to why this was added. But while debugging other code, we noticed this. Comparing by Exec key is not in the specification, and triggered a problem where while converting services to use systemd for activation, a change was made to use Exec=/bin/false and simply rely on systemd to activate. While I think it was broken for the service files to be changed to Exec=/bin/false, we shouldn't be doing something here that's not in the spec either. Reviewed-by: Will Thompson <will.thompson@collabora.co.uk> Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35750
* oops, fix XML mis-nestingSimon McVittie2011-04-071-0/+1
|
* list which parts of the Desktop Entry spec apply to service filesSven Herzberg2011-04-071-3/+22
| | | | | Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk> Bug: https://bugs.freedesktop.org/show_bug.cgi?id=19159
View Lorry Controller Status