[vlc-devel] commit: DBus: remove unused macro, check memory allocation ( Rafaël Carré )

[git version control]

vlc | branch: master | Rafaël Carré <funman at videolan.org> | Thu Mar 27 01:10:58 2008 +0100| [fa9d4d0534ded4bbe9d970f84b45d88e7f74d124]

DBus: remove unused macro, check memory allocation

> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=fa9d4d0534ded4bbe9d970f84b45d88e7f74d124
---

 modules/control/dbus.c |   18 ++++++++++++++----
 modules/control/dbus.h |    1 -
 2 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/modules/control/dbus.c b/modules/control/dbus.c
index ec2914d..514f2c1 100644
--- a/modules/control/dbus.c
+++ b/modules/control/dbus.c
@@ -136,10 +136,20 @@ DBUS_METHOD( MprisVersion )
     dbus_uint16_t i_minor = VLC_MPRIS_VERSION_MINOR;
     DBusMessageIter version;
 
-    dbus_message_iter_open_container( &args, DBUS_TYPE_STRUCT, NULL, &version );
-    dbus_message_iter_append_basic( &version, DBUS_TYPE_UINT16, &i_major );
-    dbus_message_iter_append_basic( &version, DBUS_TYPE_UINT16, &i_minor );
-    dbus_message_iter_close_container( &args, &version );
+    if( !dbus_message_iter_open_container( &args, DBUS_TYPE_STRUCT, NULL,
+            &version ) )
+        return DBUS_HANDLER_RESULT_NEED_MEMORY;
+
+    if( !dbus_message_iter_append_basic( &version, DBUS_TYPE_UINT16,
+            &i_major ) )
+        return DBUS_HANDLER_RESULT_NEED_MEMORY;
+
+    if( !dbus_message_iter_append_basic( &version, DBUS_TYPE_UINT16,
+            &i_minor ) )
+        return DBUS_HANDLER_RESULT_NEED_MEMORY;
+
+    if( !dbus_message_iter_close_container( &args, &version ) )
+        return DBUS_HANDLER_RESULT_NEED_MEMORY;
     REPLY_SEND;
 }
 
diff --git a/modules/control/dbus.h b/modules/control/dbus.h
index c9057d7..5956f0d 100644
--- a/modules/control/dbus.h
+++ b/modules/control/dbus.h
@@ -80,7 +80,6 @@
 #define ADD_BOOL( b ) DBUS_ADD( DBUS_TYPE_BOOLEAN, b )
 #define ADD_INT32( i ) DBUS_ADD( DBUS_TYPE_INT32, i )
 #define ADD_BYTE( b ) DBUS_ADD( DBUS_TYPE_BYTE, b )
-#define ADD_UINT16( i ) DBUS_ADD( DBUS_TYPE_UINT16, i )
 
 /* XML data to answer org.freedesktop.DBus.Introspectable.Introspect requests */