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

Commit afb0e3bb authored by Hansong Zhang's avatar Hansong Zhang
Browse files

GattServcer: Check invalid offset 

Test: manual
Bug: 143231677
Change-Id: I97e2c3ae15fccc482d07d8d621c455cc74900cfd
Merged-In: I0ca22e7c60292d61c758120c1cd67f6e6edd8ae8
parent 964cafb8

system/service/gatt_server.cc

+7 −0
Original line number Diff line number Diff line
@@ -18,6 +18,7 @@ 


#include <base/logging.h>



#include "osi/include/log.h"

#include "service/logging_helpers.h"

#include "stack/include/bt_types.h"
@@ -116,6 +117,12 @@ bool GattServer::SendResponse(const std::string& device_address, int request_id, 
    return false;

  }



  if (offset < 0) {

    android_errorWriteLog(0x534e4554, "143231677");

    LOG(ERROR) << "Offset is less than 0 offset: " << offset;

    return false;

  }



  if (value.size() + offset > BTGATT_MAX_ATTR_LEN) {

    LOG(ERROR) << "Value is too large";

    return false;