From: Andrei Emeltchenko <andrei.emeltchenko@xxxxxxxxx>
Remove snprintf error check. 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 | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/obexd/client/map.c b/obexd/client/map.c
index 47afc31..520e492 100644
--- a/obexd/client/map.c
+++ b/obexd/client/map.c
@@ -447,8 +447,7 @@ static DBusMessage *map_msg_get(DBusConnection *connection,
return g_dbus_create_error(message,
ERROR_INTERFACE ".InvalidArguments", NULL);
- if (snprintf(handle, sizeof(handle), "%" PRIx64, msg->handle) < 0)
- goto fail;
+ snprintf(handle, sizeof(handle), "%" PRIx64, msg->handle);
transfer = obc_transfer_get("x-bt/message", handle, target_file, &err);
if (transfer == NULL)
@@ -746,8 +745,7 @@ static void set_status(const GDBusPropertyTable *property,
contents[0] = FILLER_BYTE;
- if (snprintf(handle, sizeof(handle), "%" PRIx64, msg->handle) < 0)
- goto fail;
+ snprintf(handle, sizeof(handle), "%" PRIx64, msg->handle);
transfer = obc_transfer_put("x-bt/messageStatus", handle, NULL,
contents, sizeof(contents), &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
