activation-helper: Never crash if unable to write oom_score_adj

_dbus_warn() normally only logs a warning, but can be made fatal by
environment variables. In particular, we do that during unit testing,
which can result in a build-time test failure if dbus is built in a
sandbox environment that prevents write access.

_dbus_log() does only the logging part of _dbus_warn(), which seems
more appropriate here.

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

Merge branch 'win32-uds' into 'master'

Add Unix socket support on Windows

See merge request dbus/dbus!249

spec: AF_UNIX now available on Windows as well

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

spec: try to improve comment about AF_UNIX path

The comment refers to the nul-padding of sockaddr_un member sun_path and
using an addrlen of sizeof(sockaddr_un). There is not much need to
document an old now "broken" behaviour.

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

dbus: extract out _dbus_server_new_for_dir()

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

gitlab: skip failing runtime directory check, add FIXME

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

dbus/win: use SIO_AF_UNIX_GETPEERPID to lookup peer PID

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

test: enable AF_UNIX server-oom test on !unix

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

test/server-oom: do not hardcode /tmp

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

test: enable AF_UNIX relay tests on !unix

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

test/relay: do not hardcode /tmp on !unix

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

test: enable AF_UNIX loopback tests on !unix

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

tests/loopback: do not hardcode /tmp for !unix

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

test: fix loopback AF_UNIX tests to work on Windows

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

test: enable AF_UNIX corrupt test on !unix

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

tests/corrupt: do not hardcode /tmp on !unix

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

test: add test_check_af_unix_works()

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

dbus: add a few directory separator macros

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

dbus: move DBUS_IS_DIR_SEPARATOR to dbus-internals.h

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

dbus: move AF_UNIX code to server-socket

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

dbus: handle unix server in a new function

Split _dbus_server_listen_platform_specific() to handle unix listenable
address independently, allowing Windows support in following commit.

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

dbus: move AF_UNIX code to transport-socket

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

dbus: add _dbus_transport_open_unixexec()

Split _dbus_transport_open_unix_socket() to leave the "unixexec:"
handling to the unix-specific unit.

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

dbus: handle unix transport in a new common function

Split out the Unix socket handling from open_platform_specific(),
enabling "unix:" connectable addresses on Windows in next patch.

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

dbus: add function for Unix sockets on Windows

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

dbus: move _DBUS_MAX_SUN_PATH_LENGTH to sysdeps.h

Share the same constant for all systems.

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

build-sys: check for afunix.h

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

dbus: move unix socket declarations to common sysdeps.h

The function declaration can be moved to system-agnostic header and be
implemented on Windows in following commits.

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

dbus: change unix socket functions to return DBusSocket

This will allow to abstract away for Windows Unix socket support.

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

dbus: simplify socket close() & invalidate() calls

Now that _dbus_close_socket() invalidates the socket on its own already.

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

dbus: set the socket as invalid in _dbus_close_socket()

This can simplify error handling in many situation where a socket is
returned, such as in the following commits.

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

dbus: make _dbus_close_socket() take DBusSocket*

This will allow to invalidate the socket in the following commit.

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

dbus/win: fix a code comment about get_tmpdir()

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

s/PF_UNIX/AF_UNIX

PF_UNIX is an old BSD 4.x relic, and even there they promise that PF_UNIX
is the same as AF_UNIX. (Linux socket(2))

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

Misc style fix

As pointed out by Simon McVittie <smcv@collabora.com>:
https://gitlab.freedesktop.org/dbus/dbus/-/merge_requests/249#note_1323102

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

Merge branch 'leak-sanitizer' into 'master'

Silence some LeakSanitizer warnings

See merge request dbus/dbus!326

dbus-launch-x11: Ignore X11 connection when checking for memory leaks

The X11 connection is opened and never closed. Because dbus-launch
forks and continues to run non-trivial code in a forked child, it is
not clear whether (or where) it would be safe to close it; instead, we
leave it open until process exit, at which point the socket is cleaned
up by the kernel.

Any memory allocated for the X11 connection is only allocated once per
run of dbus-launch, so there's no need to keep track of it, and we can
silence these memory leak warnings as uninteresting.

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

