GDBusMessage: Fix bug when deserializing a message

See https://bugzilla.gnome.org/show_bug.cgi?id=621838 for the whole
story. The problem was that we ended up reading data from arrays of
arrays when we were just supposed to be aligning the buffers.

Also add a host of debug infrastructure that was needed to find the
root cause. For now it can be turned on only via defining
DEBUG_SERIALIZER. In the future we might want to make it work via
G_DBUS_DEBUG. In a nutshell, the added debug info looks like this

Parsing blob (blob_len = 0x0084 bytes)
  0000: 6c 01 00 01  3c 00 00 00  41 00 00 00  37 00 00 00    l...<...A...7...
  0010: 08 01 67 00  08 61 61 79  61 7b 73 76  7d 00 00 00    ..g..aaya{sv}...
  0020: 01 01 6f 00  08 00 00 00  2f 66 6f 6f  2f 62 61 72    ..o...../foo/bar
  0030: 00 00 00 00  00 00 00 00  03 01 73 00  06 00 00 00    ..........s.....
  0040: 4d 65 6d 62  65 72 00 00  00 00 00 00  34 00 00 00    Member......4...
  0050: 03 00 00 00  63 77 64 00  01 73 00 00  23 00 00 00    ....cwd..s..#...
  0060: 2f 68 6f 6d  65 2f 64 61  76 69 64 7a  2f 48 61 63    /home/davidz/Hac
  0070: 6b 69 6e 67  2f 67 6c 69  62 2f 67 69  6f 2f 74 65    king/glib/gio/te
  0080: 73 74 73 00                                           sts.

Parsing headers (blob_len = 0x0084 bytes)
  Reading type a{yv} from offset 0x000c: array spans 0x0037 bytes
    Reading type {yv} from offset 0x0010
      Reading type y from offset 0x0010: 0x08 '
      Reading type v from offset 0x0011
        Reading type g from offset 0x0014: 'aaya{sv}'
    Reading type {yv} from offset 0x001e
      Reading type y from offset 0x0020: 0x01 '\ 1'
      Reading type v from offset 0x0021
        Reading type o from offset 0x0024: '/foo/bar'
    Reading type {yv} from offset 0x0031
      Reading type y from offset 0x0038: 0x03 '\ 3'
      Reading type v from offset 0x0039
        Reading type s from offset 0x003c: 'Member'
Parsing body (blob_len = 0x0084 bytes)
  Reading type (aaya{sv}) from offset 0x0047
    Reading type aay from offset 0x0048: array spans 0x0000 bytes
    Reading type a{sv} from offset 0x004c: array spans 0x0034 bytes
      Reading type {sv} from offset 0x0050
        Reading type s from offset 0x0050: 'cwd'
        Reading type v from offset 0x0058
          Reading type s from offset 0x005b: '/home/davidz/Hacking/glib/gio/tests'
OK

Signed-off-by: David Zeuthen <davidz@redhat.com>

Distcheck fix

Add a missing symbol to gobject.symbols.

Make g_settings_sync() a proper prototype.

Fix the build

Updates

GDBus: Complain via g_warning() if an invalid message is received

While we are already propagating the error to the user via the
GDBusConnection::disconnected signal (because the only safe thing is
to disconnect the other peer), changes are the user is simply not
listening to this signal.

This should never ever happen unless there's a bug in the
serializaer/deserializer so it's fine to complain via g_warning()
here.

Bug 621838, see

 https://bugzilla.gnome.org/show_bug.cgi?id=621838

is related to this.

Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusMessage: Properly check error and bail if set

Otherwise we may set a GError on top of it.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Documentation fixups

fix a typo

Fix GApplication for the G_TYPE_VARIANT change

Update api docs to current api

Catch up with G_TYPE_VARIANT changes

The GType for a GVariant is now a fundamental GType instead of a boxed
one so use the right marshaller.

Signed-off-by: David Zeuthen <davidz@redhat.com>

Fix a typo

GSettings: remove "context" from backend too.

Note: this is a GSettingsBackend API/ABI change.

Add fundamental type and pspec for GVariant

Make G_TYPE_VARIANT a fundamental type instead of boxed, and add
g_variant_{set,get,dup}_variant.

Add GParamSpecVariant.

Bug #610863.

Revert "GVariant: add g_variant_default_value()"

This reverts commit b205dc77cb14f67818eaac23d0eb3f5d1b867921.

We decided to go the other route with the GParamSpec.

Only run the schema compiler from the test cases

Bug 621266  - GSettings "context" clarification

Remove the concept of "context" in favour of dealing with
GSettingsBackend directly.

Bug 621905 – Assume a ref when doing async work

When asynchronously acquiring the DBus connection, assume a reference to
the proxy object, to avoid destroying it in the middle of the operation.

See https://bugzilla.gnome.org/show_bug.cgi?id=621905

GSettings-related distcheck fixups

Add 'if' that was absorbed by $(AM_V_GEN)

