Fixed bug that checked the wrong buffer size when inserting UUID information into an advertisement payload.

AGlass0fMilk · AGlass0fMilk · commit 4598a28cb7a6 · 2019-05-22T15:56:14.000-04:00
diff --git a/features/FEATURE_BLE/source/gap/AdvertisingDataBuilder.cpp b/features/FEATURE_BLE/source/gap/AdvertisingDataBuilder.cpp
@@ -498,7 +498,7 @@ ble_error_t AdvertisingDataBuilder::setUUIDData(
     size_t old_size = getFieldSize(shortType) + getFieldSize(longType);
 
     /* if we can't fit the new data do not proceed */
-    if (new_size > data.size() - (_payload_length - old_size)) {
+    if (new_size > _buffer.size() - (_payload_length - old_size)) {
         return BLE_ERROR_BUFFER_OVERFLOW;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -498,7 +498,7 @@ ble_error_t AdvertisingDataBuilder::setUUIDData(`
`498`	`498`	`size_t old_size = getFieldSize(shortType) + getFieldSize(longType);`
`499`	`499`
`500`	`500`	`/* if we can't fit the new data do not proceed */`
`501`		`- if (new_size > data.size() - (_payload_length - old_size)) {`
	`501`	`+ if (new_size > _buffer.size() - (_payload_length - old_size)) {`
`502`	`502`	`return BLE_ERROR_BUFFER_OVERFLOW;`
`503`	`503`	`}`
`504`	`504`