GitLab

The shared-mime-info that we use to detect mime-type of the contents we
are serving reports "application/xml" for XML, which seems to break
compatibility with some crappy XML parsers (See
https://bugzilla.gnome.org/show_bug.cgi?id=626196#c41). According to UPnP
Device Architecture 1.1, we MUST use the following value:

text/xml; charset="utf-8"

Instead of calling g_hash_table_foreach_remove() with a callback
function that always returns TRUE, just call g_hash_table_remove_all().

Read and write floats and doubles using the '.' as decimal
separator, independent of the locale that is currently in use.

The code used to use atoi() to parse unsigned integer values
from XML strings. Use the proper functions instead.

Avoid 64bit problems if glib hands out GTypes > INT_MAX

Don't use deprecated glib, gobject and libsoup APIs.

Based on a patch by Nader Morshed.

soup_message_headers_get() -> soup_message_headers_get_one().

soup_message_headers_get_content_length() only returns non-0 if the
encoding in use is 'Content-Length'. Although I'm not sure we are
correctly handling 'chunked' encoding but at least now we pass the DLNA
CTT test 7.2.5.7.

Signed-off-by: Sven Neumann <s.neumann@raumfeld.com>

GUPnPService should remove all handlers installed on the
shared SoupServer instance when it is disposed.
Signed-off-by: Sven Neumann <s.neumann@raumfeld.com>

Signed-off-by: Sven Neumann <s.neumann@raumfeld.com>

DLNA requirement 7.2.22.1 mandates devices to enforce subscription timeout
of 300 seconds.

Since we are using the collected value in place and unsetting it
directly after, there is no need to actually copy contents to it.
This saves some needless string copies.
Signed-off-by: Sven Neumann <s.neumann@raumfeld.com>

Although we stopped doing this for very good reasons in commit 7ebd4453,
we are now re-introducing this to be compatible with server-side
implementations using older GUPnP.

Signed-off-by: Sven Neumann <s.neumann@raumfeld.com>

http://bugzilla.openedhand.com/show_bug.cgi?id=2084

http://bugzilla.openedhand.com/show_bug.cgi?id=2086

Fixes `make distcheck`.

On some machines (particularly inside scratchbox) this field would
break GCC's strict aliasing rules with -O2 or higher by casting
from a SubscriptionData to a gpointer.  Since we don't strictly
need it typed as a SubscriptionData and we can let GCC to strict
aliasing with it as a gpointer we can safely type it as just a
gpointer and prevent warnings/errors.

Co-author & reviewer: Zeeshan Ali (Khattak) <zeeshanak@gnome.org>

Variants of gupnp_service_action_get/set that use lists rather than
varargs. This is mainly done to allow PyGI to deal with service actions.
gobject-introspection do not (yet) support vararg and on top of that PyGI
maintainer refuses to handle reference type 'in' parameters correctly
because the only way to do that is 'inefficient'.

Co-author and reviewer: Zeeshan Ali (Khattak) <zeeshanak@gnome.org>

Variant of gupnp_service_action_get_value that creates a new GValue and
returns that instead of taking an already initialized GValue and setting
it. This is mainly done to allow PyGI to deal with service actions. PyGI
maintainer refuses to handle reference type 'in' parameters correctly
because the only way to do that is 'inefficient'.

Co-author and reviewer: Zeeshan Ali (Khattak) <zeeshanak@gnome.org>

This is mainly done to allow PyGI to be able to call actions.
gobject-introspection do not (yet) support vararg and on top of that
PyGI maintainer refuses to handle reference type 'in' parameters
correctly because the only way to do that is 'inefficient'.

Co-author and reviewer: Zeeshan Ali <zeeshanak@gnome.org>