Piteball/android_device_samsung_msm8226-common
1
0
Fork 0
mirror of https://github.com/S3NEO/android_device_samsung_msm8226-common.git synced 2024-11-06 21:55:45 +00:00
Code Issues Projects Releases Packages Wiki Activity

msm8226-common: gps: Fix for buffer overrun crash at copying nmea string

Browse source 
Add zero clearing of allocated nmea buffer to ensure
the nmea string is null terminated.

Change-Id: I78e8b03bc7d0f9b69e1a51356da379a57a647cad
CRs-Fixed: 2041933
This commit is contained in:
Katz Yamada 2019-11-03 11:45:02 +02:00 committed by RomanDesigner
parent 7f16443813
commit 52a845523e
No known key found for this signature in database
GPG key ID: 6757965E5D54CB61
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
gps/loc_api/libloc_api_50001/loc_eng.cpp
View file

@ -891,9 +891,9 @@ inline void LocEngReportStatus::log() const {
// case LOC_ENG_MSG_REPORT_NMEA: // case LOC_ENG_MSG_REPORT_NMEA:
LocEngReportNmea::LocEngReportNmea(void* locEng, LocEngReportNmea::LocEngReportNmea(void* locEng,
const char* data, int len) : const char* data, int len) :
LocMsg(), mLocEng(locEng), mNmea(new char[len]), mLen(len) LocMsg(), mLocEng(locEng), mNmea(new char[len+1]), mLen(len)
{ {
memcpy((void*)mNmea, (void*)data, len); strlcpy(mNmea, data, len+1);
locallog(); locallog();
} }
void LocEngReportNmea::proc() const { void LocEngReportNmea::proc() const {