doc: Introduce XML catalog

This adds a nice way of loading the DTD files based on the identifier
in the DOCTYPE declaration, no matter where the DTDs are installed.
See also ‘XML catalog’ Wikipedia entry and update-xmlcatalog(8).

Merge branch 'add-confidential-mr-doc' into 'master'

CONTRIBUTING.md: creation of "confidential" merge requests added

See merge request dbus/dbus!252

CONTRIBUTING.md: creation of "confidential" merge requests added

See #368

Merge branch 'fix-issue-377' into 'master'

cmake: fix for ninja phony cycle related configuring errors

Closes #377

See merge request dbus/dbus!253

cmake: fix for ninja phony cycle related configuring errors

A prefix has been added to the custom target names in docbook-related
macros to fix the reported error:

  ninja: error: build.ninja:xxx: multiple rules generate
  doc/dbus-xxx.1.html [-w dupbuild=err]

Fixes #377

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

cmake: drop obsolete dependencies for generated docbook source files

The docbook source files are already dependencies of the generated
docbook files.

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

cmake: let macro docbook handle all requested formats in one call

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

cmake: in macro generate_docbook_file use easier to read parameter names

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

Merge branch 'dbus-server-signal-config-reloaded' into 'master'

dbus-daemon: Inform dbus clients about configuration changes

Closes #376

See merge request dbus/dbus!250

dbus-daemon: Implement signal 'ActivatableServicesChanged'

After any reload of the activatable service files the mentioned signal is
emitted to the current bus to inform clients.

The calls to signal emmission have not been implemented in the platform
specific functions _dbus_daemon_report_reloaded() to avoid duplicate
implementations.

Fixes #376
Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

dbus-specification: Add signal 'ActivatableServicesChanged' and related bus feature

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

Merge branch 'address-escape-doc' into 'master'

spec: add * to optionally-escaped bytes set

See merge request dbus/dbus!248

spec: add * to optionally-escaped bytes set

Commit ee71e1ff6033a added * to the list of optionally escaped bytes
set, but did not update the documentation. I guess this happened because
the change is not backward compatible.

It seems that the period of 14+ years should be enough to not cause any
backward compatibility issues, so let's document this.

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>

Merge branch 'ci-fix' into 'master'

docker/windows: fix bad path from MR !218

See merge request dbus/dbus!247

docker/windows: fix bad path from MR !218

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

Merge branch 'win-ci' into 'master'

CI: add Windows native builds

See merge request dbus/dbus!218

gitlab-ci: add cmake Windows builds with VS and mingw-ucrt

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gitlab-ci: build a Windows docker image, with VS15 & msys

Based on GStreamer & Mesa freedesktop gitlab CI, use the Windows 1809
runner provided by the GStreamer Foundation.

https://www.freedesktop.org/wiki/Infrastructure/

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gitlab-ci: move "before_script" and "cache" to .debian-build

As we are introducing Windows builds next, move out Unix-specific
job definitions to the base job.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

gitlab-ci: factorize build jobs with "extends"

It is easier to read and modify than anchors. And we are going to move
more Debian-specific to the base ".debian-build" next.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

bus/signal: fix compilation when only -Dembedded-tests

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

cmake: add -fno-common when !MSVC

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

cmake: fix warnings variables

Do not use strings for compiler flags.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

Merge branch 'badoom' into 'master'

auth: fix a no-memory case

See merge request dbus/dbus!246

auth: fix a no-memory case

Jumping to "out" will call _dbus_string_zero() unconditionally on tmp
string variables. Do not attempt to zero what wasn't even initialized.

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

Merge branch 'cmake-msvc-fixes' into 'master'

cmake: do not use gcc specific warning when building with msvc

See merge request dbus/dbus!245

cmake: do not use gcc specific warning when building with msvc

NEWS: Mention CMake 3.4 dependency

Signed-off-by: Simon McVittie <smcv@collabora.com>

Bump version number for continued development

Signed-off-by: Simon McVittie <smcv@collabora.com>

doc: Uninstall *.svg generated by Doxygen

Signed-off-by: Simon McVittie <smcv@collabora.com>

Prepare v1.13.20

Signed-off-by: Simon McVittie <smcv@collabora.com>

Update AUTHORS from git log

