Your IP : 216.73.216.48
commit c4f335cbddf29f271896f491bbb7949ac0680ff5
Author: Simon McVittie <smcv@debian.org>
Date: 2016-01-14 08:38:50 +0000
manual test: move to test/core
The strange use of source from a sibling directory seems to be
breaking
distcheck:
Making distclean in manual
make[3]: Entering directory
'/.../dbus-glib-0.105/_build/sub/test/manual'
Makefile:409: ../core/.deps/my-object.Po: No such file or directory
make[3]: *** No rule to make target '../core/.deps/my-object.Po'.
Stop.
configure.ac | 1 -
test/Makefile.am | 4 +-
test/core/Makefile.am | 9 ++
test/core/manual/invalid-usage.c | 320
+++++++++++++++++++++++++++++++++++++++
test/manual/Makefile.am | 30 ----
test/manual/invalid-usage.c | 320
---------------------------------------
6 files changed, 331 insertions(+), 353 deletions(-)
commit f9b23a4d7f536b5d4c42c39d170bc561811cbe6f
Author: Simon McVittie <smcv@debian.org>
Date: 2016-01-14 08:34:36 +0000
test: stop generating marshallers
They haven't been needed since GLib 2.30. Until now, we were also
incorrectly distributing the generated marshallers in tarball
releases.
test/core/Makefile.am | 29 +++++++++++------------------
test/core/my-object.c | 19 ++-----------------
test/core/my-object.h | 1 -
test/core/test-dbus-glib.c | 2 --
test/manual/Makefile.am | 2 --
5 files changed, 13 insertions(+), 40 deletions(-)
commit 43a446bbd4e7c5726e87882b03185df37e97fe73
Author: Simon McVittie <smcv@debian.org>
Date: 2016-01-14 07:00:13 +0000
test: stop calling g_mem_profile(), no longer supported
This now raises a g_warning() which makes the test fail.
test/core/test-dbus-glib.c | 6 ------
1 file changed, 6 deletions(-)
commit 6211894d92d45271fa5e8c964521ca8ee2cef1ac
Author: Simon McVittie <smcv@debian.org>
Date: 2016-01-14 06:53:22 +0000
Enable subdir-objects Automake option to silence warnings
automake: warning: possible forward-incompatibility.
automake: At least a source file is in a subdirectory, but the
'subdir-objects'
automake: automake option hasn't been enabled. [etc.]
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
commit ebda5f95126d6adc38c2511a65e330d472af4f60
Author: Simon McVittie <smcv@debian.org>
Date: 2016-01-14 06:52:30 +0000
Replace deprecated INCLUDES with AM_CPPFLAGS
dbus/Makefile.am | 4 ++--
dbus/examples/Makefile.am | 2 +-
dbus/examples/statemachine/Makefile.am | 2 +-
test/Makefile.am | 2 +-
test/core/Makefile.am | 2 +-
test/interfaces/Makefile.am | 2 +-
tools/Makefile.am | 9 ++++++++-
7 files changed, 15 insertions(+), 8 deletions(-)
commit be7177fafb94bc3db50843e739df4575d4dabac0
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2016-01-04 14:08:01 +0000
NEWS: mention GLib 2.47.2, credit Colin
NEWS | 9 +++++----
1 file changed, 5 insertions(+), 4 deletions(-)
commit af8e9f140d50eaf46d89503d2e765699aa923252
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2016-01-04 14:05:30 +0000
NEWS
NEWS | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
commit 14d608cc0bd83ce5d5bdf469ec0e83a6afd97afd
Author: Colin Watson <cjwatson@debian.org>
Date: 2015-12-26 18:01:04 +0000
Stop using G_HAVE_INLINE
GLib removed this macro from its headers in
db2367e8782d7a39fc3e93d13f6a16f10cad04c2 (GLib 2.47.2). It's
unnecessary here for the same reasons, and relying on it now causes an
unused-function warning from GCC.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=93513
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
dbus/dbus-binding-tool-glib.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
commit 94c6ec840575a11c485ebb3a652a4bc89917dcd0
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2015-02-09 16:26:16 +0000
0.105
NEWS | 5 +++++
configure.ac | 2 +-
2 files changed, 6 insertions(+), 1 deletion(-)
commit 3000eaff772abadc1dcd22742d7d80545bd7269f
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2015-02-09 15:58:57 +0000
0.104
NEWS | 38 ++++++++++++++++++++++++++++++++++++--
configure.ac | 6 +++---
2 files changed, 39 insertions(+), 5 deletions(-)
commit 82ca3f62848b72061bf041df529bea58e53d55e7
Author: Philip Withnall <philip.withnall@collabora.co.uk>
Date: 2015-02-09 13:38:27 +0000
doc: Make the deprecation warning bigger
Make the deprecation warning in the documentation introduction bigger.
Add a copy of it to the front page of the documentation too, which is
what people see when they first find the documentation online.
doc/reference/dbus-glib-docs.sgml | 27 ++++++++++++++++++++-------
1 file changed, 20 insertions(+), 7 deletions(-)
commit 983ea5ae211dcac80007cc8ef2b3be0aad35ecb1
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2015-02-09 15:02:17 +0000
30574 test: squash some compiler warnings
test/core/30574.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
commit 63d86aad0624cb9c861cb8aae53467893b47a497
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2015-02-09 14:49:06 +0000
Drop unused functions
dbus/dbus-gobject.c | 55
-----------------------------------------------------
1 file changed, 55 deletions(-)
commit b972d42d074335d2043fd0c2b4300825a22783b1
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2015-02-09 14:48:59 +0000
Use GLIB_VERSION_MAX_ALLOWED, GLIB_VERSION_MIN_REQUIRED
configure.ac | 4 ++++
1 file changed, 4 insertions(+)
commit fa9065942e51cac4bf3927d7c38e350457ae085a
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2015-02-09 14:43:49 +0000
Stop calling g_thread_init or linking libgthread-2.0
Since GLib 2.32, which we now depend on, this is unnecessary.
configure.ac | 2 +-
test/core/peer-client.c | 2 +-
test/core/peer-server.c | 2 +-
test/core/test-profile.c | 1 -
test/core/test-service-glib.c | 2 +-
test/core/test-thread-client.c | 1 -
test/core/test-thread-server.c | 1 -
7 files changed, 4 insertions(+), 7 deletions(-)
commit b5528d7ff526d700aab5b9e11aed871180d4f81e
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-09-16 13:32:45 +0100
Require a modern libdbus and a modern GLib
This means we can assume that GLib and libdbus are thread-safe by
default.
Also explicitly document that the object-mapping layer of dbus-glib
is not thread-safe.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=64214
Reviewed-by: Colin Walters <walters@verbum.org>
[with review comments to be fixed in subsequent commit -smcv]
configure.ac | 7 ++-----
dbus/dbus-gthread.c | 9 ++++++---
test/core/Makefile.am | 6 ------
3 files changed, 8 insertions(+), 14 deletions(-)
commit c0d1ffc09b5b008fe9b3ee09330ad663927669af
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-09-16 13:36:28 +0100
Use g_cclosure_marshal_generic for all marshalling
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=64214
Reviewed-by: Colin Walters <walters@verbum.org>
dbus/dbus-gobject.c | 117
++--------------------------------------------------
dbus/dbus-gobject.h | 6 ---
dbus/dbus-gproxy.c | 18 ++------
3 files changed, 6 insertions(+), 135 deletions(-)
commit d3a4fb94e10bacb2c18e0b56456e565471898508
Author: Mike Gorse <mgorse@suse.com>
Date: 2012-08-31 13:24:19 -0500
Remove erroneous assert from connection_setup_add_timeout
When switching a connection from one main loop to another, timeouts
are
moved to the new connection, so connection_setup_add_timeout gets
called on
the new connection for all timeouts that were set on the old
connection.
This means that the timeout's data can, in fact, be non-NULL, and
it will be
freed / removed from the old connection as a side-effect of adding
it to the
new connection. A similar assert was removed from
dbus_connection_add_watch
a while ago as part of the original patch for this bug.
Also, clean up the regression test a bit, and test that a
DBusConnection
can be migrated when a pending call is present.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=30574
[resolve conflicts -smcv]
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
dbus/dbus-gmain.c | 2 --
test/core/30574.c | 65
++++++++++++++++++++++++++++++-------------------------
2 files changed, 36 insertions(+), 31 deletions(-)
commit 40ae208c3309cfa0be27ef458e6947433ff3841e
Author: Jiří Klimeš <jklimes@redhat.com>
Date: 2012-02-06 14:00:06 +0100
docs: fix a typo in dbus_g_type_get_struct() description
Signed-off-by: Jiří Klimeš <jklimes@redhat.com>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=45686
dbus/dbus-gtype-specialized.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
commit b8d44600179172cbf72337f46f4bad58fbefaf8f
Author: Jiří Klimeš <jklimes@redhat.com>
Date: 2012-02-06 12:29:54 +0100
docs: fix a typo in dbus_g_type_struct_get()'s doc
Signed-off-by: Jiří Klimeš <jklimes@redhat.com>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=45686
dbus/dbus-gtype-specialized.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
commit 8f9eb639229410a127958424fe1687135be3a38d
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-09-16 13:31:15 +0100
Document the entire library as deprecated
dbus/dbus-glib.c | 40 ++++++++++++
dbus/dbus-gmain.c | 16 +++++
dbus/dbus-gobject.c | 53 ++++++++++++++++
dbus/dbus-gproxy.c | 70 +++++++++++++++++++++
dbus/dbus-gthread.c | 3 +
dbus/dbus-gtype-specialized.c | 127
++++++++++++++++++++++++++++++++++++++
dbus/dbus-gvalue-parse-variant.c | 2 +
dbus/dbus-gvalue.c | 12 ++++
doc/reference/dbus-glib-docs.sgml | 53 +++-------------
9 files changed, 332 insertions(+), 44 deletions(-)
commit a868cfe62f36bd11f60b5e68a8af854f95cb4931
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-09-16 12:40:52 +0100
Use gtk-doc in no-tmpl mode, moving documentation into the source code
Part of a patch from Rafał Mużyło.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=58698
configure.ac | 2 +-
dbus/dbus-glib.c | 4 ++++
dbus/dbus-gobject.c | 4 +++-
dbus/dbus-gproxy.c | 13 +++++++++++++
dbus/dbus-gtype-specialized.c | 1 +
doc/reference/dbus-glib-docs.sgml | 4 ++++
doc/reference/dbus-glib-sections.txt | 3 ++-
7 files changed, 28 insertions(+), 3 deletions(-)
commit c67ec31532d678c893a6f29060db4a0e2d9f0770
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-09-16 12:39:49 +0100
Use AC_LANG_PROGRAM to check for socklen_t
Part of a patch from Rafał Mużyło.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=58698
configure.ac | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
commit 6efaae028bed13d022020ecf728b7442e8fd737b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-09-16 12:39:04 +0100
Replace deprecated AM_PROG_LIBTOOL with LT_INIT
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
commit 019b5f844f4c7bf33759e1e98236b599a542ba6c
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-09-16 12:38:09 +0100
Use AC_LANG_PROGRAM in AC_COMPILE_IFELSE
Part of a patch from Rafał Mużyło.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=58698
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
commit ffa9f9e5e0926cabbfac5cd661a02cfb43bcc952
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-09-16 12:35:51 +0100
configure.ac: modernize AC_OUTPUT
Part of a patch from Rafał Mużyło.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=58698
configure.ac | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
commit 7678b6f8f43789a880147749bd4b6dc3454d4dec
Author: Alban Crequy <alban.crequy@collabora.co.uk>
Date: 2014-07-29 11:16:47 +0100
demarshal_static_variant: fail gracefully with unknown message types
Refuse to demarshal variants containing unknown types such as
DBUS_TYPE_UNIX_FD
or DBUS_TYPE_MAYBE. Otherwise we will not be able to free them
automatically
with g_value_unset.
Symptoms:
> GLib-GObject-CRITICAL **: g_value_unset: assertion 'G_IS_VALUE
(value)' failed
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=80557
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
dbus/dbus-gvalue.c | 23 +++++++++++++++--------
1 file changed, 15 insertions(+), 8 deletions(-)
commit 7b81cf6445b0802c582dfca4a7e5afec5c9c389e
Author: Alban Crequy <alban.crequy@collabora.co.uk>
Date: 2014-07-28 17:32:16 +0100
_dbus_gtype_from_signature_iter: fail gracefully with unknown
message types
_dbus_gtype_from_signature_iter now returns G_TYPE_INVALID on
unknown message
types such as DBUS_TYPE_UNIX_FD or DBUS_TYPE_MAYBE.
Symptoms:
> arguments to dbus_signature_iter_recurse() were incorrect, assertion
> "dbus_type_is_container (dbus_signature_iter_get_current_type
(iter))" failed
> in file ../../dbus/dbus-signature.c line 213.
> This is normally a bug in some application using the D-Bus library.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=80557
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
dbus/dbus-gsignature.c | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
commit 52e42240ed11a1aeb1e6b6d1c77497e620b36015
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-09-05 15:06:20 +0100
tests: only remove sources that have not already been removed
With modern GLib, misusing GSource provokes a critical warning, which
is made fatal by our test framework.
It is possible (indeed likely) that some or all of the
cancel_exit_timeout() calls just before re-adding the timeout are
redundant, but I didn't want to waste time on assessing that - easier
to just do it every time.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=83530
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/core/test-dbus-glib.c | 54
+++++++++++++++++++++++++++++++++++-----------
1 file changed, 41 insertions(+), 13 deletions(-)
commit eb14381d59ac3fdf7bbd7b75705cbf133cada399
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-02-12 15:36:38 +0000
nano version
NEWS | 5 +++++
configure.ac | 2 +-
2 files changed, 6 insertions(+), 1 deletion(-)
commit 397e8297d433547c9bf4150ddd2b9e0b4c39628c
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-02-12 13:39:09 +0000
update NEWS, release 0.102
NEWS | 17 ++++++++++++++---
configure.ac | 2 +-
2 files changed, 15 insertions(+), 4 deletions(-)
commit f53a35e5d20a2c0010718e7bfd863fc09ff9b1a7
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-02-12 13:32:29 +0000
ignore more Automake droppings
.gitignore | 2 ++
1 file changed, 2 insertions(+)
commit ee0f90d5d619ef53f30edbbeb19c7b6a5055a84b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2014-01-14 11:22:34 +0000
dbus-binding-tool: check for valid interface, member and property
names
Properties are currently allowed to be arbitrary UTF-8 since this
matches
dbus-glib's runtime behaviour, although ideally new interfaces should
use the more restrictive member naming rules (leading to names like
MyProperty) for interop with QtDBus.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=7909
Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
dbus/dbus-gidl.c | 11 +++++++++++
dbus/dbus-gparser.c | 36 +++++++++++++++++++++++++++++++++++-
2 files changed, 46 insertions(+), 1 deletion(-)
commit f1311ce39c6bcab1e25de20eb684bb78cce56e59
Author: Xavier Claessens <xavier.claessens@collabora.co.uk>
Date: 2013-11-19 16:41:50 -0500
dbus_g_value_build_g_variant(): Fix NULL string case
g_variant_new_string(NULL) is not accepted.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71811
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
dbus/dbus-gtype-specialized.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
commit e0edaf682056f487b315c3d8236a06ac8c156b8d
Author: Xavier Claessens <xavier.claessens@collabora.co.uk>
Date: 2013-11-19 16:16:33 -0500
dbus_g_value_build_g_variant(): Fix NULL strv case
g_variant_new_strv(NULL, -1) is not accepted, but (NULL, 0) is good.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=71811
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
dbus/dbus-gtype-specialized.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
commit 2b1b3c4f6c271a64f6767c2cbd9f5713d0bdedaf
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2012-12-04 18:13:10 +0000
Fix yet another leak, this one caused by poor documentation (#29195)
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/core/error-mapping.c | 7 +++++++
1 file changed, 7 insertions(+)
commit b78ab1e3f845ae075afe8880c96568506135506c
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 13:51:13 +0100
Add a list of Valgrind suppressions for false-positives
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
tools/dbus-glib-tests.supp | 64 ++++++++++
tools/dbus-glib.supp | 297
+++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 361 insertions(+)
commit 30a03399929cf943449968d537499c63019d36b3
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2012-12-04 17:56:24 +0000
Add infrastructure for running tests under valgrind
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Makefile.am | 3 +++
test/Makefile.am | 4 ++++
test/core/Makefile.am | 16 ++++++++++++++++
test/interfaces/Makefile.am | 16 ++++++++++++++++
4 files changed, 39 insertions(+)
commit 45869f2d43de75c3fef4b3f5978f096296797257
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 19:06:29 +0100
test-dbus-glib: free numerous leaked objects, and clean up more
thoroughly
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/core/test-dbus-glib.c | 30 ++++++++++++++++++++++++++----
1 file changed, 26 insertions(+), 4 deletions(-)
commit 27b65f054da59f2ce660af6b1c3ecdc2647c07bd
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 19:04:23 +0100
Separate the test for shared-bus equivalence into its own binary
When valgrinding tests it's useful to avoid using the shared bus,
which
cannot be closed. Move all the essential shared-bus use into a
test which
can be treated specially.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
.gitignore | 1 +
test/core/Makefile.am | 6 +-
test/core/run-test.sh | 1 +
test/core/shared-bus.c | 135
+++++++++++++++++++++++++++++++++++++++++++++
test/core/test-dbus-glib.c | 5 --
5 files changed, 142 insertions(+), 6 deletions(-)
commit f3538c4210447c572aa82cb2eb0305ef7f70b2b3
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 18:32:35 +0100
test-types: use a private bus, and close it afterwards
This frees more of the memory.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/core/test-types.c | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
commit 72e865d96b266cc8c9c0586fff2a36382b23d7ec
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 18:32:09 +0100
registrations test: run the connections to completion and call
dbus_shutdown
This frees more of the memory.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/core/registrations.c | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)
commit 48e799e7184722047c4944ab16dc37751cc89227
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 18:31:40 +0100
30574 test: use a private bus so we can free it more thoroughly
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/core/30574.c | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
commit ce8f21c62064a3ccd3bfd2d01444e017312ab1d3
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 18:31:12 +0100
test-variant-recursion: use a private bus so we can free it more
thoroughly
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/core/test-variant-recursion.c | 12 ++++++++++--
1 file changed, 10 insertions(+), 2 deletions(-)
commit 021d157ba43702b05973cbd96fafcf6ec3b96ece
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 18:29:27 +0100
test-variant-recursion: free our GValues and the output string
correctly
Previously, if we'd freed the GValues, it would have crashed, because
g_value_take_boxed on a value of type G_TYPE_VALUE requires that
the inner
GValue was g_malloc'd individually, but we were allocating them as a
block. This only "worked" because *none* of them were freed...
The returned string was also leaked; free it too, but only on success.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/core/test-variant-recursion.c | 45
++++++++++++++++++++++----------------
1 file changed, 26 insertions(+), 19 deletions(-)
commit 8f7fbad772460e1ec9509de481bd572a7a9233d0
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 18:25:37 +0100
registrations test: bin some unused variables
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/core/registrations.c | 3 ---
1 file changed, 3 deletions(-)
commit dacf9d5d09b104469c1b30c6efd4107d03ce79bc
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 13:50:40 +0100
interfaces test: don't call an undefined function on failure, just
use set -e
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/interfaces/run-test.sh | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
commit 79d817e8c11f9dd45f83da30f864f31641e05ac0
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 13:50:11 +0100
test-client: wait for disconnection, to get more state freed
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/interfaces/test-client.c | 15 ++++++++++++++-
1 file changed, 14 insertions(+), 1 deletion(-)
commit 230751f4e63a65294739087ea994bafb320ca3f6
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-09-22 13:49:05 +0100
test-client: don't leak NodeInfo and one proxy
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/interfaces/test-client.c | 4 ++++
1 file changed, 4 insertions(+)
commit 15afdc4c10720c31a4142db66845e806244310a8
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-05-12 12:41:05 +0100
30574 test: don't leak two messages and a pending call per run
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41129
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/core/30574.c | 3 +++
1 file changed, 3 insertions(+)
commit f210a9025fb17b91902b3815411f26eaad4597e7
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2013-02-15 12:00:41 +0000
Add a regression test for CVE-2013-0292
Inspired by Sebastian Krahmer's 'darklena.c', which used
this bug to exploit pam_fprintd.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=68603
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/core/Makefile.am | 4 +
test/core/proxy-noc.c | 264
++++++++++++++++++++++++++++++++++++++++++++++++++
2 files changed, 268 insertions(+)
commit d5e48a022b7da2d073df22775f8aeab98a82f693
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2013-02-18 13:46:23 +0000
Update HACKING for current status
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=68602
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
HACKING | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
commit e1f1926c6b67726206c4c199781831e82553aae9
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2013-10-21 12:24:52 +0100
Bring back NEWS
I'm the only one who does dbus-glib releases these days, and I find it
easier to track changes as we go along than to construct a change
list during the release process.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=68602
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
NEWS | 372
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
1 file changed, 368 insertions(+), 4 deletions(-)
commit f1fc0540d861c7fad17a7d9917db7ff1ddedaa46
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2013-02-18 11:35:47 +0000
README: replace with a deprecation warning
Most of the README was really about libdbus anyway.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=68602
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
README | 33 +++++----------------------------
1 file changed, 5 insertions(+), 28 deletions(-)
commit 725105b4534e90db01adc7cddc88420a419333fd
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2013-08-27 13:16:51 +0100
Avoid underlinking internal library libdbus-gtool.la
Anything that links libdbus-gtool needs to pull in GLib/GObject,
and the order matters (things that libdbus-gtool depends on must
appear after libdbus-gtool itself). libtool understands this, but
only if you tell it the full dependencies.
This broke compilation of test-dbus-glib.exe when cross-compiling
for 32-bit Windows with mingw-w64 (GNU tuple: i686-w64-mingw32).
The linking order used here for dependencies is "lowest in the stack
first", as recommended by the GStreamer documentation.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=68601
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
dbus/Makefile.am | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
commit a466946c5ef882ff4710c624b2f33b3ff81a5ea9
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2013-08-27 12:23:00 +0100
tests: fix under-linking
The test for errors uses dbus_set_error() so it should link -ldbus-1.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=68601
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
test/Makefile.am | 1 +
1 file changed, 1 insertion(+)
commit 9cd58adf12ec11069947032c01e8d12fde9d34fd
Merge: 75c61e7 02b44f8
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2013-02-25 12:56:34 +0000
Merge tag 'dbus-glib_0.100.2'
dbus-glib_0.100.2
Conflicts:
configure.ac
commit 75c61e7d357fac288d4591d32536d395f50ad8bc
Author: Marko Lindqvist <cazfi74@gmail.com>
Date: 2013-02-25 12:40:11 +0000
use AC_CONFIG_HEADERS, not AM_CONFIG_HEADER
Automake-1.13 removed long obsolete AM_CONFIG_HEADER completely
<http://lists.gnu.org/archive/html/automake/2012-12/msg00038.html> and
errors out upon seeing it.
[commit message added -smcv]
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=59003
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
commit 02b44f8427ea48463d7b9f2a9f7099be7643b60d
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2013-02-25 12:24:35 +0000
0.102: respin tarball
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
commit f2d10073a23987c9eef3a6384faa0f7813108936
Merge: ee85b07 78f3e95
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2013-02-15 17:13:33 +0000
Merge tag 'dbus-glib_0.100.1'
commit 78f3e953e4a05a1126874dd2ca6cec5777a15b3b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2013-02-15 12:37:04 +0000
0.100.1 point release
configure.ac | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
commit 166978a09cf5edff4028e670b6074215a4c75eca
Author: Colin Walters <walters@verbum.org>
Date: 2013-02-14 10:19:34 -0500
CVE-2013-0292: dbus-gproxy: Verify sender of NameOwnerChanged signals
to be o.f.DBus
Anyone can hop on the bus and emit a signal whose interface is
o.f.DBus; it's expected at the moments that clients (and notably DBus
libraries) check the sender.
This could previously be used to trick a system service using
dbus-glib
into thinking a malicious signal came from a privileged source, by
claiming that ownership of the privileged source's well-known name had
changed from the privileged source's real unique name to the
attacker's
unique name.
[altered to be NULL-safe so it won't crash on peer connections -smcv]
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
dbus/dbus-gproxy.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
commit ee85b071fb51efb9f9a48d4a658cbefefd14544a
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2011-04-20 16:15:20 +0100
Add a feature test for fd.o #23633, non-default main context
[rebased and added to run-test.sh -dbcw]
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=23633
.gitignore | 1 +
test/core/Makefile.am | 7 ++
test/core/private.c | 268
++++++++++++++++++++++++++++++++++++++++++++++++++
test/core/run-test.sh | 1 +
4 files changed, 277 insertions(+)
commit a90fe0f1d742579b6bcc07980551969c556c184b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2012-11-19 16:12:32 +0000
test-profile: exit a bit more gracefully
We close the connection to ourselves, resulting in libdbus shooting us
in the head unless we ask it not to. Nice to see how thoroughly this
code has been tested...
For future reference, it can be tested like this:
DBUS_TOP_BUILDDIR=$(pwd) ./test/core/run-test.sh profile
(or substitute the real absolute top build directory, if out-of-tree).
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40711
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Dan Williams <dcbw@redhat.com>
test/core/test-profile.c | 6 ++++++
1 file changed, 6 insertions(+)
commit fac9b965fb65352641521d380b1428702015012b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2012-11-19 16:09:14 +0000
test-profile: actually initialize GLib, if required
It's alarming how often the phrase "I'm amazed this ever worked" comes
up while maintaining dbus-glib.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40711
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Dan Williams <dcbw@redhat.com>
test/core/test-profile.c | 4 ++++
1 file changed, 4 insertions(+)
commit e4a163431f2cd26959c9f46e573fc200c5b9fd3d
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date: 2012-11-19 15:55:53 +0000
Remove check for abstract sockets, not relevant to dbus-glib
test-profile.c was its only use, and all it was used for was to listen
on an abstract or path-based socket, matching what D-Bus would do,
when obtaining comparative performance figures for D-Bus vs.
plain Unix sockets.
test-profile.c isn't normally run, only works on Unix, and
the check for abstract sockets was broken on current glibc (fd.o
#29895),
leading to us using the non-abstract code path anyway, so this clearly
wasn't very important. I'm tempted to delete test-profile.c entirely,
but until then, let's just make it use path-based sockets.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40711
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Reviewed-by: Dan Williams <dcbw@redhat.com>
Android.mk | 1 -
configure.ac | 67
------------------------------------------------
test/core/test-profile.c | 20 +--------------
3 files changed, 1 insertion(+), 87 deletions(-)
commit bed0713c0a67b7bd558285b748ff212a3dfdd003
Author: Dan Williams <dcbw@redhat.com>
Date: 2012-10-07 10:55:50 -0500
Add DBusGMethodInvocation accessor for the DBusGConnection