GVariant: One more FreeBSD fix

FreeBSD's malloc() sometimes returns unaligned memory if you are
requesting small sizes.  This can get GVariant into trouble.  For
example, consider the type "mmi" containing the value "just nothing".
According to the type signature, the memory containing this should be
aligned to a boundary of 4 since it might contain an int.  The
serialised size of this value is 1 byte, however, and when you ask
FreeBSD to allocate memory of that size, it knows you can't put an int
into it so it doesn't bother aligning it.

This patch modifies the GVariant serialiser to not assert the alignment
constraint in the case that the size of the serialised data is smaller
than its own alignment requirement.

More alignment-related fixes.

Partial-backout 8a21d8d23317ecebe46007f1fd5f7459bf182415.  The
assertions should have remained relaxed since these functions are used
with non-posix_memalign()ed data.

Remove unused code.

include "config.h" for HAVE_POSIX_MEMALIGN

Use aligned memory in GVariant test cases.

The GVariant serialiser works well with non-8-aligned memory, but the
comparison serialiser in the test case depends on memory being
8-aligned.  Use posix_memalign() to get the memory used by this
serialiser.

[mainloop-test] Fix compilation errors

[tests/gvariant] Handle flavored_free(NULL), since flavored_malloc can return NULL

Updated Spanish translation

Updated Spanish translation

bugster#6955199, on hsfs portfs will fail, and FEN backend will fail
to run lstat and port_associate on root node.

GSource: add g_source_set_name, g_source_get_name, g_source_set_name_by_id

These allow applications to give meaningful names to their sources.
Source names can then be used for debugging and profiling, for
example with systemtap or gdb.

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

GSource: add g_source_set_name, g_source_get_name, g_source_set_name_by_id

These allow applications to give meaningful names to their sources.
Source names can then be used for debugging and profiling, for
example with systemtap or gdb.

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

GDBus: Big-endian fixes

Tested this on my Powerbook G4 12", 867MHz PowerPC G4 running Fedora
11ish.

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

GVariant: deal with non-8-aligned malloc()

Closes bug #619585.

Document that vtable is not copied

Updated Galician translations

.gitignore manpages (*.1)

post-release version bump.

Release 2.25.7.

another occurrence of the last fix (@XSLTPROC@)

man pages: fix broken @XSLTPROC@ substitution

change GSettingsBackend.list() API

Improve the g_file_make_symbolic_link docs

Following a proposal by Neil Williams in bug 619527.

GIO: Add g_d{,c}gettext to PLT check exceptions

GDBusConnection.call(): add 'reply_type' argument

This allows the caller to specify the reply type that they are expecting
for this call.  If the reply comes back with the wrong type, GDBus will
generate an appropriate error internally.

  - add a GVariantType * argument to g_dbus_connection_call() and
    _call_sync().

  - move the internal API for computing message types from introspection
    data to be based on GVariantType instead of strings.  Update users
    of this code.

  - have GDBusProxy pass this calculated GVariantType into
    g_dbus_connection_call().  Remove the checks done in GDBusProxy.

  - Update other users of the code (test cases, gdbus-tool, GSettings
    tool, etc).  In some cases, remove redundant checks; in some other
    cases, we are fixing bugs because no checking was done where it
    should have been.

Closes bug #619391.

Fix up a typo in the g_variant_builder_add() docs

Most of the gdbus test programs build only on Unix

Updated Norwegian bokmål translation

Add two missing files

Updated Galician translations

Updated Galician translations

Updated Galician translations

Clean up man page handling

Remove all formatted man pages from git, and use the same
Makefile fragment in all doc dirs.

add .SECONDARY: rule to preserve generated schemas

If the .gschema.xml file was generated as the result of an implicit make
rule then make would 'rm' it after creating the validity stamp.  This
would cause 'make install' to fail.

Return an error for calls into unknown interfaces or unknown objects

