Add a stress-test for GDBusProxy in threads with no default main context

Destroying a GDBusProxy in a thread used to race with NameOwnerChanged
being delivered to the main context's thread (GNOME #651133).

Also, g_dbus_proxy_call_sync in a thread would race with NameOwnerChanged
being delivered to the main context's thread and rewriting the name_owner
(GNOME #656039).

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=656039
Bug-NB: NB#259760
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: David Zeuthen <davidz@redhat.com>

GDBusProxy: hold properties_lock while using any mutable property

This changes the meaning of "properties_lock" from "lock for D-Bus
properties" to "lock for GObject properties".

The most common problem, and the only one I've reproduced in a regression
test, is name_owner, which can be updated by the thread that owns
the GDBusProxy's main context (i.e. the thread-default main context of
the thread that constructed it) at the same time that a blocking call
is made. When a GDBusProxy is constructed in a thread-pool thread for
short-term use, the main context will typically be the global default
main context (which is actively running in the main thread!), making
this extremely problematic.

The interface info is perhaps a theoretical concern - one thread could
conceivably set it at the same time that another thread uses it, but only
in relatively pathological situations. The current API for this does have
the problem that it returns a borrowed ref, but interface info is
hopefully permanent anyway.

The default timeout is probably only a theoretical concern - it's just an
int, so writes are indivisible, and there's no worry about whether
something has been freed - but to be safe, let's hold the lock for that
too.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=656039
Bug-NB: NB#259760

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

GDBusProxy: factor out async_init_data_set_name_owner

This removes the need for async_init_get_name_owner_cb to cope with being
called without a real GAsyncResult, and will simplify the addition of
correct thread-locking.

In async_init_data_set_name_owner, use the name_owner parameter instead
of the corresponding member of GDBusProxyPrivate, partly to reduce
pointer-chasing but mainly to avoid needing to hold the lock.

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

g_dbus_proxy_get_property: use accessors for all mutable state

These ought to have thread-locking, and having it in the accessor seems
better than duplicating it here.

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

GDBusProxy: if a well-known name is not owned, don't GetAll from the dbus-daemon

If you run:

    ( cd gio/tests && G_DBUS_DEBUG=all ./gdbus-proxy-well-known-name )

you can see that in the case where the name com.example.TestService isn't
owned yet, the GDBusProxy calls GetAll() with no destination, resulting
in an error reply from the peer (the dbus-daemon itself). That's clearly
not right!

However, if priv->name is NULL, that indicates the special case where we
really do want to talk directly to a peer, instead of via the bus daemon
(most likely to be used on peer-to-peer connections); in that special
case, do call GetAll().

Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>
Signed-off-by: David Zeuthen <davidz@redhat.com>

g_settings_bind: add some g_return checks

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

ghash: fix error in "as a set" documentation

GDBusObjectManagerClient: Emit signals on proxy before emitting on manager

This is needed because the proxy may need to update its internal state
which a signal handler connected to the manager may rely on.

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

open(2): POSIX compatibility.

Posix allows for open(2) to fail with errno = EINTR.
Normal this isn't seen when opening files. However in some case we are
opening a fifo for write which will block until another process opens it
for read. If a signal is received while blocked, open(2) fails with
errno = EINTR.

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

Add g_mkdtemp in the spirit of g_mkstemp

At the same time, also add g_mkdtemp_full and g_dir_make_tmp
variants. The patch also unifies the unique-name-generating
code for all variants of mkstemp and mkdtemp and adds tests
for the new functions.

Based on patches by Paolo Bonzini,
http://bugzilla.gnome.org/show_bug.cgi?id=118563

GHmac: pedantic doc fixes

Add a link to an explanation of what HMAC is, and tweak
some formatting.

News for 2.29.16

hmac: Implementation of HMAC in glib

This implements g_hmac_xxx() functionality using the standard checksum
functions supported by glib.

HMAC is a secure way to hash a key and a password. Many other
approaches fraught with append and prepend issues.

Includes test cases defined in relevant RFCs

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

Add G_VALUE_INIT

The implementation of GValue is not public or documented.  When
allocated on the stack, initializing a GValue is usually done as
documented with:

GValue value = { 0, };

There is lot code around (including WebKit) that added all the missing
fields, resulting in this ugly and non-obvious:

GValue value = { 0, { { 0 } } };

However, this doesn't play nice with -Wmissing-field-initializers for
example. Thus, G_VALUE_INIT.

http://bugzilla.gnome.org/show_bug.cgi?id=654793
http://bugzilla.gnome.org/show_bug.cgi?id=577231

Don't finish gtls connection if the handshake failed

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

GDBusProxy: on_properties_changed initialize some variables

