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 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 c6cbdf9ed99f82983dd529319475dd02c53ad2aa
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2012-06-25 18:23:30 +0100

    Fix builds without tests

 test/manual/Makefile.am |    5 +++++
 1 file changed, 5 insertions(+)

commit a0dd0c8c240896a5dd205c1dbc51924b9d41f833
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2012-06-25 17:26:48 +0100

    Prepare version 0.100

 configure.ac |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

commit 8b7e4a1c4ae055864e26db4addbcc2dc38ee6963
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2012-04-16 12:11:38 +0100

    Fix two crashes when dbus_g_proxy_new_for_peer is used on a bus

    The first part of the bug is that when NameOwnerChanged is received
    with
    a dbus_g_proxy_new_for_peer (which has no name) alive, checking
    whether it was affected by the NameOwnerChanged caused a NULL
    dereference and segfault.

    The second part of the bug is that if the last proxy in existence is
    for a peer, when it was unregistered there would be no
    owner_match_rules,
    causing a crash.

    Both are exercised in the new test added here.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41126
    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 .gitignore              |    1 +
 dbus/dbus-gproxy.c      |    5 +-
 test/core/Makefile.am   |    3 +
 test/core/peer-on-bus.c |  148
 +++++++++++++++++++++++++++++++++++++++++++++++
 test/core/run-test.sh   |    1 +
 5 files changed, 156 insertions(+), 2 deletions(-)

commit 82d45d15088d206f1e1f024215e95301fa04e92c
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2012-04-16 12:10:37 +0100

    Add a utility function to tear down a private connection in tests

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41126
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 configure.ac                |    1 +
 test/Makefile.am            |    4 +--
 test/core/Makefile.am       |    4 ++-
 test/interfaces/Makefile.am |    4 ++-
 test/lib/Makefile.am        |   19 ++++++++++++
 test/lib/util.c             |   69
 +++++++++++++++++++++++++++++++++++++++++++
 test/lib/util.h             |   32 ++++++++++++++++++++
 7 files changed, 129 insertions(+), 4 deletions(-)

commit 403c5bf4d98fcba83caa94aee96da27128bbf9c8
Author: Matthias Klose <doko@ubuntu.com>
Date:   2012-06-25 17:11:45 +0100

    Fix build error with -Werror=format-security

    Origin: vendor, Ubuntu
    Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

 dbus/dbus-gobject.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

commit ae0ed3e435c550c3fae1a86a1c3cb42da319dff2
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-28 17:42:44 +0100

    Move use of 0 as an error domain into the invalid-usage test

    I think this is invalid, although others might disagree.

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40151
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 test/core/error-mapping.c   |   26 --------------------------
 test/manual/invalid-usage.c |   25 +++++++++++++++++++++++++
 2 files changed, 25 insertions(+), 26 deletions(-)

commit 92c613b8f429a4fe6c762805f1a4f3588511e6a7
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-28 17:35:42 +0100

    Add a manual test for various invalid behaviour

    Most of this has been sitting in a branch since fd.o #30171; fixing
    fd.o #40151, another case of library-user error leading to undefined
    behaviour and a hard-to-diagnose crash, seems a good time to get this
    merged.

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40151
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 .gitignore                  |    1 +
 configure.ac                |    1 +
 test/Makefile.am            |    4 +-
 test/manual/Makefile.am     |   27 ++++
 test/manual/invalid-usage.c |  295
 +++++++++++++++++++++++++++++++++++++++++++
 5 files changed, 326 insertions(+), 2 deletions(-)