Ryan pointed out on IRC that we didn't do anything here. Looking at
the code, it's painfully obvious that we should be returning an error
here since a comment already says that we've exhausted all possible
options.

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

Use stack-allocated GVariantBuilders

This saves a few allocations.
Also simplify the code a bit in gdbusconnection.

Bug #618616.

Add g_dc[p]gettext, and use it in gsettings

Avoid using LC_MESSAGES, and just call g_d[c]gettext directly.

Bug #617004.

Updated Galician translations

Bug 619142 – Build fixes

 - Fix various #include issues

 - Change #error to #warning for the EXTERNAL authentication mechanism.
   It is not clear if this should work on Win32 at all.

 - Call close() before unlink() for the SHA1 keyring

 - Change #error to #warning so we don't forget to do
   permission checking of the .dbus-keyrings directory

 - Use Win32 SID for the SHA1 auth mech

 - Apparently we can't use word 'interface' as an identifier

 - Implement a _g_dbus_win32_get_user_sid() function. For now it's
   private. Don't know if it should be public somewhere. Maybe in
   a future GCredentials support for Win32? I don't know.

 - GFileDescriptorBased is not available on Win32. So avoid using
   it in GLocalFile stuff. Now, Win32 still uses GLocalFile + friends
   (which works with file descriptors) so expose a private function
   to get the fd for an OutputStream so things still work.

 - Fixup gio.symbols

 - Fixup tests/gdbus-peer.c so it builds

With this, at least things compile and the gdbus-peer.exe test case
passes. Which is a great start. I've tested this by cross-compiling on
a x86_64 Fedora 13 host using mingw32 and running the code on a 32-bit
Windows 7 box.

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

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

Estonian translation updated

Add G_TYPE_CLASS_GET_PRIVATE macro to docs

Version bump.

Release 2.25.6

Add --uninstall option to glib-compile-schemas

If --uninstall is given then don't give an error if the schema directory
is empty.  Instead, erase the gschemas.compiled file, if it exists.
This is the right thing to do in the 'make uninstall' rule, where the
schema directory could very well be left empty as a result.

Modify gsettings.m4 to use this option.

Bug 619038 - increase gsettings.m4 power

handle schema checking, installation, uninstallation, cleaning

Make config.h.win32.in match what configure produces

No semantic changes.

glib-compile-schemas rename missed in docs

change a mention of gschema_compile in the docs

Fix trivial typo in GCredentials code

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

Bug 619031 - method-calls-in-thread test failing

The test was assuming that g_timeout_add() waited for at least the amount of
time given to it before running the function.  This is not the case -- the
function can be run as much as 1ms early.  Make the lower time bound asserted
in the test more permissive to account for this.

Bug 618839  - Typo at translation message

Fix trivial typo in GCredentials code

Add sync method to GSettingsBackend, and pad

whitespace fix

Add missing gunixfdlist.h include in gdbus-example-server.c

GDBus: Fix obvious crasher when looking up annotations

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

Updated Galician translations

gdbus(1): Don't fetch props if introspection data indicates none are available

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

GDBus: Fix introspection of objects registered at /

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

improve thread safety in GDelayedSettingsBackend

  - hold a lock while accessing the tree of delayed values
  - use weak reference counts with the owner object to avoid doing
    g_object_notify on a dead object
  - dispatch the "has-unapplied" notify to the proper main context

GSettingsBackend: make signal dispatch threadsafe

This commit fixes up a few race conditions in the GSettingsBackend, mostly with
respect to change notifications occuring at the same time as the last reference
count on a GSettings is dropped.  With GDBus feeding us our incoming signals in
a separate thread, this is something that could easily happen.

gitignore additions for gdbus, new test cases

GSettings: support emitting signals in threads

The thread-default context that was in effect at the time that the
GSettings was created will be used for emitting signals on that
GSettings.

GSettings tool: work-around GDBus issue

