From 1539ff9dd6862f22cb4a1e760104e088aea70ad1 Mon Sep 17 00:00:00 2001 From: Szymon Janc Date: Tue, 22 Sep 2015 20:56:52 +0200 Subject: [PATCH] audio/avrcp: Remove not needed null check in Browsing handlers are always getting valid pdu pointer so there is no need to double check that. Also pdu pointer is already dereferenced before calling handler. --- profiles/audio/avrcp.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/profiles/audio/avrcp.c b/profiles/audio/avrcp.c index 53170ec95..24deac5fb 100644 --- a/profiles/audio/avrcp.c +++ b/profiles/audio/avrcp.c @@ -1946,7 +1946,7 @@ static void avrcp_handle_get_folder_items(struct avrcp *session, uint8_t scope; uint8_t status = AVRCP_STATUS_SUCCESS; - if (!pdu || ntohs(pdu->param_len) < 10) { + if (ntohs(pdu->param_len) < 10) { status = AVRCP_STATUS_INVALID_PARAM; goto failed; } -- 2.47.3