code.ossystems Code Review - openembedded-core.git/blob

   1 From 7225bf8e8e9bee42d7d7d02ba754b9fb30a877b6 Mon Sep 17 00:00:00 2001
   2 From: Paul Eggleton <paul.eggleton@linux.intel.com>
   3 Date: Mon, 16 Apr 2012 19:15:35 +0100
   4 Subject: [PATCH] connman-gnome: fix segfault due to unchecked null value
   5
   6 If value is NULL here we should not pass it to g_value_get_boxed().
   7
   8 Upstream-Status: Submitted
   9
  10 Signed-off-by: Paul Eggleton <paul.eggleton@linux.intel.com>
  11 ---
  12  common/connman-dbus.c |    2 +-
  13  1 files changed, 1 insertions(+), 1 deletions(-)
  14
  15 diff --git a/common/connman-dbus.c b/common/connman-dbus.c
  16 index 33ac623..822fc98 100644
  17 --- a/common/connman-dbus.c
  18 +++ b/common/connman-dbus.c
  19 @@ -437,7 +437,7 @@ static void service_properties(DBusGProxy *proxy, GHashTable *hash,
  20         DBG("name %s type %d icon %s", name, type, icon);
  21
  22         value = g_hash_table_lookup(hash, "IPv4.Configuration");
  23 -       ipv4 = g_value_get_boxed (value);
  24 +       ipv4 = value ? g_value_get_boxed (value) : NULL;
  25
  26         if (!ipv4)
  27                 goto done;
  28 --
  29 1.7.5.4
  30