| Commit message (Collapse) | Author | Age | Files | Lines |
|\
| |
| |
| |
| |
| |
| | |
Revert "CI: Disable native Windows builds for now"
Closes #456
See merge request dbus/dbus!406
|
|/
|
|
|
|
|
|
|
| |
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>
|
|\
| |
| |
| |
| | |
Support /proc in _dbus_file_get_content
See merge request dbus/dbus!401
|
|/
|
|
|
|
|
|
|
| |
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>
|
|\
| |
| |
| |
| | |
do not use __FUNCTION__ directly
See merge request dbus/dbus!404
|
| |
| |
| |
| |
| |
| |
| |
| | |
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_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.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_message_iter_get_signature: Fix two memory leaks
See merge request dbus/dbus!403
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|\
| |
| |
| |
| | |
Fix CI
See merge request dbus/dbus!405
|
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| | |
Having some CI is better than having no CI.
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
| |
| |
| |
| | |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
| |
| |
| |
| | |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| | |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
| |
| |
| |
| | |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
| |
| |
| |
| | |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
| |
| |
| |
| | |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
| |
| |
| |
| | |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
|/
|
|
|
|
|
|
| |
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>
|
|\
| |
| |
| |
| | |
cmake: add missing HAVE_SYS_SYSCALL_H define
See merge request dbus/dbus!400
|
|/
|
|
| |
Signed-off-by: Luca Boccassi <bluca@debian.org>
|
|\
| |
| |
| |
| | |
doc/dbus-api-design: fix wrong closing tag
See merge request dbus/dbus!396
|
|/
|
|
|
|
|
| |
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>
|
|\
| |
| |
| |
| |
| |
| | |
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
|
|/
|
|
|
|
|
|
|
| |
dbus package isn't installed
This fixes a CI installation issue with the meson build system on openSUSE
distribution.
Fixes #445
|
|
|
|
| |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
|
|
|
| |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
|
|
|
| |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
|
|
|
| |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
| |
[smcv: Fix a memory leak]
Reproduces: https://gitlab.freedesktop.org/dbus/dbus/-/issues/421
|
|
|
|
| |
Signed-off-by: Simon McVittie <smcv@collabora.com>
|
|\
| |
| |
| |
| | |
gitlab-ci: build libdbus as subproject
See merge request dbus/dbus!388
|
|/
|
|
|
|
|
| |
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>
|
|
|
|
|
|
|
|
| |
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
|
|\
| |
| |
| |
| | |
cmake: add missing test for header-fields
See merge request dbus/dbus!322
|
| |
| |
| |
| |
| |
| | |
vs15-64 cmake'
Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>
|
|/
|
|
| |
Signed-off-by: Ralf Habacker <ralf.habacker@freenet.de>
|
|\
| |
| |
| |
| |
| |
| | |
CI: Avoid changing uid, except when running installed-tests
Closes #447
See merge request dbus/dbus!392
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| | |
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>
|
|/
|
|
|
|
|
|
|
|
|
|
| |
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.
|
|\
| |
| |
| |
| | |
Define _DBUS_ALIGNOF using _Alignof when using C11 or newer
See merge request dbus/dbus!389
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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>
|
|\
| |
| |
| |
| | |
bus/selinux: Move vsnprintf call to avoid va_list reuse
See merge request dbus/dbus!386
|