GDBusMessage: Validate header fields when serializing/deserializing

The D-Bus spec mentions exactly what header fields are required for
various message types. Add tests for this as well.

Also disallow empty interfaces for signals since the D-Bus spec says
this is Verboten already.

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

GDBusMessage: Add a way to get/set byte order of a message

Also use this in the test cases to check that serialization to and
from both big and little endian works.

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

GDBusMessage: Assert various things when serializing to a blob

We use g_assert() instead of setting the GError because it is a
programming error if the GVariant contains invalid data - see commit
5e6f762d61db1a5c64bd1d33e5ba112755106581 for where the last hole in
GVariant was closed.

So if we can trust GVariant to only contain valid data (ignoring the
case where unsafe API such as g_variant_new_from_data() is used), why
g_assert() at all with costly g_utf8_validate() checks? Because a) it
is relatively inexpensive; and b) it helps find bugs such as the one
fixed in commit 5e6f762d61db1a5c64bd1d33e5ba112755106581.

If performance is a concern we can play games like introducing
environment variables or other machinery to avoid such "costly"
checks. I doubt it will ever be an issue.

Also replace two "Hmm" TODO item with a static assert - the code that
serializes a gdouble into the D-Bus wire format by treating it as a
guint64 is indeed correct - endianess needs to be taken into account
(see the D-Bus reference implementation for similar code). But we want
to make sure that we're indeed using an architecture/compiler where a
gdouble takes up 8 bytes - hence the assertion.

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

GDBusMessage: Validate UTF-8 strings when serializing from blob

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

remove a junk line

Updated Norwegian bokmål translation

binding: Add a default marshaller for the closure API

The g_object_bind_property_with_closures() function should set a
marshaller if the two GClosures don't have one already. This simplifies
the caller code and avoids duplication.

We need to add a new marshaller to the gmarshal.list matching the
signature of the GBindingTransformFunc function.

GDBusConnection: Debug messages when receiving signals and method invocations