There is currently no way (near as I can tell) to ensure that a message
has been sent when using GDBus.  If we exit() before we are sure, then
it is very possible that the message isn't sent at all.  This behaviour
was observed when using the GSettings commandline tool with dconf.

A quick and dirty workaround for now.

intern a key name instead of using strdup()

Reworked Solaris file event notification for GIO. See
https://defect.opensolaris.org/bz/show_bug.cgi?id=10194

Updated copyright.

Updated Spanish translation

Add GDBus files to POTFILES.in

Updated Galician translations

Updated Indonesian translation

Fix gio tests linking with binutils gold linker

Fix build of gdbus tool with binutils gold linker

Expand information about schema translation

I have added some hints on how to use intltool for translation
of summary and description elements, taken from comments in bug #618523.

Fix build on !linux

Don't define __USE_GNU, thats a glibc-internal macro, and
don't use SOL_SOCKET when not including sys/socket.h.
Maybe this file should be called glinuxcredentialsmessage.c...

Bug #618730

Use the new option name

It's --schema-file now, not --schema-files.
Bug #616864.

Bump version

2.25.5

Updates

Fix issues with GSETTINGS_CHECK_RULE

Rename the --schema-files option to --schema-file, since it only
accepts one file at a time. Change the GSETTINGS_CHECK_RULE to
use it that way, too. And also make it work better with !srcdir
builds.

Bugs #616731 and #616864

GDBus: Use specific variant type in GetAll()

Without this fix, we segfault if the exported object returned an error
on all get_property() calls (in reality, this never happens).

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

GDBus: Fix a double free

Fix an unintentional double free introduced in commit
4ad4c306c3b80620185cf975b402e17a6174aea9.

This bug manifested itself when trying to complete this

 $ gdbus introspect --system --dest <tab>

Plug mem leaks in gdbus tests & examples

Use "&s" instead of "s", and free the variant iters after use.

Bug #618663.

Plug a mem leak in gdbusauth

From valgrind running gdbus-peer test:

==20513== 32 bytes in 1 blocks are definitely lost in loss record 1 of 15
==20513==    at 0x4024E4C: realloc (vg_replace_malloc.c:429)
==20513==    by 0x4079BB1: g_realloc (gmem.c:174)
==20513==    by 0x4099472: g_string_maybe_expand (gstring.c:396)
==20513==    by 0x409A42A: g_string_insert_c (gstring.c:1050)
==20513==    by 0x42169AC: g_string_append_c_inline (gstring.h:153)
==20513==    by 0x421682C: _my_g_input_stream_read_line_safe (gdbusauth.c:336)
==20513==    by 0x421843E: _g_dbus_auth_run_server (gdbusauth.c:1265)
==20513==    by 0x4222B94: initable_init (gdbusconnection.c:1783)
==20513==    by 0x41CF8D5: g_initable_init (ginitable.c:106)
==20513==    by 0x41CFA8D: g_initable_new_valist (ginitable.c:219)
==20513==    by 0x41CF920: g_initable_new (ginitable.c:139)
==20513==    by 0x4223479: g_dbus_connection_new_sync (gdbusconnection.c:2046)

Bug #618650.

Plug mem leak in g_dbus_address_get_stream_sync

==6279== 21,615 (4,708 direct, 16,907 indirect) bytes in 169 blocks are
definitely lost in loss record 12 of 13
==6279==    at 0x4024D2E: malloc (vg_replace_malloc.c:207)
==6279==    by 0x4079A90: g_malloc (gmem.c:135)
==6279==    by 0x4079DC8: g_malloc_n (gmem.c:252)
==6279==    by 0x4097E66: g_strsplit (gstrfuncs.c:2434)
==6279==    by 0x42169A2: g_dbus_address_get_stream_sync
(gdbusaddress.c:875)

Bug #618622.

Plug a mem leak in get_uninitialized_connection

Free the bus address after creating the singleton.

