summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'wip/smcv/reinstate-windows-ci' into 'master'HEADmasterSimon McVittie2023-05-161-4/+0
|\ | | | | | | | | | | | | Revert "CI: Disable native Windows builds for now" Closes #456 See merge request dbus/dbus!406
| * Revert "CI: Disable native Windows builds for now"Simon McVittie2023-05-161-4/+0
|/ | | | | | | | | It seems they do work, it was just very slow to build the image. This reverts commit a3a14ea09eb3ade08cd6b4c6af57afd8b1c0b8bb. Resolves: https://gitlab.freedesktop.org/dbus/dbus/-/issues/456 Signed-off-by: Simon McVittie <smcv@collabora.com>
* Merge branch 'read_procfs' into 'master'Simon McVittie2023-05-155-4/+64
|\ | | | | | | | | Support /proc in _dbus_file_get_content See merge request dbus/dbus!401
| * Support /proc in _dbus_file_get_contentLuca Boccassi2023-05-155-4/+64
|/ | | | | | | | | procfs has special semantics: most files are 0 size, only one read can be done on a file, and they are not larger than 4MB. Enhance _dbus_file_get_content() so that we can read files from /proc with it. Signed-off-by: Luca Boccassi <bluca@debian.org>
* Merge branch 'use_func_macro' into 'master'Simon McVittie2023-05-153-7/+7
|\ | | | | | | | | do not use __FUNCTION__ directly See merge request dbus/dbus!404
| * dbus-spawn-win: use `_DBUS_FUNCTION_NAME` instead of `__FUNCTION__`Barnabás Pőcze2023-05-151-4/+4
| | | | | | | | | | | | | | | | dbus-internals.h already defines a macro which expands to the name of the current function based on C standard version, etc. So use that instead of hard-coding `__FUNCTION__`. Signed-off-by: Barnabás Pőcze <pobrn@protonmail.com>
| * dbus-sysdeps-win: do not log function name twiceBarnabás Pőcze2023-05-151-2/+2
| | | | | | | | | | | | | | `_dbus_verbose()` already logs the function name, do not log it again in the message. Signed-off-by: Barnabás Pőcze <pobrn@protonmail.com>
| * dbus-internals: use `_DBUS_FUNCTION_NAME` in `_dbus_verbose()`Barnabás Pőcze2023-05-151-1/+1
|/ | | | | | | | dbus-internals.h already defines a macro which expands to the name of the current function based on C standard version, etc. So use that instead of hard-coding `__FUNCTION__`. Signed-off-by: Barnabás Pőcze <pobrn@protonmail.com>
* Merge branch 'dbus_msg_iter_fix_memleak' into 'master'Simon McVittie2023-05-151-4/+8
|\ | | | | | | | | dbus_message_iter_get_signature: Fix two memory leaks See merge request dbus/dbus!403
| * dbus_message_iter_get_signature: Fix two memory leaks on OOMBarnabás Pőcze2023-05-121-4/+8
|/ | | | | | | | | | | | | | Previously, `retstr` would not be freed when `_dbus_string_append_len()` or `_dbus_string_steal_data()` failed. Fix those by: * jumping to `_dbus_string_free()` when `_dbus_string_append_len()` fails * ignoring the return value of `_dbus_string_free()`. The latter works because in case of failure, `ret` will be set to NULL by `_dbus_string_steal_data()`. Signed-off-by: Barnabás Pőcze <pobrn@protonmail.com>
* Merge branch 'wip/smcv/fix-ci' into 'master'Simon McVittie2023-05-124-14/+30
|\ | | | | | | | | Fix CI See merge request dbus/dbus!405
| * CI: Disable native Windows builds for nowSimon McVittie2023-05-121-0/+4
| | | | | | | | | | | | | | | | These are extremely slow (the image build is currently at 36 minutes and still running) which is standing in the way of us having functional CI at all. They can be re-enabled if someone will maintain them. Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Disable "opensuse mingw64 cmake debug" until #455 is fixedSimon McVittie2023-05-121-0/+2
| | | | | | | | | | | | Having some CI is better than having no CI. Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Avoid using a no-op download location that gives a 403 errorSimon McVittie2023-05-121-1/+1
| | | | | | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Make creation of user idempotentSimon McVittie2023-05-121-2/+4
| | | | | | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Disable OOM-testing code paths for Meson, matching Autotools and CMakeSimon McVittie2023-05-121-0/+4
| | | | | | | | | | | | | | | | Repeatedly re-running each test with different malloc() calls failing is really slow, and in particular this is making dbus:dbus / marshal-recursive time out on freedesktop.org CI. Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Install mingw64-cross-cmake in openSUSE imageSimon McVittie2023-05-122-1/+2
| | | | | | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Sort lists of openSUSE packages alphabeticallySimon McVittie2023-05-121-3/+3
| | | | | | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Update Windows runnersSimon McVittie2023-05-122-4/+4
| | | | | | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Remove an obsolete workaroundSimon McVittie2023-05-121-2/+0
| | | | | | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Update to current fdo CI templatesSimon McVittie2023-05-121-1/+1
| | | | | | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Only run for pushes to dbusSimon McVittie2023-05-121-1/+1
| | | | | | | | | | | | | | | | In practice the pipeline is going to fail for namespaces other than dbus, so don't waste time on trying to run it there; only run the detached pipeline for the MR. Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Run a detached pipeline for merge requestsSimon McVittie2023-05-121-0/+5
|/ | | | | | | | After abuses of fdo infrastructure were mitigated in freedesktop/freedesktop#540, contributors cannot usually run pipelines in their own forks of dbus. Signed-off-by: Simon McVittie <smcv@collabora.com>
* Merge branch 'cmake_unistd' into 'master'Simon McVittie2023-03-211-0/+1
|\ | | | | | | | | cmake: add missing HAVE_SYS_SYSCALL_H define See merge request dbus/dbus!400
| * cmake: add missing HAVE_SYS_SYSCALL_H defineLuca Boccassi2023-03-201-0/+1
|/ | | | Signed-off-by: Luca Boccassi <bluca@debian.org>
* Merge branch 'fix-dbus-api-design-wrong-closing-tag' into 'master'Simon McVittie2023-03-021-2/+2
|\ | | | | | | | | doc/dbus-api-design: fix wrong closing tag See merge request dbus/dbus!396
| * doc/dbus-api-design: fix wrong closing tagYen-Chin, Lee2023-03-021-2/+2
|/ | | | | | | The original code of 'ProgressNotification' had a wrong closing tag, which should use '</signal>' instead of '</method>' Signed-off-by: Yen-Chin, Lee <coldnew.tw@gmail.com>
* Merge branch 'fix-issue-445' into 'master'Simon McVittie2023-02-211-0/+4
|\ | | | | | | | | | | | | CI: Make sure we always have a messagebus user, even if the dbus package isn't installed Closes #445 See merge request dbus/dbus!394
| * tools/ci-install.sh: Make sure we always have a messagebus user, even if the ↵Ralf Habacker2023-02-211-0/+4
|/ | | | | | | | | dbus package isn't installed This fixes a CI installation issue with the meson build system on openSUSE distribution. Fixes #445
* Start 1.15.6Simon McVittie2023-02-083-2/+7
| | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
* Update AUTHORSSimon McVittie2023-02-081-0/+10
| | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
* Prepare v1.15.4dbus-1.15.4Simon McVittie2023-02-084-20/+34
| | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
* NEWS: Add #421Simon McVittie2023-02-081-0/+9
| | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
* _dbus_string_skip_blank(): Let the final assert follow the previous searchRalf Habacker2023-02-081-1/+1
| | | | | | | | | | | | | | | | | | This appears to have been a copy/paste mistake. If only blanks (defined as spaces or tabs) were removed, then it cannot be right to check for white space (defined as spaces, tabs, carriage return or linefeed) afterwards. If libdbus was compiled with assertions enabled, then this is a denial-of-service issue for dbus-daemon or other users of DBusServer: an unauthenticated user with access to the server's socket can send whitespace that triggers this assertion failure. We recommend that production versions of dbus, for example in OS distributions, should be compiled with checks but without assertions. [smcv: expanded commit message] Thanks: Evgeny Vereshchagin Resolves: https://gitlab.freedesktop.org/dbus/dbus/-/issues/421
* Add test for _dbus_string_skip_blank()Ralf Habacker2023-02-084-0/+54
| | | | | | [smcv: Fix a memory leak] Reproduces: https://gitlab.freedesktop.org/dbus/dbus/-/issues/421
* Update NEWS for 1.15.xSimon McVittie2023-02-081-2/+49
| | | | Signed-off-by: Simon McVittie <smcv@collabora.com>
* Merge branch 'add-ci-build-libdbus-subproject' into 'master'Simon McVittie2023-02-076-0/+81
|\ | | | | | | | | gitlab-ci: build libdbus as subproject See merge request dbus/dbus!388
| * gitlab-ci: build libdbus as subprojectDaniel Wagner2023-02-076-0/+81
|/ | | | | | | Test it's possible to consume libdbus as a subproject. Suggested-by: Simon McVittie <smcv@collabora.com> Signed-off-by: Daniel Wagner <dwagner@suse.de>
* build: Use list for dbus_static_flagsDaniel Wagner2023-02-061-1/+1
| | | | | | | | Meson complains that it is only allowed to concatenate list to lists and not strings. Signed-off-by: Daniel Wagner <dwagner@suse.de> Origin: https://gitlab.freedesktop.org/dbus/dbus/-/merge_requests/388
* Merge branch 'add-test-header-fields' into 'master'Simon McVittie2023-02-062-1/+2
|\ | | | | | | | | cmake: add missing test for header-fields See merge request dbus/dbus!322
| * .gitlab-ci.yml: exclude non-functioning test 'header-fields' in job 'windows ↵Ralf Habacker2023-02-061-1/+1
| | | | | | | | | | | | vs15-64 cmake' Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>
| * cmake: add missing test for header-fieldsRalf Habacker2023-02-061-0/+1
|/ | | | Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>
* Merge branch 'gitlab-ci' into 'master'Simon McVittie2023-02-063-59/+30
|\ | | | | | | | | | | | | CI: Avoid changing uid, except when running installed-tests Closes #447 See merge request dbus/dbus!392
| * CI: Re-run some tests as root or as non-root, as appropriateSimon McVittie2023-02-061-8/+23
| | | | | | | | | | | | | | | | | | On Gitlab-CI we're always running the overall script as root (and therefore we'll only enter the code path to re-run as non-root), but when using these scripts for manual testing they might be run as non-root to begin with. Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Remove vestigial support for re-running tests in a Docker containerSimon McVittie2023-02-062-44/+3
| | | | | | | | | | | | | | | | Travis CI needed this, but Gitlab-CI always runs our tests in a Docker container of our choice, so there's never any need to enter another (and it's not allowed anyway). Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Re-clone the git repository every timeSimon McVittie2023-02-061-0/+3
| | | | | | | | | | | | | | This cleans up checkouts that were subjected to `chown -R` prior to this. Resolves: dbus/dbus#447 Signed-off-by: Simon McVittie <smcv@collabora.com>
| * CI: Don't change ownership of source directorySimon McVittie2023-02-062-7/+1
|/ | | | | | | | | | | | These CI scripts were originally used on Travis-CI, which starts all builds as an ordinary user that has the ability to become root via `sudo`. On Gitlab-CI, we don't need that: we start as uid 0, and can do the whole CI run like that. This also means we get somewhat better test coverage, because some of our unit tests benefit from being run as uid 0. The only test coverage we lose by being uid 0 is that test_pending_fd_timeout() in test/dbus-daemon.c is skipped, because uid 0 bypasses the limit that's under test there.
* Merge branch 'alignof' into 'master'Simon McVittie2023-01-161-0/+4
|\ | | | | | | | | Define _DBUS_ALIGNOF using _Alignof when using C11 or newer See merge request dbus/dbus!389
| * Define _DBUS_ALIGNOF using _Alignof when using C11 or newerKhem Raj2023-01-141-0/+4
|/ | | | | | | | | | | | | | | WG14 N2350 made very clear that it is an UB having type definitions within "offsetof" [1]. This patch changes the implementation of macro _DBUS_ALIGNOF to builtin "_Alignof" to avoid undefined behavior. clang 16+ has started to diagnose this [2] Fixes build when using -std >= gnu11 and using clang16+ [1] https://www.open-std.org/jtc1/sc22/wg14/www/docs/n2350.htm [2] https://reviews.llvm.org/D133574 Signed-off-by: Khem Raj <raj.khem@gmail.com>
* Merge branch 'jepio/fix-log-callback-segfault' into 'master'Simon McVittie2023-01-101-3/+2
|\ | | | | | | | | bus/selinux: Move vsnprintf call to avoid va_list reuse See merge request dbus/dbus!386
View Lorry Controller Status