test-privserver: Exit if DBusLoop cannot be allocated

Otherwise, we'd just crash when we dereference the null pointer.

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

test-privserver: Move cleanup of DBusLoop into test_service_data_free()

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

test-privserver: Don't leak a copy of the server address

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

Merge branch 'compiler-args' into 'master'

meson: Improve handling of compiler arguments

See merge request dbus/dbus!325

meson: Enable more compiler warnings

This provides parity with what we did in Autotools (most of it via
AX_COMPILER_FLAGS_CFLAGS).

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

meson: Pass project-wide compile arguments to has_header_symbol, etc.

Otherwise, we'll miss symbols that only appear when _GNU_SOURCE is
defined, like environ in Linux unistd.h.

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

meson: Comment why we disable certain compiler warnings

This knowledge shouldn't be lost when we delete the Autotools build
system.

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

meson: Build with -fno-common if supported

This turns off a poorly-supported feature of older compilers. Even when
using an older compiler, we want to behave more strictly.

The Autotools build already did this.

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

meson: Build with -fno-strict-aliasing if supported

We know that dbus is not fully strict-aliasing-compliant. In Autotools
we got this from AX_COMPILER_FLAGS_CFLAGS.

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

meson: Set up functionally necessary compiler arguments separately

We have to set up feature-flag options like _GNU_SOURCE before we
do compiler checks like cc.has_header_symbol, otherwise we'll miss
symbols that are guarded by a feature-flag test, like environ in Linux
unistd.h.

However, we don't want to pass flags for extra compiler warnings
when doing these checks, because that can cause false negatives:
in particular, Meson's check programs nearly always trigger -Wunused.
So the warnings need to be in a separate list.

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

test: Work around TAP test regression in Meson 0.63.0

With Meson 0.63.0, detailed output of TAP tests is not logged, and the
test deadlocks if the stderr pipe fills up. I'm hoping this will be fixed
before 0.63.1, but in the meantime we can work around it by falling back
to the 'exitcode' protocol: this means we lose machine-readable detailed
test results, but at least our tests pass.

See https://github.com/mesonbuild/meson/issues/10577 and
https://github.com/mesonbuild/meson/issues/10563 for details of the
Meson regression.

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

build: Distribute update-authors.sh in tarballs

Newer versions of Meson require the script for a run_target() to be
present at configure time, even if the script isn't run.

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

Update NEWS

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

Merge branch 'meson' into 'master'

Meson build system (fork of !135)

Closes #325

See merge request dbus/dbus!303

maintainer-upload-docs: Use paths pointing into the source directory

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

CI: take cmake config.h as artifacts

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

CI: Don't run tests for openSUSE Meson mingw builds for now

They don't find their library dependencies.

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

CI: Link libexpat statically when using Wine to run tests

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

CI: Skip tests that run dbus-daemon when running on Windows

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

CI: Explicitly build before running tests

If we don't do this, Meson will build everything that is a direct
dependency of the test executables, but it won't build indirect
dependencies like GLib's gspawn-win32-helper.exe.

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

CI: Ensure that subprojects (if used) are on the Wine PATH

If we are not using Meson subprojects, this will have no practical
effect, because the directories won't exist.

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

CI: Collect artifacts for Windows builds

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

CI: Force use of subprojects for Windows 32-bit build

The Windows image has expat and zlib installed, but they're 64-bit DLLs
which won't link successfully into our 32-bit build.

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

CI: Collect config.h

This will help to verify that the Meson and Autotools/CMake builds are
equivalent.

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

CI: Enable Ducktype-based documentation on Debian/Ubuntu

We don't currently know the equivalent of python3-mallard-ducktype on
other OSs like openSUSE, but we know this should work on Debian (it does
when we do Autotools builds).

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

CI: Run Meson with -v so we can see compiler command-lines

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

CI: Enable modular tests for all Meson-based CI builds

The whole point of these tests is that they're safe to enable because
they don't affect the production binaries.

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

CI: Factor out Wine setup so that it applies equally to Meson

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

CI: Install cross pkg-config on openSUSE

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

CI: Explicitly enable wrap dependencies

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

CI: Tell Meson build where to find the source directory

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

