Prepare release v1.13.10 and Specification v0.35

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

Update NEWS

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

Merge branch 'unix-gids' into 'master'

Add UnixGroupIDs to connection credentials

See merge request dbus/dbus!105

Fixes: dbus/dbus#196

Update NEWS

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

Merge branch 'cmake-mkdir' into 'master'

cmake: Create all output directories for Doxygen

Closes #266

See merge request dbus/dbus!113

cmake: Create all output directories for Doxygen

CI builds intermittently fail with

    error: Could not create output directory /.../doc/api/xml

or

    error: Could not create output directory /.../doc/api/man

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

Merge branch 'rlimit-nofile' into 'master'

bus: Try to raise soft fd limit to match hard limit

See merge request dbus/dbus!103

Reviewed-by: @poettering
[smcv: Correct a comment]
Reviewed-by: @pwithnall

Merge branch 'msys2-chmod' into 'master'

build: Don't assume we can set permissions on a directory

Closes #216

See merge request dbus/dbus!110

bus: Try to raise soft fd limit to match hard limit

Linux systems have traditionally set the soft limit to 1024 and the hard
limit to 4096. Recent versions of systemd keep the soft fd limit at
1024 to avoid breaking programs that still use select(), but raise the
hard limit to 512*1024, while in recent Debian versions a complicated
interaction between components gives a soft limit of 1024 and a hard
limit of 1024*1024. If we can, we might as well elevate our soft limit
to match the hard limit, minimizing the chance that we will run out of
file descriptor slots.

Unlike the previous code to raise the hard and soft limits to at least
65536, we do this even if we don't have privileges: privileges are
unnecessary to raise the soft limit up to the hard limit.

If we *do* have privileges, we also continue to raise the hard and soft
limits to at least 65536 if they weren't already that high, making
it harder to carry out a denial of service attack on the system bus on
systems that use the traditional limit (CVE-2014-7824).

As was previously the case on the system bus, we'll drop the limits back
to our initial limits before we execute a subprocess for traditional
(non-systemd) activation, if enabled.

systemd activation doesn't involve us starting subprocesses at all,
so in both cases activated services will still inherit the same limits
they did previously.