From commit 1b8ee5196eb8ed684b0ee0d2f0d26e6bd82656af

More GSettings doc fixups

<alias> takes mandatory target='' arg

GSettings: Fix comment formatting, whitespace.

Add new api

Add some schema examples

Typo fix

Update the dtd to match the schema parser

The aliases element was not mentioned here.

Updates

GSettings: enum/choices/aliases/range test cases

Fix a small bug that one of the tests uncovered.

move GSettings(Backend) to giotypes.h

Bug 621319 - more leaked GVariants in GSettings

Caught by Felix Riemann.

GVariant serialiser: fix small type error

GSettings: major refactor.  Add enums, range.

GVariant: add g_variant_default_value()

Returns various kinds of false, zero, empty arrays, etc.

Typo fix

Typo fix

Rework GApplication API to use GInitable

https://bugzilla.gnome.org/show_bug.cgi?id=620952

Add gdbus-proxy-well-known-name to the ignore file

binding: Use a hash table

Since an object instance might have a lot of bindings, using a list
might become a performance issue when removing them. Using a simple
hash table guarantees constant time lookup, which is the most common
operation.

tests: Add a chain binding

Test the case with a chain like A → B, B → C, and what happens when
switching to a direct A → C link.

https://bugzilla.gnome.org/show_bug.cgi?id=621782

Properly initialize GError

Pointed out by Florian Müllner in bug #621702

Sprinkle some $(AM_V_GEN) around to make the build more silent.

Correct some documentation mistakes in gunixmounts.c

Pointed out by Ross Burton in bug #618904

gobject: Add GBinding

GBinding is a simple, opaque object that represents a binding between a
property on a GObject instance (source) and property on another GObject
instance (target).

https://bugzilla.gnome.org/show_bug.cgi?id=348080

Introduce g_object_notify_by_pspec()

g_object_notify_by_pspec() will emit the "notify" signal on the given
pspec, short-circuiting the hash table lookup needed by
g_object_notify(). The suggested and documented way of using
g_object_notify_by_pspec() is similar to the way of emitting signals
with their ID.

Emission tests (with no handler attached to the notify signal) show a
10-15% speedup over using g_object_notify().

https://bugzilla.gnome.org/show_bug.cgi?id=615425

Prevent division by zero if no tests

Forgotten file

Add subunit support to gtester-report