==26308== 39,736 (10,517 direct, 29,219 indirect) bytes in 388 blocks
are definitely lost in loss record 14 of 15
==26308==    at 0x4024D2E: malloc (vg_replace_malloc.c:207)
==26308==    by 0x4079A90: g_malloc (gmem.c:135)
==26308==    by 0x4079DC8: g_malloc_n (gmem.c:252)
==26308==    by 0x4095607: g_strdup (gstrfuncs.c:102)
==26308==    by 0x4216B9A: g_dbus_address_get_for_bus_sync
(gdbusaddress.c:961)
==26308==    by 0x422A7AE: get_uninitialized_connection
(gdbusconnection.c:5241)

Bug #618622.

Plug mem leaks in parse_value_from_blob

The result of read_string() was leaked.

Bug #618615.

Plug a mem leak

This code leaked the return value of g_variant_get_child_value();
use g_variant_get() instead and free the iter when done.

GDBus: Remove cached value if a property is invalidated

Also add a test case to catch this.

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

GDBus: Fix serialization of empty arrays

It turns out that we didn't observe padding (neither when reading nor
writing) for empty arrays which (apparently) is needed according to
the D-Bus spec and reference implementation. A simple test case to
provoke this behavior is as follows (notice the lack of 4 bytes worth
of padding at position 0x0064):

 Error calling dbus_message_demarshal() on this blob: org.freedesktop.DBus.Error.InvalidArgs: Message is corrupted (Alignment padding not null)
 0000: 6c 01 00 01  2e 00 00 00  41 00 00 00  37 00 00 00    l.......A...7...
 0010: 08 01 67 00  08 73 61 7b  73 76 7d 61  73 00 00 00    ..g..sa{sv}as...
 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  11 00 00 00  30 31 32 33    Member......0123
 0050: 34 35 36 37  38 39 30 31  32 33 34 35  36 00 00 00    4567890123456...
 0060: 00 00 00 00  0e 00 00 00  09 00 00 00  53 6f 6d 65    ............Some
 0070: 74 68 69 6e  67 00                                    thing.

 The blob was generated from the following GVariant value:
 ('01234567890123456', @a{sv} {}, ['Something'])

 If the blob was encoded using DBusMessageIter, the payload would have been:

 0000: 6c 01 00 01  32 00 00 00  41 00 00 00  36 00 00 00    l...2...A...6...
 0010: 01 01 6f 00  08 00 00 00  2f 66 6f 6f  2f 62 61 72    ..o...../foo/bar
 0020: 00 00 00 00  00 00 00 00  03 01 73 00  06 00 00 00    ..........s.....
 0030: 4d 65 6d 62  65 72 00 00  08 01 67 00  08 73 61 7b    Member....g..sa{
 0040: 73 76 7d 61  73 00 00 00  11 00 00 00  30 31 32 33    sv}as.......0123
 0050: 34 35 36 37  38 39 30 31  32 33 34 35  36 00 00 00    4567890123456...
 0060: 00 00 00 00  00 00 00 00  0e 00 00 00  09 00 00 00    ................
 0070: 53 6f 6d 65  74 68 69 6e  67 00                       Something.
 ** ERROR:gdbus-serialization.c:547:check_serialization: code should not be reached
 Aborted

and this is now in the libdbus-1-using serialization test case.

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

Updated Spanish translation

bump version

2.25.4

fix glaring inaccuracy in GVariant docs

another !srcdir build fix

More !srcdir build fixes

include gunixcredentialsmessage from unixsocket

fixes a build issue

Cleanups

Sort #includes a bit better, and don't include platform-specific
headers in gio.h.

Fix !srcdir build

Thankfully we already had a SRCDIR define.

Fix a typo

Updates

Documentation cleanups

Merge branch 'gdbus-merge'

Conflicts:
docs/reference/gio/gio-docs.xml
docs/reference/gio/gio-sections.txt
gio/tests/Makefile.am