Reviewed-by: Lennart Poettering <lennart@poettering.net>
[smcv: Correct a comment based on Lennart's review, reword commit message]
Signed-off-by: Simon McVittie <smcv@collabora.com>

build: Don't assume we can set permissions on a directory

MSYS2 has enough of a Unixish environment to run Autotools, but
apparently not enough of a Unixish environment to have functional
permissions.

Closes: dbus#216

Merge branch 'disable-traditional-activation' into 'master'

Configure option to disable traditional activation

See merge request dbus/dbus!107

bus_driver_credentials_fill_unix_gids: Comment on a non-error

As described in the spec, we want GetConnectionCredentials() to report
less information, successfully, if it sees a group ID that it can't
represent.

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

spec: Document what happens for unrepresentable group IDs

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

test-dbus-daemon: Fix a misleading variable name

In D-Bus I'd expect "ay_iter" to be an iterator over the type 'ay',
i.e. a byte-array. Abbreviate a little less to avoid this.

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

test-dbus-daemon: Say why we don't check the value of LinuxSecurityLabel

Suggested on !105 by Matthijs van Duin.

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

Add UnixGroupIDs to connection credentials

Signed-off-by: Matthijs van Duin <matthijsvanduin@gmail.com>

Add _dbus_asv_add_fixed_array

This generalizes _dbus_asv_add_byte_array.

Signed-off-by: Matthijs van Duin <matthijsvanduin@gmail.com>

Configure option to disable traditional activation

Traditional activation could be disabled if all services use
SystemdService activation instead. Provide an example of a hardened
DBus systemd service drop-in file for such a setup.

Signed-off-by: Topi Miettinen <toiwoton@gmail.com>

Merge branch 'split-spawn-oom-test' into 'master'

Split spawn oom test

See merge request dbus/dbus!99

test-spawn-oom: Separate single spawn tests from oom related

This makes it possible to run each test independent from oom condition.

Merge branch 'msys2-fixes' into 'master'

cmake: Drop unused iconv dependency

Closes #262

See merge request dbus/dbus!101

Merge branch 'cal-fix-cmake-pkgconfig' into 'master'

cmake: Avoid overwriting PKG_CONFIG_PATH env var

See merge request dbus/dbus!96

Reviewed-by: @rhabacker

cmake: Avoid overwriting PKG_CONFIG_PATH env var

The CMake config file installed by DBus will run in the context of other
projects. Consequently, changing the value of the PKG_CONFIG_DIR,
PKG_CONFIG_PATH or PKG_CONFIG_LIBDIR environment variables will affect
any further calls to pkg-config made by such projects, which can cause
problems.

A common case of this happening are pkg-config files installed in
usr/share/pkgconfig for .pc files that are architecture-independent, as
for example systemd does.

Avoid clobbering the environment variables by saving and restoring their
values. Note that for some of the variables, setting them to an empty
string is different from not setting them at all.

Signed-off-by: Clemens Lang <clemens.lang@bmw-carit.de>

Merge branch 'patch-1' into 'master'

Update dbus-specification.xml to remove redundancies in the interface name requirements.

See merge request dbus/dbus!102

Reviewed-by: @rhabacker

Update dbus-specification.xml to remove redundancies in the interface name requirements.

Merge branch 'fix-cmake-doc-deps' into 'master'

Fixes rebuilding of dbus documentation at every make call

See merge request dbus/dbus!94

Try to fix doxygen error on CI reporting "Output directory ... does not exist and cannot be created"

Create top level dir for generating doxygon files

Fix dependency chain for doc files generated from docbook source

Update doxygen doc only if an associated source or Doxyfile has been changed

Use doxygen.stamp as output for generating doxygen doc similar to
autotools.

Target devhelp2 does not require dependencies that have already been specified in dbus.devhelp2

Cleanup dbus.devhelp2 make target pathes

Use absolute paths to avoid confusion about where
the associated file is located.

Add target devhelp2 only when xsltproc has been found

It doesn't make sense to add a make target if it can't be created.

Make the output directory used by doxygen absolute to avoid dependency on a particular working directory

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

CMake install fixes

See merge request dbus/dbus!98

Do not let doxygen build man pages on Windows

This fixes an issue on gitlab CI not been able to create man page
output dir. Also man pages does not make sense on Windows.

cmake: generate and install doc for dbus-run-session

cmake: install missing files to keep in sync with autotools builds

cmake: fix dtd installation dir

cmake: Enable pkg-config generating also on Windows

pkgconfig support is used by client packages on cross compiled
obs builds.

Merge branch 'FGasper/dbus-2ormore'

Update dbus-specification.xml to avoid implying that an interface name may consist of only 1 element.

Merge branch 'readme-cmake-add-cross-compile-hints' into 'master'

README.cmake: Add cross compile hints

See merge request dbus/dbus!100

cmake: Drop unused iconv dependency

Closes dbus#262

README.cmake: Fix doc for building on linux

Remove -G flag and let user use the default generator.

README.cmake: Add cross building for Windows

Merge branch 'nss-fds' into 'master'

_dbus_test_main: Exercise NSS modules before running tests

Closes #256

See merge request dbus/dbus!93

Reviewed-by: pwithnall
Tested-by: rhabacker

Merge branch 'unused-userdb' into 'master'

Remove unused bits of userdb

See merge request dbus/dbus!92

Reviewed-by: pwithnall

_dbus_test_main: Exercise NSS modules before running tests

Some NSS modules like those for sssd and LDAP might allocate fds
on a one-per-process basis, for example a socket to talk to sssd.
Make sure those have already been allocated before we enter the code
under test, so that they don't show up as having been "leaked" by the
first module of code under test that happens to do a NSS lookup.

The call to _dbus_test_check_memleaks tears down libdbus' own memory
allocations, but not any hidden state in libc or NSS.

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

Merge branch 'finish-reformat-cmake-files' into 'master'

Finish formatting cmake files

See merge request dbus/dbus!91

Fixes remaining indentations that are not covered by the cmake formatting script in CMakeLists.txt files

Adjust indents of CMakeLists.txt files always to 4 spaces

tools/cmake-format: Add option --check-indents to check indentations independently from other options

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

Remove obsolete parameter in end... cmake keywords

Remove spaces between cmake command and opening bracket

Use lower case cmake keywords in CMakeLists.txt files

Replace tabs by 4 spaces in CMakeLists.txt files

Remove trailing spaces in CMakeLists.txt files

tools/cmake-format: Do not exclude CMakeLists.txt files

This fixes a regression of merge request !82 introduced
with the rebase to !84.

Merge branch 'reformat-cmake-files' into 'master'

Reformat cmake files

Closes #252

See merge request dbus/dbus!82

Merge branch 'code-coverage' into 'master'

Adapt to API change in AX_CODE_COVERAGE version 28

Closes #249

See merge request dbus/dbus!88

Reviewed-by: @pwithnall

Adjust indents of cmake build system files always to 4 spaces

Remove obsolete parameter in end... cmake keywords

Remove spaces between cmake command and opening bracket

Use lower case cmake keywords

Replace tabs by 4 spaces in cmake build system files

Remove trailing spaces in cmake related files

Add script for formatting cmake files

The prefered call order is

  tools/cmake-format --trailing-spaces
  # create git commit
  tools/cmake-format --tabs
  # create git commit
  tools/cmake-format --keyword-case
  # create git commit
  tools/cmake-format --keyword-spaces
  # create git commit
  tools/cmake-format --end-args
  # create git commit
  tools/cmake-format --indents
  # create git commit

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

Merge branch 'move-cmake-toplevel' into 'master'

Move cmake related build system to top level

See merge request dbus/dbus!84

Reviewed-by: @smcv

_dbus_user_database_get_groupname: Inline into its only caller

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

_dbus_user_database_get_gid: Remove unused function

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

_dbus_parse_uid: Remove unused function

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

dbus-userdb: Remove unused _dbus_homedir_from_username()

Everything that wants a home directory uses either
_dbus_homedir_from_uid() or _dbus_homedir_from_current_process().

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

dbus-userdb: Remove unimplemented declaration of _dbus_credentials_from_uid()

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

cmake: Set working directory to be a native path

If we're cross-compiling for Windows on Unix, CTest needs to chdir() to
the native Unix path.

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

Add cmake macros add_executable_version_info and add_library_version_info

This helps to reduce duplicated code.

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

Remove duplicated cmake variable definition

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

Install devhelp2 and api doc with cmake

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

Generate documentation html index file from doc/index.html.in for autotools and cmake

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

Move cmake related build system to top level

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

Consistently add CODE_COVERAGE_CPPFLAGS everywhere

We forgot this in a couple of places.

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

Consistently add CODE_COVERAGE_LIBS everywhere

We need to link the code coverage objects, directly or indirectly,
into every executable and every shared library. The rule I've followed
to make it clear that we do this, without too much repetition, is:
each executable, shared library or convenience library has
CODE_COVERAGE_LIBS in its LDADD or LIBADD, unless it is linked to a
convenience library in the same directory that has CODE_COVERAGE_LIBS
in *its* LIBADD.

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

Adapt to API change in AX_CODE_COVERAGE version 28

AX_CODE_COVERAGE recently changed the way it embedded its Makefile rules
in the output file: instead of using @CODE_COVERAGE_RULES@, users
are now meant to include aminclude_static.am.

The new AX_CODE_COVERAGE is only in the latest autoconf-archive release,
version 2019.01.06, which is inconveniently new, so bundle everything
we need for the moment.

This requires us to stop using the deprecated CODE_COVERAGE_LDFLAGS
(which we still used to support older versions of autoconf-archive)
and replace them with CODE_COVERAGE_LIBS.

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

Merge branch 'cmake-dist' into 'master'

CI: Do the CMake native debug build from an Autotools `make dist`

Closes #255

See merge request dbus/dbus!87

Merge branch 'ax-missing' into 'master'

configure.ac: Forbid AX_-prefixed patterns more selectively

Closes #249

See merge request dbus/dbus!86

Reviewed-by: @pwithnall

Merge branch 'test-bundling' into 'master'

Move some test code from dbus/ and bus/ into test/

Closes #223

See merge request dbus/dbus!1

Reviewed-by: @rhabacker
[smcv: Use ${TEST_DIR} as requested during review]

test: Move _dbus_list_test() here

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

test: Move _dbus_transport_unix_test() here

This requires exposing _dbus_connection_get_address() as an internal
symbol, but that seems worth it.

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

test: Move _dbus_server_test() here

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

test: Move _dbus_misc_test() here

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

test: Move _dbus_signature_test() here

This is all public API, so there's no need to embed it in the library.

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

dbus-signature: Move static assertion outside test code

This always needs to be true, even if we're not embedding test goo
in the library. These static assertions have no runtime overhead, only
compile-time, so it's OK to make it unconditional.

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

test: Move sysdeps tests here

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

test: Move string tests (most of dbus-string-util.c) here

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

test: Move marshalling tests into test/

dbus-marshal-recursive-util.c contains utility functions used by the
other two, as well as tests. The others are purely test code.

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

dbus-marshal-*-util: Remove all trailing whitespace

This is in preparation for moving the files: our commit hook would
object to the delete + add otherwise.

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

test: Move dbus-message-factory and most of dbus-message-util into test/

As with previous moves, I've tried to extract the possible copyright
holders from the git history.

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

dbus-message-factory: Remove all trailing whitespace

This is in preparation for moving the files: our commit hook would
object to the delete + add otherwise.

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

test: Move auth, credentials tests from libdbus-internal into test/

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

dbus-credentials-util: Remove all trailing whitespace

This is in preparation for moving the file: our commit hook would
object to the delete + add otherwise.

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