Signed-off-by: Simon McVittie <smcv@collabora.com>

Merge branch 'fix-issue-365' into 'master'

Fix memory leaks in _dbus_get_install_root_as_hash in case of OOM condition

Closes #365

See merge request dbus/dbus!239

In _dbus_get_autolaunch_address() avoid uninitialized mutex

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

Cleanup zero string length case condition in Windows helper functions

This cleanup has been processed in _dbus_daemon_publish_session_bus_address()
_dbus_get_install_root_as_hash() and _dbus_get_address_string().

The zero length condition has been catched in each function calling
the mentioned functions.

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

Fix "out of memory" handling in _dbus_get_install_root_as_hash()

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

Fix "out of memory" handling in _dbus_get_address_string()

To avoid partial strings, appending to the returned string
is only done if fully successful.

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

_dbus_get_install_root_as_hash() now expects an initialized DBusString instance as a parameter

Since the _dbus_get_address_string() function is affected by the change,
the code calling the mentioned function been adapted.

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

_dbus_get_address_string() now expects an initialized DBusString instance as a parameter

The mentioned function now expects an initialized DBusString instance as
parameter for the return of a string to have a clear responsibility
regarding initialization and usage and thus simplify the "out of memory"
management.

Since the _dbus_get_shm_name() and _dbus_get_mutex_name() functions are
affected by the change, the related documentation has been adapted.

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

Indentation correction of Windows specific utility functions

The corrections refer to the following commits, so that they are not mixed
up with blank changes.

Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>

Merge branch 'todo' into 'master'

doc: Don't include TODO list in API documentation

See merge request dbus/dbus!238

doc: Don't include TODO list in API documentation

This is sorted non-deterministically, which is undesired for reproducible
builds, and is not really part of the API in any case.

Signed-off-by: Simon McVittie <smcv@collabora.com>

Merge branch 'tap-skip' into 'master'

test: Avoid misleading diagnostic when some tests are skipped

Closes #363

See merge request dbus/dbus!241

test: Avoid misleading diagnostic when some tests are skipped

If we have 7 test-cases, of which we ran 3 and skipped 4, then it's
misleading to say "7 tests passed". Diagnose this as
"3 tests passed (4 skipped)" instead.

We can either skip tests by putting a specific test name on the test
command-line, or programmatically (for example the fd-passing test is
always skipped on Windows, because Windows cannot implement Unix
fd-passing).

Resolves: https://gitlab.freedesktop.org/dbus/dbus/-/issues/363
Signed-off-by: Simon McVittie <smcv@collabora.com>

Merge branch 'arrays-of-tests' into 'master'

test-utils: Don't assume all arrays of tests are zero-terminated

Closes #366

See merge request dbus/dbus!242

test-utils: Don't assume all arrays of tests are zero-terminated

Fixes: dbus!233
Resolves: dbus#366
Signed-off-by: Simon McVittie <smcv@collabora.com>

Merge branch 'sysdeps-publish' into 'master'

sysdeps: Clarify bus address publication

See merge request dbus/dbus!240

sysdeps: Document _dbus_daemon_unpublish_session_bus_address

Signed-off-by: Simon McVittie <smcv@collabora.com>

sysdeps: Make some functions Windows-specific

_dbus_daemon_publish_session_bus_address had an empty implementation
but was never actually called on Unix, and
_dbus_daemon_is_session_bus_address_published wasn't even implemented.
Move them to the Windows-specific header to indicate that interface
changes here would only affect other Windows-specific code.

Signed-off-by: Simon McVittie <smcv@collabora.com>

Merge branch 'offsetof' into 'master'

internals: Use Standard C offsetof macro if available

See merge request dbus/dbus!237

internals: Use Standard C offsetof macro if available

clang 13 fails to compile our current implementation with:

