Here the variable agent never be NULL, so no need to check it against NULL. --- src/adapter.c | 3 --- 1 files changed, 0 insertions(+), 3 deletions(-) diff --git a/src/adapter.c b/src/adapter.c index 18dd5b6..16d379c 100644 --- a/src/adapter.c +++ b/src/adapter.c @@ -1618,9 +1618,6 @@ static DBusMessage *register_agent(DBusConnection *conn, DBusMessage *msg, agent = agent_create(adapter, name, path, cap, (agent_remove_cb) agent_removed, adapter); - if (!agent) - return btd_error_failed(msg, "Failed to create a new agent"); - adapter->agent = agent; DBG("Agent registered for hci%d at %s:%s", adapter->dev_id, name, -- 1.7.4.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