commit 539bd92b8703134057c039f965ff7a5b5434c3a7
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-28 17:41:35 +0100

    Remove tests in test-dbus-glib which basically just test error mapping

    Also remove the methods on MyObject that only existed to support these
    tests; ThrowError is now versatile enough to implement them all.

    Leave ThrowUnregisteredError as it is, since it violates GError
    expectations (see GNOME#660371), but stop using it in test-dbus-glib -
    it's enough to use it in test-error-mapping.

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40151
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 test/core/my-object.c           |   36 -------------------------------
 test/core/test-dbus-glib.c      |   45
 ---------------------------------------
 test/core/test-service-glib.xml |    9 --------
 3 files changed, 90 deletions(-)

commit b0dc0f419b641f4ab8fdd8eaf69f4ed4c2d61159
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-28 14:56:45 +0100

    Add a new test for error mapping

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40151
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 .gitignore                |    1 +
 test/core/Makefile.am     |    7 +
 test/core/error-mapping.c |  327
 +++++++++++++++++++++++++++++++++++++++++++++
 test/core/run-test.sh     |    1 +
 4 files changed, 336 insertions(+)

commit 5aa25d4f3c44e7f46e95b8d8de64bf63f2c7c42e
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-28 17:23:18 +0100

    MyObject: make ThrowError, AsyncThrowError throw a caller-specified
    error

    This can be used to test arbitrary errors, but only in-process; in
    tests with the service out-of-process, like test-dbus-glib,
    the initial
    error (matching the error they previously threw) will always be used.

    This obsoletes ThrowErrorUnderscore, ThrowErrorMultiWord and
    ThrowNotSupported, but not ThrowUnregisteredError due to some strange
    assumptions about the validity of GError domains in that method
    (see GNOME#660731).

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40151
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 test/core/my-object.c |   49
 +++++++++++++++++++++++++++++++++++--------------
 test/core/my-object.h |    5 +++++
 2 files changed, 40 insertions(+), 14 deletions(-)

commit df3949806818ba0eab495fcaad2c7f3ad38007c7
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-28 16:14:52 +0100

    test-dbus-glib.c isn't GTest yet, but add bug numbers anyway

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40151
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 test/core/test-dbus-glib.c |    4 ++++
 1 file changed, 4 insertions(+)

commit ead4ef065f700a4b586e78400789a7a18c856cc4
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-28 14:57:02 +0100

    Add copyright/licensing information to test-dbus-glib

    I've tried to dig up the copyright holders from git history; possibly
    incomplete, but none of them cared enough to add their own copyright
    notices, so this is the best we'll get.

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40151
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 test/core/test-dbus-glib.c |   26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

commit 309bce6d7965037bc6f1827715d2372ae19c98d5
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-28 13:23:29 +0100

    Form a valid D-Bus error name if an unmapped error has a negative code

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40151
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 dbus/dbus-gobject.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

commit ffd32e448a76045696d81a82fec9aeac7dd5e64b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-28 12:20:16 +0100

    If an error code is out of range for its domain, warn about it

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40151
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 dbus/dbus-gobject.c |   10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

commit a31371cb5852c485b85926125988c0a170a640ca
Author: David Woodhouse <dwmw2@infradead.org>
Date:   2011-09-28 12:16:01 +0100

    Don't crash in gerror_domaincode_to_dbus_error_name if code is out
    of range

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=40151
    Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

 dbus/dbus-gobject.c |    3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

commit 86ba4b0b8591b882803be1a064df4c213349fba0
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-22 18:25:02 +0100

    dbus_g_value_parse_variant: fix several GVariant ref leaks

    g_variant_get_child_value() (and hence g_variant_get_variant(),
    which is
    a simple wrapper) returns a ref to the child; we were assuming
    it didn't.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=41125
    Reviewed-by: Jonny Lamb <jonny.lamb@collabora.co.uk>

 dbus/dbus-gvalue-parse-variant.c |   56
 ++++++++++++++++++++++++++------------
 1 file changed, 38 insertions(+), 18 deletions(-)

commit 51222609f1af2eda0c223881dda3985435c3ad3f
Author: Colin Walters <walters@verbum.org>
Date:   2012-01-08 11:29:33 -0500

    test-service-glib: Fix build with GLib 2.30

    Only <glib.h> can be included directly.

 test/core/test-service-glib.c |    1 -
 1 file changed, 1 deletion(-)

commit 0215b0af4c95c9b9fc79ada5b99882f1261e2644
Author: Colin Walters <walters@verbum.org>
Date:   2012-01-08 11:28:10 -0500

    autogen.sh: Honor NOCONFIGURE=1 environment variable

    http://people.gnome.org/~walters/docs/build-api.txt

 autogen.sh |   24 ++++++++++++++----------
 1 file changed, 14 insertions(+), 10 deletions(-)

commit 6a8daa9569a4a6c2610758b122f46f181ce52448
Author: Derek Foreman <derek.foreman@collabora.co.uk>
Date:   2011-04-20 12:14:23 -0400

    Support building on Android

    Reviewed-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=42532

 .gitignore       |    1 +
 Android.mk       |   35 +++++++++++++++++++++++++++++++++++
 dbus/Makefile.am |    9 +++++++++
 3 files changed, 45 insertions(+)

commit dfdb0de78563a01c3d722e1de83d85e90163feaa
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-30 16:12:29 +0100

    Micro version

 configure.ac |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

commit f669a4aa6c7caaf6cbf9395ed14a1a357c4f6004
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-30 12:06:50 +0100

    Prepare 0.98

 configure.ac |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

commit 1803b6cf35876247bb3e577bec9be8eed8b1b825
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-08-17 19:28:04 +0100

    invoke_object_method: simplify how we return

    Based on review feedback from Cosimo.

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35767

 dbus/dbus-gobject.c |    5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

commit 83bdd09f6124289dd19556da9334906f28064838
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-08-17 19:26:46 +0100

    dbus-gobject: centralize death-by-OOM and use oom() instead of goto

    You can't recover from OOM (in GLib-land) so there's no point in
    complicating the code for the reader.

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35767

 dbus/dbus-gobject.c |   37 +++++++++++++++++++------------------
 1 file changed, 19 insertions(+), 18 deletions(-)

commit 705761ad983392c311c3e66c3ddf7e44e0efbc6f
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-27 18:42:33 +0100

    Fix various minor documentation bugs

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-glib.c                     |    2 +-
 dbus/dbus-glib.h                     |    2 +-
 dbus/dbus-gmain.c                    |   10 +++++-----
 dbus/dbus-gobject.c                  |   33
 +++++++++++++++++++--------------
 dbus/dbus-gproxy.c                   |   33
 ++++++++++++++++++++++++++++-----
 dbus/dbus-gvalue.c                   |    2 +-
 doc/reference/dbus-glib-sections.txt |    6 +++++-
 7 files changed, 60 insertions(+), 28 deletions(-)

commit 0d058307cb914fa61e809b9e0500ac139e286ec6
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-27 17:47:03 +0100

    specialized types: improve documentation and document more things

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-gtype-specialized.c        |  436
 +++++++++++++++++++++++++++++-----
 dbus/dbus-gtype-specialized.h        |    6 +-
 doc/reference/dbus-glib-sections.txt |    1 +
 3 files changed, 388 insertions(+), 55 deletions(-)

commit b3af6fdb643f9293d156c95860270a997b9f6843
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-27 17:50:07 +0100

    dbus_g_type_collection_value_iterate, etc.: check that the type
    is suitable

    Otherwise we'd probably crash when we cast the vtable to an
    inappropriate
    type, and call its methods with inappropriate arguments as a result.

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-gtype-specialized.c |   29 +++++++++++++++++++++++++----
 1 file changed, 25 insertions(+), 4 deletions(-)

commit 3183d79b119d0296924602f19d9fc2a58a7bd597
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-27 17:47:46 +0100

    dbus_g_type_collection_get_fixed: check preconditions on the type
    and vtable

    Previously, if it wasn't a collection or didn't have the
    fixed_accessor,
    we'd just segfault. Not ideal.

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-gtype-specialized.c |   14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

commit 7fcaf6c8c6d0e6ade0e8d5460b5311b564a24d3f
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-27 17:41:36 +0100

    dbus-gtype-specialized: warn if vtables have missing callbacks

    It's just about conceivable that they wouldn't segfault, if
    application
    authors carefully avoided the unimplemented functionality... but
    still.

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-gtype-specialized.c |   19 ++++++++++++++++++-
 1 file changed, 18 insertions(+), 1 deletion(-)

commit 58ed26382bf23fe9fb592d38b5be2c2c7b2e77e2
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-05-31 16:17:10 +0100

    Tidy up docs for DBusGError and its pseudo-methods

    * attach the introductory doc-comments to the enum and macro names,
      not the internal function
    * remove docs for method parameters which didn't actually exist
    * more cross-references
    * make the domain macro visible in the documentation index

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-glib.c                     |   33
 ++++++++++++++++++---------------
 doc/reference/dbus-glib-sections.txt |    1 -
 2 files changed, 18 insertions(+), 16 deletions(-)

commit 4b6482d5987095b474c9582e7415bf94a154d9b4
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-27 16:25:21 +0100

    Make gtk-doc DOC_SOURCE_DIR absolute, fixing out-of-tree docs

    I used an absolute path to avoid differing behaviour with older
    versions
    of gtk-doc, where DOC_SOURCE_DIR was relative to the *source*
    directory.

    Also resync doc/reference/Makefile.am with gtk-doc's example.

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Will Thompson <will.thompson@collabora.co.uk>

 doc/reference/Makefile.am |   51
 +++++++++++++++++++++++++++++++++------------
 1 file changed, 38 insertions(+), 13 deletions(-)

commit 83471e869e464c63063e9e254bc8c40b25de03b7
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-05-31 16:04:29 +0100

    DBusGProxy: be more pedantic about boolean returns

    Returning false doesn't strictly imply that the error is set;
    if you're
    being pedantic enough, it's really a tri-state where TRUE indicates
    success, FALSE indicates failure, and calling the function incorrectly
    results in undefined behaviour (currently a critical warning and
    returning FALSE).

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: NB#218973

 dbus/dbus-gproxy.c |   10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

commit c6aa1d7dbdf04501495e4a3b4cfe6e7d808af6dd
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-05-31 15:57:44 +0100

    DBusGProxy: misc documentation tidying

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-gproxy.c                   |   37
 +++++++++++++++++++++++++---------
 doc/reference/dbus-glib-sections.txt |    4 ++++
 2 files changed, 32 insertions(+), 9 deletions(-)

commit c568ace5ee6a9c13f748d876d69ad50234b2bc78
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-05-31 15:57:15 +0100

    DBusGProxy: make argument names in declaration, definition and
    docs consistent

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-glib.h   |    8 +++----
 dbus/dbus-gproxy.c |   59
 +++++++++++++++++++++++++---------------------------
 2 files changed, 32 insertions(+), 35 deletions(-)

commit dc766e475f3a42872edbd9c3bcc98714dde8bff7
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-05-31 15:55:47 +0100

    DBusGProxy: mark struct contents as private

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-glib.h |    2 ++
 1 file changed, 2 insertions(+)

commit 9d1cce7c64662733f0b48e52e4d5a1e805a3e246
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-05-31 15:40:40 +0100

    Remove declaration of dbus_pending_call_get_g_type, which doesn't
    exist

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-glib-lowlevel.h            |    8 --------
 doc/reference/dbus-glib-sections.txt |    2 --
 2 files changed, 10 deletions(-)

commit 35e049459945a22b87cc61c05bd903790f05cf40
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-05-31 16:17:34 +0100

    Add DBUS_TYPE_CONNECTION, DBUS_TYPE_MESSAGE to the docs, and document
    them better

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-glib.c                     |   15 +++++++++++----
 doc/reference/dbus-glib-sections.txt |    2 +-
 2 files changed, 12 insertions(+), 5 deletions(-)

commit 43a4bf8b038c08d6b8c565495cbf69433bc2dcc6
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-05-31 16:15:43 +0100

    Tidy up docs for DBusGMessage

    * document the instance struct
    * make argument names match the headers
    * document the GType macro, not the function it calls, and mention it
      in the documentation (since this is a boxed type, so it's
      non-obvious)

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-glib.c                     |   28 ++++++++++++++++++----------
 doc/reference/dbus-glib-sections.txt |    2 +-
 2 files changed, 19 insertions(+), 11 deletions(-)

commit 53ac9ee390e173f8e63f276d8c7e211e4fadca4b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-03-29 16:34:38 +0100

    Clean up docs for DBusGConnection

    * introductory doc-comment for the (pseudo-)class
    * attach the doc-comment for the GType to the macro, not the internal
      function
    * rename function parameters to match what the header file says

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 dbus/dbus-glib.c                     |   26 +++++++++++++++++---------
 doc/reference/dbus-glib-sections.txt |    2 +-
 2 files changed, 18 insertions(+), 10 deletions(-)

commit be937e037a6c076cf650bb79aba9c1138da39511
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-03-29 16:42:19 +0100

    Ignore non-API source files correctly

    gtk-doc only cares about the basename of the files.

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 doc/reference/Makefile.am |   17 ++++++++++-------
 1 file changed, 10 insertions(+), 7 deletions(-)

commit 053d41e56050706b512562f13b9ebde7dac3c60b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-03-29 16:42:44 +0100

    Remove stray empty subsection

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 doc/reference/dbus-glib-sections.txt |    3 ---
 1 file changed, 3 deletions(-)

commit 3d02f22d475286db8160604755ee85d24b133005
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-05-31 16:13:24 +0100

    Mention which header to include in the gtk-doc

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 doc/reference/dbus-glib-sections.txt |    8 ++++++++
 1 file changed, 8 insertions(+)

commit 30b6a106ed590c0fd97a783998a19779002e65d8
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-05-31 16:13:07 +0100

    Add some missing symbols to the docs

    Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=37793
    Bug-NB: related to NB#218973

 doc/reference/dbus-glib-sections.txt |    6 ++++++
 1 file changed, 6 insertions(+)

commit a4bc263721fc5a80a78c6b551f35d281c22694ff
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-21 18:14:46 +0100

    Micro version

 configure.ac |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

commit 80e37faeac62a4694514410ec17c2ab4a75d50e6
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-09-21 17:36:11 +0100

    dbus-glib 0.96

 configure.ac |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

commit 4bc172fd5f31123ee883c4027e729b38c2777d7d
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-04-05 17:43:15 +0100

    test-dbus-glib: make lose() abort, and make lose_gerror() more
    informative

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 test/core/test-dbus-glib.c |   21 ++++-----------------
 1 file changed, 4 insertions(+), 17 deletions(-)

commit 9e97f9744e7ffeb04a105087da6143931195ce77
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-04-05 16:25:15 +0100

    get_all_object_properties: if _dbus_gvalue_marshal fails, bail out

    This isn't necessarily OOM: _dbus_gvalue_marshal can fail due to
    programming errors. If so, raise a critical warning, then (if
    that wasn't
    fatal) return a D-Bus error to be sent to the caller.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |   27 +++++++++++++++++++++++++--
 1 file changed, 25 insertions(+), 2 deletions(-)

commit bfd1c05174b86c9dfcf1b22a4fa0b2c552dfc902
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-04-05 18:02:47 +0100

    get_all_object_properties: note which operations can only fail via OOM

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |   12 ++++++++++++
 1 file changed, 12 insertions(+)

commit 30198a2368c9cd19db4b8f595bfad7a719ada7e9
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-04-05 16:21:12 +0100

    get_all_object_properties: skip invalidly-named properties, with
    a critical

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |    9 +++++++++
 1 file changed, 9 insertions(+)

commit a2a3315ac9747dcc7dda860ceb1d1f94ab56d29b
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:   2011-08-17 19:15:05 +0100

    dbus-gobject: add and use connection_send_or_die to check for OOM

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Conflicts:
	dbus/dbus-gobject.c

 dbus/dbus-gobject.c |	 40 +++++++++++++++++++++++++---------------
 1 file changed, 25 insertions(+), 15 deletions(-)

commit 22262581b0956f69f321b9e2eb702e3f15b49823
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:50:38 +0100

    emit_signal_for_registration: assert that signal headers are all valid

    This is an assertion, not a return_if_fail, because we already checked
    the object path in dbus_g_connection_register_g_object and the others
    in export_signals.

    After these checks, dbus_message_new_signal can really only fail
    on OOM.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	  4 ++++
 1 file changed, 4 insertions(+)

commit cccf1e6672fba6b649fe02082c2f087cdf022dcd
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:49:18 +0100

    export_signals: check interface, signal names for validity

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 14 ++++++++++++++
 1 file changed, 14 insertions(+)

commit d09dfcb130be0a17f337bd198a9fb43aab7dfa28
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:43:30 +0100

    object_registration_message: check for OOM when a property is absent

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	  8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

commit 7b92ac1bfd1e50e842549dbb14b845478d114652
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:42:49 +0100

    object_registration_message: make logic/assertions slightly clearer

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

commit b5b27e4ca0d8ca250fa7649bcf2613e8d889a722
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:42:29 +0100

    object_registration_message: handle remote caller errors better

    If the remote process sends us a wrong message, that's its fault, not
    ours; we should send back a comprehensible D-Bus error, and not spam
    to stderr.

    Previously, in each of these cases libdbus would have sent back
    NoReply,
    because we declined to handle the method call.

    One case that's still wrong is passing extra arguments to Get,
    GetAll or
    Set, like so:

	Get("com.example.Iface", "MyProperty", "extra")
	Set("com.example.Iface", "MyProperty", Variant("foo"), "extra")
	GetAll("com.example.Iface", "extra")

    dbus-glib has historically warned, ignored the extra argument(s)
    and sent
    back a reply as if they hadn't been there, whereas a stricter
    implementation (like telepathy-glib's TpDBusPropertiesMixin) would
    have sent back an error reply and done nothing.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 20 ++++++++++++++------
 1 file changed, 14 insertions(+), 6 deletions(-)

commit e5cf2be0e03af4df19618076015cc2a6556ca5d4
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:36:09 +0100

    get_object_property: add brief documentation

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	  5 +++++
 1 file changed, 5 insertions(+)

commit d3aa6c7942cce0cf7b602c6b0854488938f0548e
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:25:55 +0100

    get_object_property: unwind on errors, and avoid returning NULL

    Also treat all errors here as programming errors (because this method
    should never fail), upgrading them from warning to critical; return a
    D-Bus error reply anyway, to be nicer to our callers.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 33 +++++++++++++++++++++++----------
 1 file changed, 23 insertions(+), 10 deletions(-)

commit 98b3b3f323fd4001be0630c3e0a1dc4fb83cdeab
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 15:21:56 +0100

    check_property_access: centralize error handling (and check for OOM)

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 69
 ++++++++++++++++++++++++++++++++-------------------
 1 file changed, 43 insertions(+), 26 deletions(-)

commit 6293849bf0469ce8d6d122b38db5091c21304a30
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-08-17 19:11:14 +0100

    dbus-gobject: check for NULL when producing messages

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
    Conflicts:
	dbus/dbus-gobject.c

 dbus/dbus-gobject.c |	  6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

commit 8ec3166b6a40508a91144efedc10dcb32964498e
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-05 14:56:16 +0100

    dbus-gobject: factor out reply_or_die, error_or_die and add
    error-checking

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35766
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>

 dbus/dbus-gobject.c |	 83
 +++++++++++++++++++++++++++++++--------------------
 1 file changed, 50 insertions(+), 33 deletions(-)

commit 7a1abf5243fe5b7adf7211fd7d8c82851f448192
Author: Simon McVittie <simon.mcvittie@collabora.co.uk>
Date:	2011-04-04 16:52:58 +0100

    invoke_object_method: if marshalling an out argument fails, discard
    message

    We shouldn't report this as OOM, because it probably wasn't (it's much
    more likely to be programming error); we should also continue
    through the
    arguments, so that we don't leak them.

    By abandoning the message as soon as we detect a programming error,
    we can use reply == NULL as an indicator of whether to keep appending.

    Bug: https://bugs.freedesktop.org/show_bug.cgi?id=35767
    Reviewed-by: Cosimo Alfarano <cosimo.alfarano@collabora.co.uk>