.../dbus/dbus-message.c:2070:3: error: variable length array folded to constant array as an extension [-Werror,-Wgnu-folding-constant]
  _DBUS_STATIC_ASSERT (_DBUS_ALIGNOF (DBusMessageRealIter) <=
  ^
.../dbus/dbus-internals.h:460:25: note: expanded from macro '_DBUS_STATIC_ASSERT'
  typedef struct { char _assertion[(expr) ? 1 : -1]; } \

This appears to be because the "traditional" definition of
offsetof(), which we're hard-coding here, does not qualify as a constant
expression under C rules due to its use of pointer casts.

Modern compilers like gcc and clang have a built-in implementation
of offsetof that *is* a constant expression.

Signed-off-by: Simon McVittie <smcv@collabora.com>

Update NEWS

Since we're heading for a 1.14.x branch (dbus#350), also draft the
summary of what has changed since 1.12.x.

Signed-off-by: Simon McVittie <smcv@collabora.com>

Merge branch 'disable-containers' into 'master'

Disable the Containers interface

See merge request dbus/dbus!236

Disable the Containers interface

We've had a request for a 1.14.x stable-branch, but the Containers
interface is only partially implemented, not yet described in the
D-Bus Specification, and not ready to be part of our API guarantees.

Signed-off-by: Simon McVittie <smcv@collabora.com>

Merge branch 'cmake-verbose-config' into 'master'

cmake: add option ENABLE_VERBOSE_CONFIG

See merge request dbus/dbus!230

cmake: add option ENABLE_VERBOSE_CONFIG

Checking the filenames of generated configuration files is now optionally
possible with this cmake option. They are no longer displayed by default
to avoid unnecessarily flooding the output.

Merge branch 'cmake-fix-warnings-flags' into 'master'

cmake: Make gcc builds use similar compiler warnings to autotools

Closes #356

See merge request dbus/dbus!227

cmake: add configure option 'ENABLE_WERROR' to keep in sync with autotools

cmake: Make gcc builds use similar compiler warnings to autotools

Fixes #356

cmake: use the string 'append' command to add compiler flags for convenience

The minimum version of cmake has been increased because the mentioned
string command has been added with cmake 3.4.

Merge branch 'fix-issue-362' into 'master'

Fix `test applications do not display invalid test selection`

Closes #362

See merge request dbus/dbus!233

In _dbus_test_main() check if a specified test name is available

Fixes #362

test/test-utils.c: Add command line option --list-tests to test applications

With this command line option a list of available tests is printed to
stdout.

test/test-utils.c: Add command line option --help to test applications

Merge branch 'fix-issue-360' into 'master'

Fix  test-bus related memory leaks on Windows

Closes #360

See merge request dbus/dbus!232

Avoid assertation in bus_activation_reload()

In the mentioned function a local DBusError instance is now used to
fulfill the requirement of dbus_error_has_name() that the parameter
'error' must not be null.

See #360

Fix memory leaks in Windows variant of _dbus_directory_open()

If the memory is not sufficient, the created DBusString instance must be
released. This belongs to the mentioned function and
_dbus_string_init_from_string().

Fixes #360

Merge branch 'fix-test-dbus-daemon-windows' into 'master'

tools/ci-build.sh: Double maximum test time when running tests on Windows

Closes #320 and #359

See merge request dbus/dbus!231

tools/ci-build.sh: Double maximum test time when running tests on Windows

The currently timeout of 60 seconds is too short for test-dbus-daemon on
Windows, depending on the load of the CI system.

Fixes #359

Merge branch 'fix-issue-357' into 'master'

test-spawn-oom: Fix regression with recent Windows refactoring

Closes #357 and #279

See merge request dbus/dbus!228

Add returning oom error in _dbus_spawn_program() when building for WINCE

Adjust _dbus_spawn_async_with_babysitter() on Windows to follow Unix like OOM behavior

That way, the spawn-oom unit test could expect OOM on Windows to behave
the same as OOM on Unix, raising DBUS_ERROR_NO_MEMORY.

_dbus_decrement_fail_alloc_counter(): Enable out of memory testing on Windows

Fix memory leak in protect_argv() in case of memory shortage

protect_argv(): Only set return variable in success case

This avoids overwriting the variable even if there is no result.

Merge branch 'windows-error-oom' into 'master'

_dbus_win_set_error_from_last_error: Always set the error

See merge request dbus/dbus!229

_dbus_win_set_error_from_last_error: Always set the error

If we run out of memory while setting an error, we need to recover by
setting the error to "out of memory" instead of the original error.
Otherwise, the error indicator would be unset, breaking the rules of
our error model.

Signed-off-by: Simon McVittie <smcv@collabora.com>

Fix gcc compile error: redundant redeclaration of ‘environ’ [-Werror=redundant-decls]

Cherry picked from dbus/dbus!227

Merge branch 'issue-355' into 'master'

Fix MinGW build error

Closes #355

See merge request dbus/dbus!226

Fix MinGW build error: cast between incompatible function types from 'FARPROC' [-Werror=cast-function-type]'

The build error occurred in 'dbus/dbus-sysdeps-win.c:129:43:

Fixes #355

tools/ci-*.sh: Show used command line for CI build scripts

This makes it much easier to determine the command line used for the
purpose of reproducing the build.

Merge branch 'dbus-run-session-refactor' into 'master'

_dbus_spawn_program(): Add additional parameter to return error

See merge request dbus/dbus!223

dbus-spawn-win.c: Refactoring compose_string() and related functions to use DBusString

Since the code to compose strings is already used multiple times the
refactoring results in a new function _dbus_string_append_strings().

_dbus_spawn_program(): Add additional parameter to return error

By specifying an error instance via the additional parameter, errors that
occur in it are transported to the caller in a coordinated manner.

This eliminates the need to query GetLastError() outside the function,
which can return an incorrect value if the implementation changes.

_dbus_spawn_program() now returns #NULL for unification in case of error

Add _DBUS_GNUC_PRINTF annotation to _dbus_win_set_error_from_last_error()

This makes printf like format string related issues visible.

Remove `build:` prefix from jobs to make labels more visible in pipeline page

The length if the displayed jobs is limited on the gitlab CI pipeline page
and is occupied partly be the prefix.

Cherry-picked from dbus/dbus/!192

Merge branch 'master' into 'master'

autotools: use pkg-config instead of AC_PATH_XTRA

See merge request dbus/dbus!212

autotools: use pkg-config instead of AC_PATH_XTRA

Merge branch 'issue-354' into 'master'

dbus-run-session: also catches a server-exit under Windows

Closes #354

See merge request dbus/dbus!225

dbus-run-session: also catches a server-exit under Windows

If the dbus-daemon fails before the --ready-event-handle is signalled,
then dbus-run-session now detects that, and exit unsuccessfully.

Fixes #354

Fix printf format string in _dbus_win_event_create_inheritable()

Fixup for commit 3f7c36f4.

Merge branch 'clarify-doc-win-code' into 'master'

Clarify NULL vs. INVALID_HANDLE_VALUE in _dbus_win_event_free()

Closes #352

See merge request dbus/dbus!224

Clarify NULL vs. INVALID_HANDLE_VALUE in _dbus_win_event_free()

Fixes dbus/dbus#352

Merge branch 'dbus-run-session-add-delay' into 'master'

tools/dbus-run-session: fix race between manual and automatically started dbus-daemon on Windows

Closes #297

See merge request dbus/dbus!195

tools/dbus-run-session.c: use _dbus_win_set_error_from_last_error() on remaining locations

The previously used function _dbus_win_stderr_win_error() has been removed
because it is no longer used.

tools/dbus-run-session: fix race between manual and automatically started dbus-daemon on Windows

dbus-run-session starts a dbus-daemon before the client application.
We must avoid letting the application try to connect before the
dbus-daemon's DBusServer is listening for connections.

In the Unix implementation, we already achieved this via the
--print-address option. If the client tried to connect too soon,
the server would not yet be listening and the client would fail.

In the Windows implementation, we communicate the bus address to
the client application as an autolaunch: address, so if the client
tried to connect too soon, it would autolaunch a new dbus-daemon
instead of using the one that it was intended to use.

We can avoid this by using a new option to pass in a Windows event
object, which will be set when the server has started and is ready
to process connections.

Fixes #297

dbus-sysdeps-win.c: Add helper functions to handle events on Windows

These functions are in the dbus library to be used everywhere

Merge branch 'remove-nonexistent-match' into 'master'

bus: Don't return success from RemoveMatch if there was no such match-rule

See merge request dbus/dbus!222

test: Exercise successful and failed RemoveMatch calls

Reproduces: https://gitlab.freedesktop.org/dbus/dbus/-/issues/351
Signed-off-by: Simon McVittie <smcv@collabora.com>