This avoids calling g_variant_unref and g_free on uninitialized memory
if PropertiesChanged is received in the creating thread's thread-default
main context's thread, at the same time as releasing the last ref in
another thread. This would result in "goto out" before the variables
freed after that label had been initialized to NULL.

Based on a patch by Simon McVittie, bug 656282

gdatetime: Add a test for %OM

Remove a gccism

Pointed out in bug 656152.

Fix statfs/statvfs decision

We want to force use of statvfs when statfs is deficient.
This does not make any difference on Linux.

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

Trivial typo fix

In addition to /media and $HOME, also show mounts in $XDG_USER_DIR

Prepare for the future where udisks will use $XDG_USER_DIR/Volumes
instead of /media when mounting filesystems on behalf of the user.

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

Updated Indonesian translation

Update VS property sheets ("install" phase)

Copy the new GioTLS headers that were introduced into the GIO
library in commit  0f99cfa8822514dc6b7ccbc59efbe68f27aeb172 during the
"install" stage.

Updated Bulgarian translation

[gobject] Fix G_DEFINE_BOXED_TYPE compilation with C++

[gi] Add missing (out) annotation to g_dbus_{node,interface}_info_generate_xml()

[gi] Add missing Gio transfer annotations

[gi] Fix parameter name typos in Gio docstrings

Found by GIR compiler when building gobject-introspection:

gir/gio-2.0.c:33525: Warning: Gio: g_tls_password_set_description: unknown
parameter 'flags' in documentation comment, should be one of 'password',
'description'
gir/gio-2.0.c:14568: Warning: Gio: g_action_group_action_state_changed: unknown
parameter 'state' in documentation comment, should be one of 'action_group',
'action_name', 'value'

Bug 652827: Update config.h.win32.in

Add check macro for HAVE_WIN32_BUILTINS_FOR_ATOMIC_OPERATIONS, as it is
now required for MSVC builds of glib/gatomic.c GLib 2.29.15+.

It is true that the MinGW cross-compiler on Linux systems will have
HAVE_GCC_BUILTINS_FOR_ATOMIC_OPERATIONS and
HAVE_WIN32_BUILTINS_FOR_ATOMIC_OPERATIONS defined during the completion
of ./configure, but since this file is primarily meant for people
compiling -on- Windows (and that the "native" Windows MinGW would neither
./configure to define HAVE_GCC_BUILTINS_FOR_ATOMIC_OPERATIONS and
HAVE_WIN32_BUILTINS_FOR_ATOMIC_OPERATIONS), this file will be updated as
it is for now at least until the situation for "native" Windows MinGW
change. (please see Bug 652827 regarding this paragraph)

Bug 652827 - glib-2.29.8 no longer builds with mingw.org's toolchain

Check for Win32 atomic intrinsics.

Cleanup gatomic checks

Fix typo

[gi] Add missing transfer annotation to g_dbus_node_info_lookup_interface()

Updated Norwegian bokmål translation

[gi] Add missing (allow-none) annotations to g_dbus_connection_register_object()

Ignore lcov-produced files in top-level .gitignore

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=656162
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Allow current lcov (version 1.9) to be used for coverage

It seems to work fine with the same invocation already used.

Bug: https://bugzilla.gnome.org/show_bug.cgi?id=656162
Signed-off-by: Simon McVittie <simon.mcvittie@collabora.co.uk>

Document libffi as new dependency

Updated Galician translations

Require Python 2.5 explicitly

Without someone to regularly test 2.4, and since the code in reality
requires 2.5, switch to that for now.

configure: Fix statvfs/statfs detection

Add missing 'x' as pointed out by declanw@is.bbc.co.uk

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

Updated Hebrew translation.

Add missing GVariant annotations

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

gio/tests/.gitignore: fix

Updated French translation

[l10n] Updated German translation

gio enums: Remove trailing commas

Fix up ABI symbols after GTlsDatabase merge.

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

GTlsDatabase and related objects

The database is an abstract object implemented by the various TLS
backends, which is used by GTlsConnection to lookup certificates
and keys, as well as verify certificate chains.

Also add GTlsInteraction, which can be used to prompt the user
for a password or PIN (used with the database).

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

build: Pass ZLIB_CFLAGS for gio build

Otherwise we will fail if zlib is in an alternative prefix.

Commit message written by Colin Walters <walters@verbum.org>

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

g_cancellable_get_fd: silently return -1 for NULL cancellable

This keeps compatibility with previous behavior.

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

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

GDBusConnection: Only apply exit-on-closed semantics if properly initialized

Otherwise there is no point in gracefully handling the error...

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

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

Updated Russian translation

Updated Spanish translation

Updated Swedish translation

Revert "Optimize g_[s]list_free_full a bit"

This reverts commit 98b667d052b1274f80b8898a32d0753e9e2e5c1a.

