bus: Show the errno if inotify cannot be initialized

This will hopefully help to indicate why.

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

bus: Correct check for inotify_init() failure

fd 0 is a valid fd - although if we get stdin as our inotify fd, something
is weird somewhere. For the dbus-daemon, in practice this should never
happen, because we use _dbus_ensure_standard_fds() to make sure stdin
is already open. For unit tests, it could in theory be the case that
stdin is closed.

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

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>

bus: Separate RemoveMatch into prepare and commit stages

This means we don't send a spurious successful reply if a caller removes
a match rule that they never added.

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

Merge branch 'cleanup-autolaunch-win' into 'master'

dbus-sysdeps-win.c: corrections of indentations and minor refactoring

See merge request dbus/dbus!221

dbus/dbus-sysdeps-win.c: Convert the character buffer 'dbus_args' to a DBusString instance

This is necessary to avoid possible stack overflows.

dbus/dbus-sysdeps-*win.c: correct indentation when calling functions

The normal style is <function-name><space>(<no-spaces>...<no-spaces>).

dbus/dbus-sysdeps-win.c: correction of indentations in _dbus_get_autolaunch_address ()

Merge branch 'cmake-fix-include-path' into 'master'

cmake: In generated cmake support files get value for DBus1_INCLUDE_DIRS from related cmake target

Closes #346

See merge request dbus/dbus!214

cmake: In generated cmake support files get value for DBus1_INCLUDE_DIRS variable from related cmake target

This allows cmake to construct the resulting (relocatable) runtime paths.

Fixes dbus/dbus#346

Merge branch 'master' into 'master'

CMake: Set IMPORTED_IMPLIB property

See merge request dbus/dbus!172

CMake: Set IMPORTED_IMPLIB property

Setting this property allows to fix linking to the imported target with MinGW.
This only happens when dbus is built using autotools, when cmake is used the DBus1Config.variant.in
is configured and the automatically exported target by cmake is fine.

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

cmake: various variable corrections defined in the generated config.h

See merge request dbus/dbus!217

cmake: Separate setting compiler warnings between C and CXX

This is necessary because different warnings are restricted to one
compiler variant.

Add HAVE_DECL_xxx checks to cmake build system

The first two definitions are required to fix cmake build error when
compiling with -Werror=undef on Windows.

The last one completes having HAVE_DECL_xxx definitions.

Move _GNU_SOURCE to config.h.cmake

Move DBUS_BUILT_R_DYNAMIC to config.h.cmake

Fix definition of HAVE_SOCKLEN_T

Add configure checks for vsnprintf and vasprintf to cmake build system

cmake: take over some fatal warnings from autotools

Adopt various fatal warnings from the autotools build system
to see corresponding build errors also in cmake.

Merge branch 'davidre-master-patch-89698' into 'master'

Put dbus-daemon into session slice

See merge request dbus/dbus!219

Put dbus-daemon into session slice

The session slice and the  app and background slices are special slices defined by
https://systemd.io/DESKTOP_ENVIRONMENTS/, where:
  session.slice: Contains only processes essential to run the user’s graphical session
  app.slice: Contains all normal applications that the user is running
