Both conn and buff are always passed as valid pointers. No need to double check them. --- sap/server.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/sap/server.c b/sap/server.c index 8612d0d..f66e23f 100644 --- a/sap/server.c +++ b/sap/server.c @@ -244,9 +244,6 @@ static int send_message(struct sap_connection *conn, void *buf, size_t size) GError *gerr = NULL; GIOStatus gstatus; - if (!conn || !buf) - return -EINVAL; - SAP_VDBG("conn %p, size %zu", conn, size); gstatus = g_io_channel_write_chars(conn->io, buf, size, &written, -- on behalf of ST-Ericsson -- 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