The commit was not actually an optimization, since g_list_free is
pretty smart.

Added TLS PEM parser unit test

Rework PEM parsing to not be order dependent

Some valid PEM file would not work because the private key was put
before the certificate.

Docs: Correct the namespace for g_file_query_filesystem_info()

Updated glib-zip.in a bit further

Missed the bin/gsettings.exe part (corrected from bin/gsettings.)

utf8: annotate the end pointer in g_utf8_validate as out + allow-none

More gdbus-codegen fixed to build on non-Unix

https://bugzilla.gnome.org/show_bug.cgi?id=655148#c6

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

gtester: small Windows fix

This code was unconditionally present in the gtester Makefile:

test-nonrecursive: ${TEST_PROGS}

On Windows, our testcases are compiled with a .exe suffix.  That means
that if we had 'foo' in TEST_PROGS, running "make check" would depend on
'foo' (not foo.exe) being compiled.

We could bring the EXEEXT in here to fix that up, but gtester doesn't
work on Windows at all, so better to just disable it in that case.

GDateTime test: 1970 doesn't exist on Windows

It is not safe to call the system library mktime() function on Windows
with a date in the 70s.  Use 1990 instead.

g_format_size: fix on Windows

The long format that displays the exact number of bytes with separators
(ie: "123,456,789 bytes") uses the ' format modifier, which is
unsupported on Windows.  Disable that for now, until we come up with a
better solution.

g_format_size: avoid silly GString use

We were using a GString for the purpose of doing a single printf().  Do
g_strdup_printf() instead.

Update glib-zip.in

glocalfile: don't bother with fstype on win32

Fixes the broken build on Windows.

Merge branch 'gwakeup'

GWakeup: fix Windows build breakage

...from the attempt to make it private.

.gitignore the gwakeup-fallback testcase

Remove g_wakeup_* from glib.symbols

GWakeup: make it private API

Colin requests that we keep this one private for now.

Include it at each point of use (libglib, libgio, tests).

GWakeup: test fallback case

We need to test the case of eventfd in the libc but no kernel support.

In order to do that, we add a separate compile of the GWakeup testcase
that interposes an 'eventfd' symbol that always returns -1 with errno
set.  That will trigger the fallback case.

GWakeup: add signal safety note

Note that g_wakeup_signal() is safe to call from a UNIX signal handler
(since this is a likely place to want to call it from).

More fixes for non-Unix builds

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

Fix the build on non-Unix

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

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

GWakeup .gitignore fixes

gtk-doc GWakeup

GCancellable: port to GWakeup

port GMainContext to GWakeup

add a couple of testcases for GWakeup

Add GWakeup

GWakeup is a utility class to handle the cross-thread signalling needs
of GMainContext and GCancellable.  It may find some other users as well.

The desire here is to properly hide the implementation details in a
module which can be properly unit tested and used in GMainContext and
GCancellable without a rats nest of #ifdef.

g_unix_open_pipe: fix some bugs

Fix some bugs in the fallback case of g_unix_open_pipe:

  - close both halves of the pipe on error (not just one)

  - set the cloexec flag on both halves of the pipe (instead of settings
    it twice on one half)

Windows (mingw32) .gitignore additions

Docs: Fix tiny typo.

Update Windows READMEs

-Tell people about that libFFI is now needed.
-Tell people not to compile GLib in paths containing spaces.

gio/tests/socket: add some basic IPv4 and IPv6 tests

GSocket: fix connected state after async connect

socket->priv->connected was only being set if g_socket_connect()
succeeded right away; in the case where it returns G_IO_ERROR_PENDING,
it never got set. Fix that by having g_socket_check_connect_result()
set it on success.

update .gitignore

Explicitly unset the session bus address for the peer tests

Otherwise, we may run into trouble as opening a peer-to-peer
connection uses a socket client, which uses a proxy resolver
which may end up using gsettings, whose dconf backend may end
up using the session bus to talk to dconfd...

Properly bring up a session bus for application tests

As pointed out in bug 644601, session_bus_up() requires
us to set up environment variables for things to work.

Fix a typo

Updated Russian translation

[l10n] Updated Italian translation

GDesktopAppInfo: Add g_desktop_app_info_get_show_in()

Necessary for rebasing gnome-menus on top of GDesktopAppInfo.

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

glib.symbols: Update for expected ABI changes

glib-2.0.pc: Add -lrt to private libraries to assist static linking

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

Revert addition of g_key_file_has_key_full

Per IRC discussion, we can just ask bindings to use
g_key_file_get_value() to test for the existence of a key.

I left the "fixed" code in the source tree as static because it makes
more sense to me.

Don't use deprecated G_UNICODE_COMBINING_MARK