Import upstream version 0.5.0

9 files changed, 78 insertions, 17 deletions

diff --git a/ChangeLog b/ChangeLog
index bd4b9c5..4e63087 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,53 @@
 # Generated by Makefile. Do not edit.
 
+2011-09-29  Ted Gould  <ted@gould.cx>
+
+	0.5.0
+
+2011-09-29  Ted Gould  <ted@gould.cx>
+
+	More NULL protection
+
+2011-09-23  Ted Gould  <ted@gould.cx>
+
+	Protecting against NULL private areas
+
+2011-09-27  Ted Gould  <ted@gould.cx>
+
+	Protect from a NULL class
+
+2011-09-23  Ted Gould  <ted@gould.cx>
+
+	Ensure the class is correct
+
+2011-09-27  Ted Gould  <ted@gould.cx>
+
+	Protect client_about_to_show() from abusers
+
+2011-09-23  Ted Gould  <ted@gould.cx>
+
+	Attaching bug
+
+2011-09-23  Ted Gould  <ted@gould.cx>
+
+	Protect an external function better
+
+2011-09-27  Ted Gould  <ted@gould.cx>
+
+	Protect from NULL privates
+
+2011-09-23  Ted Gould  <ted@gould.cx>
+
+	Protect against an odd crash.  Not sure how the private becomes 'NULL'
+
+2011-09-23  Ted Gould  <ted@gould.cx>
+
+	Fix ref leaks in proxy code
+
+2011-09-23  Michael Terry  <michael.terry@canonical.com>
+
+	unref added children in proxy code
+
 2011-09-19  Ted Gould  <ted@gould.cx>
 
 	0.4.94
diff --git a/configure b/configure
index baba9a7..edd0a86 100755
--- a/configure
+++ b/configure
@@ -1,6 +1,6 @@
 #! /bin/sh
 # Guess values for system-dependent variables and create Makefiles.
-# Generated by GNU Autoconf 2.68 for libdbusmenu 0.4.94.
+# Generated by GNU Autoconf 2.68 for libdbusmenu 0.5.0.
 #
 # Report bugs to <ted@canonical.com>.
 #
@@ -572,8 +572,8 @@ MAKEFLAGS=
 # Identity of this package.
 PACKAGE_NAME='libdbusmenu'
 PACKAGE_TARNAME='libdbusmenu'
-PACKAGE_VERSION='0.4.94'
-PACKAGE_STRING='libdbusmenu 0.4.94'
+PACKAGE_VERSION='0.5.0'
+PACKAGE_STRING='libdbusmenu 0.5.0'
 PACKAGE_BUGREPORT='ted@canonical.com'
 PACKAGE_URL=''
 
@@ -1429,7 +1429,7 @@ if test "$ac_init_help" = "long"; then
   # Omit some internal or obsolete options to make the list less imposing.
   # This message is too long to be a string in the A/UX 3.1 sh.
   cat <<_ACEOF