CI: Implement debug build variant for Meson

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

CI: Explicitly disable features that can't work on Windows

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

CI: Add glue to build with mingw-w64 toolchain on Debian

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

CI: Only use openSUSE mingw*-meson as a replacement for `meson setup`

These pass a lot of configure arguments taken from RPM macros, which
are only accepted by the meson command if we don't explicitly select
a mode.

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

CI: Verify that Autotools `make dist` includes everything needed for Meson

This is similar to what we have done for CMake since dbus/dbus!87
(commit 1063bba "CI: Do the CMake native debug build from an Autotools
`make dist`"). Our official source releases are Autotools `make dist`
tarballs, so our CI should assert that such tarballs contain everything
necessary to do a CMake or Meson build, so that downstream distributions
using our tarballs as source can choose their preferred build system.

When the Meson build system is ready to be recommended as more preferred
than Autotools, we can do as GLib did: stop releasing `make dist`
tarballs, and start releasing `meson dist` tarballs instead (which will
change nothing for Meson or CMake users, but Autotools users will have
to run autoreconf or autogen.sh before building).

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

CI: Consistently build in ./build

This will make it easier to capture log files. All our CI builds happen
in an expendable checkout, so we can safely remove and re-create ./build.

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

CI: Collect all Meson logs, not just the test log

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

CI: Add repository for word-size-independent mingw stuff

windows:mingw:win64 depends on packages from windows:mingw.

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

CI: Reinstate mingw*-glib2-devel

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

CI: Use mingw32-meson if that's what we're targeting

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

Add meson build CI

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

test: Don't prepend test- to the names of manual tests

This makes them consistent with what we do in Autotools.

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

meson: Create /var/lib/dbus

This holds the machine ID created by dbus-uuidgen.

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

meson: Don't use cc.has_function to check for va_copy

va_copy is not an ordinary function (an extern symbol), so we can't
treat it as one. Instead, use the same compilation/linking check as
in the Autotools build system.

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

cmake: drop DBUS_VERBOSE_C_S

Apparently, not used anywhere.

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

cmake: drop unused HAVE_VASPRINTF/HAVE_VSNPRINTF checks

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

cmake/autotools: remove HAVE_DIRFD/HAVE_DDFD

dirfd() is assumed to be present on Linux, dd_fd is never used.

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

cmake: drop needless headers checks

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

meson: add missing header checks

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

meson: add windows_output_debug_string option

Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
[smcv: Rename to avoid using "Win32" to refer to both 32- and 64-bit]
Signed-off-by: Simon McVittie <smcv@collabora.com>

autotools: dist listen-autolaunch-win.conf.in as well

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

meson: add test-autolaunch-win

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

test: Set PATH or WINEPATH to find dbus-daemon.exe in build-time tests

When building for Windows, either natively or cross-compiling and
running tests with Wine, test-autolaunch-win expects to be able to
find the just-built dbus-daemon.exe in the PATH.

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

meson: Allow disabling Valgrind instrumentation

This would currently result in a hard dependency on Valgrind headers
when linking dependent projects to libdbus (see
https://github.com/mesonbuild/meson/pull/10544 for an attempt to
avoid this), so distributions likely don't want to enable this feature
in their main builds of valgrind (although they could enable it in
special debug builds like the one in Debian's dbus-tests package if
desired).

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

meson: Fix the check for --version-script support

At the time we do this check, version_script has not yet been generated,
so we have to use something else.

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

meson: Add GLib version macros

This helps to avoid unintended dependencies on newer versions of GLib.

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

meson: Define HAVE_GIO_UNIX

Some of the tests check for this.

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

meson: Check for LOG_PERROR

We use this when writing to the system log.

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

meson: Check for environ declaration

We use this in various places.

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

meson: Replicate Autotools check for __sync_sub_and_fetch

The built-in atomic intrinsics are not exactly functions, so it's
incorrect to use has_function to check for them.

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

meson: Always define HAVE_DECL_MSG_NOSIGNAL to 1 or 0

This is consistent with Autotools AC_CHECK_DECLS.

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

meson: Don't check for dirfd

We never actually look at HAVE_DIRFD.

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