From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx>
In a case snprintf fails we have NULL dereference. Fixes clang warnings
below:
...
obexd/client/map.c:471:9: warning: Access to field 'message' results in
a dereference of a null pointer (loaded from variable 'err')
err->message);
^~~~~~~~~~~~
obexd/client/map.c:772:9: warning: Access to field 'message' results in
a dereference of a null pointer (loaded from variable 'err')
err->message);
^~~~~~~~~~~~
...
---
obexd/client/map.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/obexd/client/map.c b/obexd/client/map.c
index 47afc31..ed535e2 100644
--- a/obexd/client/map.c
+++ b/obexd/client/map.c
@@ -468,7 +468,7 @@ static DBusMessage *map_msg_get(DBusConnection *connection,
fail:
reply = g_dbus_create_error(message, ERROR_INTERFACE ".Failed", "%s",
- err->message);
+ err ? err->message : "");
g_error_free(err);
return reply;
}
@@ -769,7 +769,7 @@ static void set_status(const GDBusPropertyTable *property,
fail:
g_dbus_pending_property_error(id, ERROR_INTERFACE ".Failed", "%s",
- err->message);
+ err ? err->message : "");
g_error_free(err);
}
--
1.9.1
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