-\`configure' configures libdbusmenu 0.4.94 to adapt to many kinds of systems.
+\`configure' configures libdbusmenu 0.5.0 to adapt to many kinds of systems.
 
 Usage: $0 [OPTION]... [VAR=VALUE]...
 
@@ -1499,7 +1499,7 @@ fi
 
 if test -n "$ac_init_help"; then
   case $ac_init_help in
-     short | recursive ) echo "Configuration of libdbusmenu 0.4.94:";;
+     short | recursive ) echo "Configuration of libdbusmenu 0.5.0:";;
    esac
   cat <<\_ACEOF
 
@@ -1649,7 +1649,7 @@ fi
 test -n "$ac_init_help" && exit $ac_status
 if $ac_init_version; then
   cat <<\_ACEOF
-libdbusmenu configure 0.4.94
+libdbusmenu configure 0.5.0
 generated by GNU Autoconf 2.68
 
 Copyright (C) 2010 Free Software Foundation, Inc.
@@ -2020,7 +2020,7 @@ cat >config.log <<_ACEOF
 This file contains any messages produced by compilers while
 running configure, to aid debugging if configure makes a mistake.
 
-It was created by libdbusmenu $as_me 0.4.94, which was
+It was created by libdbusmenu $as_me 0.5.0, which was
 generated by GNU Autoconf 2.68.  Invocation command line was
 
   $ $0 $@
@@ -2841,7 +2841,7 @@ fi
 
 # Define the identity of the package.
  PACKAGE=libdbusmenu
- VERSION=0.4.94
+ VERSION=0.5.0
 
 
 # Some tools Automake needs.
@@ -14309,7 +14309,7 @@ fi
 ###########################
 
 LIBDBUSMENU_CURRENT=4
-LIBDBUSMENU_REVISION=4
+LIBDBUSMENU_REVISION=5
 LIBDBUSMENU_AGE=0
 
 
@@ -15659,7 +15659,7 @@ cat >>$CONFIG_STATUS <<\_ACEOF || ac_write_fail=1
 # report actual input values of CONFIG_FILES etc. instead of their
 # values after options handling.
 ac_log="
-This file was extended by libdbusmenu $as_me 0.4.94, which was
+This file was extended by libdbusmenu $as_me 0.5.0, which was
 generated by GNU Autoconf 2.68.  Invocation command line was
 
   CONFIG_FILES    = $CONFIG_FILES
@@ -15725,7 +15725,7 @@ _ACEOF
 cat >>$CONFIG_STATUS <<_ACEOF || ac_write_fail=1
 ac_cs_config="`$as_echo "$ac_configure_args" | sed 's/^ //; s/[\\""\`\$]/\\\\&/g'`"
 ac_cs_version="\\
-libdbusmenu config.status 0.4.94
+libdbusmenu config.status 0.5.0
 configured by $0, generated by GNU Autoconf 2.68,
   with options \\"\$ac_cs_config\\"
 
diff --git a/configure.ac b/configure.ac
index 6276327..f195f47 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1,11 +1,11 @@
 
-AC_INIT(libdbusmenu, 0.4.94, ted@canonical.com)
+AC_INIT(libdbusmenu, 0.5.0, ted@canonical.com)
 AC_COPYRIGHT([Copyright 2009,2010 Canonical])
 
 AC_PREREQ(2.62)
 
 AM_CONFIG_HEADER(config.h)
-AM_INIT_AUTOMAKE(libdbusmenu, 0.4.94, [-Wno-portability])
+AM_INIT_AUTOMAKE(libdbusmenu, 0.5.0, [-Wno-portability])
 
 AM_MAINTAINER_MODE
 
@@ -172,7 +172,7 @@ AC_PATH_PROG([XSLT_PROC], [xsltproc])
 ###########################
 
 LIBDBUSMENU_CURRENT=4
-LIBDBUSMENU_REVISION=4
+LIBDBUSMENU_REVISION=5
 LIBDBUSMENU_AGE=0
 
 AC_SUBST(LIBDBUSMENU_CURRENT)
diff --git a/docs/libdbusmenu-glib/reference/version.xml b/docs/libdbusmenu-glib/reference/version.xml
index 54c3984..8f0916f 100644
--- a/docs/libdbusmenu-glib/reference/version.xml
+++ b/docs/libdbusmenu-glib/reference/version.xml
@@ -1 +1 @@
-0.4.94
+0.5.0
diff --git a/docs/libdbusmenu-gtk/reference/version.xml b/docs/libdbusmenu-gtk/reference/version.xml
index 54c3984..8f0916f 100644
--- a/docs/libdbusmenu-gtk/reference/version.xml
+++ b/docs/libdbusmenu-gtk/reference/version.xml
@@ -1 +1 @@
-0.4.94
+0.5.0
diff --git a/libdbusmenu-glib/client.c b/libdbusmenu-glib/client.c
index 588c940..a7f4da5 100644
--- a/libdbusmenu-glib/client.c
+++ b/libdbusmenu-glib/client.c
@@ -1057,7 +1057,12 @@ menuproxy_build_cb (GObject * object, GAsyncResult * res, gpointer user_data)
 
 	/* If this wasn't cancelled, we should be good */
 	DbusmenuClient * client = DBUSMENU_CLIENT(user_data);
+	/* But let's check */
+	g_return_if_fail(client != NULL);
 	DbusmenuClientPrivate * priv = DBUSMENU_CLIENT_GET_PRIVATE(client);
+	g_return_if_fail(priv != NULL);
+
+
 	priv->menuproxy = proxy;
 
 	if (priv->menuproxy_cancel != NULL) {
@@ -1604,7 +1609,11 @@ about_to_show_cb (GObject * proxy, GAsyncResult * res, gpointer userdata)
 void
 dbusmenu_client_send_about_to_show(DbusmenuClient * client, gint id, void (*cb)(gpointer data), gpointer cb_data)
 {
+	g_return_if_fail(DBUSMENU_CLIENT(client));
+	g_return_if_fail(id > 0);
+
 	DbusmenuClientPrivate * priv = DBUSMENU_CLIENT_GET_PRIVATE(client);
+	g_return_if_fail(priv != NULL);
 
 	about_to_show_t * data = g_new0(about_to_show_t, 1);
 	data->client = client;
diff --git a/libdbusmenu-glib/menuitem-proxy.c b/libdbusmenu-glib/menuitem-proxy.c
index ae6a334..fdbd892 100644
--- a/libdbusmenu-glib/menuitem-proxy.c
+++ b/libdbusmenu-glib/menuitem-proxy.c
@@ -188,6 +188,7 @@ proxy_item_child_added (DbusmenuMenuitem * parent, DbusmenuMenuitem * child, gui
 	DbusmenuMenuitemProxy * pmi = DBUSMENU_MENUITEM_PROXY(user_data);
 	DbusmenuMenuitemProxy * child_pmi = dbusmenu_menuitem_proxy_new(child);
 	dbusmenu_menuitem_child_add_position(DBUSMENU_MENUITEM(pmi), DBUSMENU_MENUITEM(child_pmi), position);
+	g_object_unref (child_pmi);
 	return;
 }
 
@@ -283,6 +284,7 @@ add_menuitem (DbusmenuMenuitemProxy * pmi, DbusmenuMenuitem * mi)
 	for (child = children; child != NULL; child = g_list_next(child)) {
 		DbusmenuMenuitemProxy * child_pmi = dbusmenu_menuitem_proxy_new(DBUSMENU_MENUITEM(child->data));
 		dbusmenu_menuitem_child_append(DBUSMENU_MENUITEM(pmi), DBUSMENU_MENUITEM(child_pmi));
+		g_object_unref (child_pmi);
 	}
 
 	return;
diff --git a/libdbusmenu-glib/menuitem.c b/libdbusmenu-glib/menuitem.c
index 4e037ee..c9c7736 100644
--- a/libdbusmenu-glib/menuitem.c
+++ b/libdbusmenu-glib/menuitem.c
@@ -1730,7 +1730,7 @@ dbusmenu_menuitem_send_about_to_show (DbusmenuMenuitem * mi, void (*cb) (Dbusmen
 	#endif
 	DbusmenuMenuitemClass * class = DBUSMENU_MENUITEM_GET_CLASS(mi);
 
-	if (class->send_about_to_show != NULL) {
+	if (class != NULL && class->send_about_to_show != NULL) {
 		return class->send_about_to_show(mi, cb, cb_data);
 	} else if (cb != NULL) {
 		cb(mi, cb_data);
diff --git a/libdbusmenu-glib/server.c b/libdbusmenu-glib/server.c
index c7057df..9c085f7 100644
--- a/libdbusmenu-glib/server.c
+++ b/libdbusmenu-glib/server.c
@@ -1194,7 +1194,9 @@ error_quark (void)
 static void
 bus_get_layout (DbusmenuServer * server, GVariant * params, GDBusMethodInvocation * invocation)
 {
+	g_return_if_fail(DBUSMENU_IS_SERVER(server));
 	DbusmenuServerPrivate * priv = DBUSMENU_SERVER_GET_PRIVATE(server);
+	g_return_if_fail(priv != NULL);
 
 	/* Input */
 	gint32 parent;