Spell out "RECEIVED SIGNAL" instead of "SIGNAL" to emphasize this is
about receiving a signal, not emitting one (which is "SIGNAL
EMISSION"). Also make the "arrows" point in the "right" direction
("<<<<" vs ">>>>") - like this:

 ========================================================================
 GDBus-debug:Signal:
  <<<< RECEIVED SIGNAL org.freedesktop.DBus.NameOwnerChanged
       on object /org/freedesktop/DBus
       sent by name org.freedesktop.DBus

and

 ========================================================================
 GDBus-debug:Incoming:
  <<<< METHOD INVOCATION org.freedesktop.PolicyKit1.Authority.EnumerateTemporaryAuthorizations()
       on object /org/freedesktop/PolicyKit1/Authority
       invoked by name :1.2176

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

glib-compile-schemas: Pass a C string, not a GString wrapper

Commit 5e6f762d61db1a5c64bd1d33e5ba112755106581 (introducing UTF-8
validity checks for GVariant instances containing strsings) actually
uncovered a bug in glib-compile-schemas - a GString was passed when a
C string was expected.

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

GVariant: validate that passed string is UTF-8

As discussed with Ryan on IRC.

This check is crucial because it guarantees that
g_variant_get_string() will _always_ return valid UTF-8. Except in
cases where the programmer used unsafe API such as
g_variant_new_from_data() and setting @trusted to TRUE.

In fact, this check revealed a flaw in my polkit gdbus port

 (lt-polkitd:11632): GLib-CRITICAL **: g_variant_new_string: assertion
 `g_utf8_validate (string, len, NULL)' failed

and with this I could easily find the problem by using gdb(1) and
G_DBUS=fatal-warnings.

Without this check we'd pass the non-UTF8 string all the way to the
message bus and the bus would then disconnect us. So instead I was
seeing

  g_dbus_connection_real_closed: Remote peer vanished with error:
  Underlying GIOStream returned 0 bytes on an async read
  (g-io-error-quark, 0). Exiting.

and then SIGTERM as raised by g_dbus_connection_real_closed() and my
polkitd process would exit. This behavior is much harder to debug than
failing early (as this patch implements).

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

Fix gdbus-exit-on-close test case

Forgot to update the test case after last commit.

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

GDBus: If exiting b/c remote peer disconnected and error is set, print it

This should make things easier to debug:

  g_dbus_connection_real_closed: Remote peer vanished with error:
  Underlying GIOStream returned 0 bytes on an async read
  (g-io-error-quark, 0). Exiting.

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

GDBusProxy: Don't leak a GSimpleAsyncResult when doing async calls

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

Fix a possible deadlock

the FdSource was calling g_cancellable_disconnect while holding the
main context lock, which is bad news if the ::cancelled handler is
trying to get that lock to wake up the mainloop...

Bug 586432

fix the completion test

improve completion test coverage

Don't assert too much

Move bookmarkfile tests to the test framework

Update the Git ignore files

Don't add (extra) DESTDIR to tapsetdir for tapset_DATA.

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

docs: Clarify user_data behavior for bind_property_full()

binding: Add G_BINDING_INVERT_BOOLEAN

Since GSettings got the same functionality and flag in commit ca3b7b75b
GBinding should also have the ability to automatically invert a boolean
value without requiring a custom transformation function.

binding: Add a closure-based variant of bind_property_full()

Since using the function pointer version muddles the memory management
requirements of language bindings, we should implement a GClosure-based
variant on top of g_object_bind_property_full().

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

GSettings: add G_SETTINGS_BIND_INVERT_BOOLEAN flag

When binding a boolean setting to a boolean property, invert the values.
This avoids the requirement for writing a pair of mapping functions for
this extremely common case.

Add a test.

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

On windows, export also symbols defined in gio/gwin32appinfo.c

Updated Galician translations

GDBusConnection: fix message flags in g_dbus_connection_call_sync()

GDBusCallFlags were used instead of GDBusSendMessageFlags for
g_dbus_connection_send_message_with_reply_sync(). Fixes bug #625753.

Add comment to g_date_set_time_val about local timezone.

Local time zone is used in conversion during g_date_set_time_val().
We now mention this in the documentation.

Fixes bug #625500

Silence valgrind

Initialize all fields of struct msghdr. Bug 625472.

Updated Spanish translation

Updated Hebrew translation.

Also use trylock in the bitlock tests

improve scanner api test coverage

Improve tree test coverage

Improve keyfile test coverage

Test some GError functionality explicitly

Add a test for g_variant_builder_add_parsed

Fix a typo

Improve string test coverage

Improve byte array test coverage

Improve gdbus test coverage

Fix a typo

Its credentials, not crendentials

Fix an off-by-one error

Add a test for g_dbus_proxy_get_cached_property_names

improve printf tests coverage

Improve option context test coverage

Improve array test coverage

Move markup parse tests to the test framework

Improve coverage of buffered output stream tests

Improve coverage of credentials tests

Improve coverage of converter stream tests

Improve coverage of memory stream tests

Remove redundant include

Improve coverage of icon tests

Improve coverage of application tests

Improve coverage of stream tests

Reenable unix-streams test

Work around deadlock in unix-streams test

Add tests about file associations

Improve regex test coverage

Remove duplicate utf8 strreverse test

Also add some tests for find next/prev char.

Improve strfuncs test coverage a bit

Improve option context test coverage a bit

Move markup collect tests to the test framework

Add some more GFile tests

Improve compressor test coverage a bit

Test g_settings_list_schemas

Improve GNode test coverage a bit

Add some logging tests

Silence log handler tests

Improve tree test coverage

Move markup escape tests to test framework

More pattern tests

Add tests for g_utf8_strreverse

Fix a warning

Don't leak in converter test

Fix an icon test

Add tests for GNetworkAddress

Drop an unneeded if

Remove some unused pcre code

Add some volume monitor tests

Although not much of this is easily testable.

Move queue tests to the test framework

Bug 625628 – GDBusProxy: wrong property name sorting

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

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

Fix gschema-compile test

This was broken by commit 1b5b87bfa0f5435dd6e973a4e36076db083dc5c7

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

Bug 625584 – Crashes application on unref with signal subscription

Don't do too much work in the finalizer - in particular, there's no
need to send RemoveMatch() messages to the bus daemon since we're
going to sever the connection and the bus will garbage collect
anyway. In this case it crashed the process.

Also add a test case that checks that the appropriate GDestroyNotify
callbacks are called when unreffing a connection with either 1)
exported objects; 2) signal subscriptions or 3) filter functions
.. yes, ideally apps would unregister such callbacks before giving up
their ref but that's not how things work :-)

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

GSettings: Mark some strings for cat translation

<vuntz> *miaou*

GDBusConnection: Fix up debug output when completing calls

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

GCredentials: use effective uid/gid

Otherwise e.g. setuid root processes can't connect to the system
bus. This was discovered when porting PolicyKit's pkexec(1) command to
a PolicyKit library using GDBus.

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

*bump*

add one more NEWS item

GIO: Add G-I annotations for several methods

Specifically:
 - g_io_stream_get_input/output_stream()
 - g_socket_client_get_local_address()
 - g_socket_connection_get_socket()
 - g_socket_listener_add_address()
 - g_socket_listener_accept_socket()
 - g_socket_listener_accept()
 - g_io_extension_point_get_extensions()

get a start on the NEWS

[gschema-compile] fix segfault

If files are passed on the command line and we don't set override_files
at all, don't try to read from the array.

Use correct type for GDBusServer:active property

It's a boolean, not a string.

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

GDBusConnection: Pass the GObject to g_simple_async_result_new()

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

Add support for vendor override files

You can drop a key-file in the schema directory that looks like:

[org.gtk.Example]
key='value'

to override the default value of 'key' in schema 'org.gtk.Example'.

Updated Galician translations

Updated Spanish translation

Updated Hebrew translation.

[gio]: gioenums.h: Remove trailing comma again.