Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 91f3d4b8 authored by Andre Eisenbach's avatar Andre Eisenbach Committed by Myles Watson
Browse files

SDP: Bounds check 'id' parameter for free_sdp_slot() 

Merged-In: I34e8296ec7ec6b4ffbe1fa0452754f2a421e6ec7

Test: manual
Fixes: 37502513
Change-Id: I34e8296ec7ec6b4ffbe1fa0452754f2a421e6ec7
(cherry picked from commit b413f1b1365af4273647727e497848f95312d0ec)
parent 8431a7b5

system/btif/src/btif_sdp_server.c

+2 −1
Original line number Original line Diff line number Diff line
@@ -215,7 +215,8 @@ static int alloc_sdp_slot(bluetooth_sdp_record* in_record) { 
static int free_sdp_slot(int id) {

static int free_sdp_slot(int id) {

    int handle = -1;

    int handle = -1;

    bluetooth_sdp_record* record = NULL;

    bluetooth_sdp_record* record = NULL;

    if(id >= MAX_SDP_SLOTS) {

    if(id < 0 || id >= MAX_SDP_SLOTS) {

        android_errorWriteLog(0x534e4554, "37502513");

        APPL_TRACE_ERROR("%s() failed - id %d is invalid", __func__, id);

        APPL_TRACE_ERROR("%s() failed - id %d is invalid", __func__, id);

        return handle;

        return handle;

    }

    }