This allows users or sysadmins to control resource allocation depending on the type
of the service.
Since v249 (https://github.com/systemd/systemd/commit/23dce98e89616092007005692a4574ab908db5a6)
systemd puts user services into the app slice by default so dbus needs to manually state
that it belongs in the session slice.

Merge branch 'fix-319' into 'master'

cmake: fix find_package related naming mismatch for GLIB2

Closes #319

See merge request dbus/dbus!216

cmake: fix find_package related naming mismatch for GLIB2

The package name passed to `find_package_handle_standard_args` (GLIB2) did not match the name of the calling package (GLib2).
This could lead to problems when calling code that expects `find_package`.
result variables (e.g. `_FOUND`) expect to follow a certain pattern.

fixes #319

Merge branch 'fix-spec' into 'master'

spec: Clean up use of the term for an array of dict entries

Closes #347

See merge request dbus/dbus!215

Clean up use of the term for an array of dict entries

fixes #347

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

Start spec 0.37 development

Merge branch 'get-machine-id' into 'master'

Specify what ID GetMachineId actually returns

See merge request dbus/dbus!198

Describe where machine ID comes from

Merge branch 'test-memleak' into 'master'

test/thread-blocking.c: Fix a memory leak

See merge request dbus/dbus!208

test/thread-blocking.c: Fix a memory leak

Free name inside the for loop. Found by Coverity.

Bug-RedHat: https://bugzilla.redhat.com/show_bug.cgi?id=1938701

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

CMake-format fixes

See merge request dbus/dbus!213

Apply cmake formatting rules

tools/cmake-format: fix sed warning `Expression #1, char 25: unknown option for »s«` in --all mode

tools/cmake-format: add support to use custom source root directory

Merge branch 'dbus-send-validate' into 'master'

dbus-send: Do more syntax validation

Closes #338

See merge request dbus/dbus!211

dbus-send: Validate interface, member names before use

Instead of a failed check or assertion failure and a core dump, let's
produce an error message on stderr and a graceful nonzero exit status.
It's still not going to *work*, but at least we can avoid crashing.

    $ dbus-send / com.example.Nope..Nope
    Interface name was not valid: 'com.example.Nope.'
    $ dbus-send / com.example.Nope.0
    Invalid signal name: Member name was not valid: '0'

Resolves: dbus#338
Signed-off-by: Simon McVittie <smcv@collabora.com>

dbus-send: Print an error message if object path is syntactically invalid

    $ dbus-send // nope
    Object path was not valid: '//'

Related to dbus#338.

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

Merge branch 'spec-correction' into 'master'

Add a few clarifications to the spec

See merge request dbus/dbus!203

spec: Clarify Array element needs to be padded even if inexistent

spec: Clarify that Variant's value needs padding

Merge branch 'ci' into 'master'

ci: bump msys2 deps, fix broken links

See merge request dbus/dbus!201

ci: bump msys2 deps, fix broken links

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

Reference CVE-2020-35512 in NEWS

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

Merge branch 'cmake-add-unit-test-macro' into 'master'

cmake: add macro add_unit_test to reduce code duplication

See merge request dbus/dbus!196

cmake: add macro add_unit_test to reduce code duplication

This macro is now used by add_test_executable and
add_session_test_executable.

Merge branch 'ci-dependency-fixes' into 'master'

Cleanup package installation on CI

See merge request dbus/dbus!187

tools/ci-install.sh: Use package=() style for local package list to avoid '\' notation

On CI use common install prefix named 'dep_prefix' for installing local packages

With this commit a new variable 'ci_local_packages' has been introduced
to have a choice for using development packages from a local installation
or from the distribution.

tools/ci-build.sh: Clean up directories from possible previous builds

Otherwise, ci-build.sh cannot be executed on a local system if there
are previous runs.

tools/ci-install.sh: merge multiple calls to apt-get

This required a reorganization of the steps that had been carried out.
The new order is:
   1. install packages with apt-get
   2. create user for build if required
   3. fetch and unpack tar balls
   4. create messagebus user

Move installing packages into tools/ci-install.sh

In tools/ci-build.sh the cross compile setup has to be moved further up
to match the correct subdirectory.

Merge branch 'misc-windows-fixes' into 'master'

sysdeps-win: Expand/fix verbose logging

See merge request dbus/dbus!193

_dbus_poll_select (): fix concating multiple verbose lines

Add verbose info to publishing session bus related functions

This is useful for tracking auto launch support in dbus-daemon.

Merge branch 'update-selinux-auditing' into 'master'

bus/selinux: Fix audit message types.

See merge request dbus/dbus!173

bus/selinux: Fix audit message types.

The SELinux log callback includes a message type. Not all messages are
auditable and those that are have varying audit types. An audit message is
a security-relevant event: security state changes, MAC permission denied,
etc.  A message that is auditable is not necessarily sensitive.  Messages
that are not auditable are not security-relevant, like messages about
socket polling errors.  Update the auditing accordingly.

If the message is not auditable, fall through and write it to syslog.

Signed-off-by: Chris PeBenito <chpebeni@linux.microsoft.com>

Merge branch 'travis-ci-cleanup' into 'master'

Drop unused travis CI configuration files

See merge request dbus/dbus!197

Drop unused travis CI configuration files

Merge branch 'bug/NEWS-machine-id-paths' into 'master'

NEWS: Add missing directory 'lib' to three paths

See merge request dbus/dbus!182

NEWS: Add missing directory 'lib' to three paths

Since ${localstatedir}/dbus should be ${localstatedir}/lib/dbus,
insert the missing directory 'lib' into the three erroneous paths.