This patch adds subunit support to gtester-report via a -s switch. Subunit
(https://launchpad.net/subunit) is a language neutral test activity protocol.
This can be used to integrate gtester tests with the Hudson CI tool amongst
other things.

Bug #611869.

Use optparse to parse gtester-report commandline options

See bug #611778.

 Add package and version to the test report XML

Reword a doc comment

To take out some awkward wording.

Let GLIB_SETTINGS macro use glib-compile-schemas from PATH when cross compiling

See bug #621172

Use native glib-compile-schemas when cross compiling

Switch to using variants for timestamps, split out signals

Like how we're handling activation, use GVariant for timestamps.  To
avoid polluting the GtkApplication API with GVariants, we rename the
GApplication signals to "quit-with-data" and "action-with-data".
GtkApplication will then wrap those as just "quit" and "action".

https://bugzilla.gnome.org/show_bug.cgi?id=621002

Add --xml to gdbus-tool to print raw introspected XML

Bug #621442.

Avoid a race in application registration

We need to register the object before taking the name.
Remove the workarounds for this race from the test suite.

Updated Galician translations

Accept no-reply errors

This can happen if the app quits before the dbus reply can be
sent out.

Make the tests not fall over every other time

Bug 621213 – GDBusProxy and well-known names

Allow constructing a GDBusProxy for well-known names as discussed here
http://mail.gnome.org/archives/gtk-devel-list/2009-October/msg00075.html
including test cases.

Make it possible to create a GDBusProxy for a GBusType instead of a
GDBusConnection. This requires G_BUS_TYPE_NONE so add that too.

Nuke g_bus_watch_proxy() since one can now more or less use GDBusProxy
for this.

Port gdbus-example-watch-proxy to this new API and include this
example in the GDBusProxy doc page.

Also nuke the GType parameter from the GDBusProxy constructors as
requested here: https://bugzilla.gnome.org/show_bug.cgi?id=621229

Also update the porting guide and other API docs for this change.

Also fix a bug in the signal dispatching code so each subscriber only
get notified once, not N times, for the same signal. Also add a test
case for this.

https://bugzilla.gnome.org/show_bug.cgi?id=621213

Signed-off-by: David Zeuthen <davidz@redhat.com>

gitignore GApplication test-cases

Close memory leak in GSettings

Fixes bug #621252.

converter streams: make code more readable

The name buffer_availabile was kinda confusing, so its been renamed
to buffer_data_size() to match buffer_data().

Also I added a comment to buffer_ensure_space because its behaviour
wasn't obvious.

Use correct offset for buffer data in GConverterOutputStream

Otherwise we overwrite already converted data in the case that more
than one call to g_converter_convert is necessary in write() or
flush().

https://bugzilla.gnome.org/show_bug.cgi?id=619945

Ensure we always have some target space in GConverterOutputStream

When the converter fills the whole buffer without reading all input,
we need to enlarge the buffer. Otherwise we get an assertion failure
for `outbuf_size > 0' in g_converter_convert.

https://bugzilla.gnome.org/show_bug.cgi?id=619945

Add a regression test for GConverterOutputStream corruption

See bug #619945 for background

GSettings test case fixes

 - call g_settings_sync()
 - fix bug #618715 by not forking if a backend is specified via
   environment variable

Add g_settings_sync() and use it

[Gio] Merge in introspection annotations from gobject-introspection gio-2.0.c

This is not an exhaustive set, but covers everything we have so far.

GSettings: store (default, options) in gvdb

gvdb just dropped the ability to have a separate "options" field.  We
now store the options into a GVariant along with the default value.

For now, we use a small shim in GSettingsSchema in order not to touch
too much code.  A more complete rewrite will follow.

This represents a change to the schema file format with another likely
to follow.  glib-compile-schemas needs to be re-run after installing
this change.

Merge remote branch 'gvdb/master'

drop "options" support

Add _with_closures alternative functions for those in GDBus that
accept more than one callback.

g_bus_own_name_with_closures
g_bus_own_name_on_connection_with_closures
g_bus_watch_name_with_closures
g_bus_watch_name_on_connection_with_closures
g_bus_watch_proxy_with_closures
g_bus_watch_proxy_on_connection_with_closures

https://bugzilla.gnome.org/show_bug.cgi?id=621092

[GApplication] Tweak docs a bit

Rewrite apps test to ensure children are killed

Create a function run_with_application that both ensures the
app is running exactly while the test is running, which most
of the tests use.  We start it beforehand, and kill it after.
This avoids having any interdependence between the tests (and
there definitely was before, because we didn't wait for
the process to actually terminate after a kill() call).

Also, open a pipe between the two, and have the child app
monitor that pipe.  If it gets closed (e.g. because the parent
died), the child exits.  This is the most reliable way to
avoid stale children; before, if we failed an assertion, the
parent would abort, and not run kill().

https://bugzilla.gnome.org/show_bug.cgi?id=621034

Mention GVariant format string docs from iter docs

It's not entirely clear what @format_string in iter_next() and
iter_loop() should be.  Include a link to the GVariant format string
docs as a hint.

Plug memory leak on Windows

Intern the string returned from g_win32_getlocale() and then free it.
Fixes bug #621168.

Fix GApplication tests to run without a session bus

We reuse code from the GDBus tests here to launch a session bus.

GDBusConnection: Do not dispatch calls to unregistered objects or subtrees

There was a slight race where we ended up calling into user code if
the user managed to unregister an object (or subtree) in the window
between

 - processing the remote call on the worker thread; and
 - continuing handling it on the user code thread (via an idle handler)

This patch fixes the problem.

Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusProxy: Fix error handling in synchronous initialization codepath

Signed-off-by: David Zeuthen <davidz@redhat.com>

g_variant_builder_add_parsed: fix type error

Problem caught by Juan A. Suarez Romero

Do not do update-po at distcheck time to avoid touching files checked into version control

Bug 621119 – GDBusProxy and objects with no properties

Fix proxy construction for objects with no properties in the case
where G_DBUS_PROXY_FLAGS_DO_NOT_LOAD_PROPERTIES isn't set.

The unfortunate side-effect here is that GDBusProxy can no longer be
used to test for "object existence", e.g. creating a GDBusProxy for
any path and interface will not fail. But that's not really a big
deal, if apps rely on that they are doing something very wrong.

https://bugzilla.gnome.org/show_bug.cgi?id=621119

Signed-off-by: David Zeuthen <davidz@redhat.com>

Fix warning

Disable functions defined but not used.

Fix warnings

Do explicit casts to avoid warnings.

Updated zh_CN translation.

Fix thinko

When replacing strcmp() with g_variant_is_of_type(), remove the "== 0"
part! Found by Colin Walters.

[GDBusConnection] Use Gio's default async implementation again

The fix was committed in git.

https://bugzilla.gnome.org/show_bug.cgi?id=620990

g_dbus_method_invocation_return_value consumes the floating variant

Bug #620953.

Simplify variant builder

Build the full return value with one builder, and don't unref the
unowned return value!

Bug #620954.

Use g_variant_is_of_type()

... instead of strcmp()'ing the type strings.

Bug #620954.

Plug a mem leak

Bug #620954.

[docs] Fix typos in some g_file_* functions

Reported by Alexander Saprykin in bug
https://bugzilla.gnome.org/show_bug.cgi?id=620947

Bump version

2.25.8

Add missing marshaler

Fix a missing parameter in a doc comment

Pointed out by David Zeuthen.

Fix a signal signature

GApplication::action was erroneously declaring the timestamp parameter
as int instead of uint.

Fix a misspelt doc comment