Commit 9eea85f

authored

Check if m_pServerCallbacks is not null before calling method. (#5603)

Fixes: #5573 To reproduce: 1. Run any sample code that starts a BLE server, and does not call `setCallbacks`. 2. Connect to the device using the "LightBlue" app on iOS. 3. Observe crash shown in the issue linked above.

1 parent 24b76cb commit 9eea85fCopy full SHA for 9eea85f

File tree

1 file changed

-1

lines changed

libraries/BLE/src
- BLEServer.cpp

1 file changed

-1

lines changed

`‎libraries/BLE/src/BLEServer.cpp`

Lines changed: 3 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -157,7 +157,9 @@ void BLEServer::handleGATTServerEvent(esp_gatts_cb_event_t event, esp_gatt_if_t`
`157`	`157`
`158`	`158`	`case ESP_GATTS_MTU_EVT:`
`159`	`159`	`updatePeerMTU(param->mtu.conn_id, param->mtu.mtu);`
`160`		`- m_pServerCallbacks->onMtuChanged(this, param);`
	`160`	`+ if (m_pServerCallbacks != nullptr) {`
	`161`	`+ m_pServerCallbacks->onMtuChanged(this, param);`
	`162`	`+ }`
`161`	`163`	`break;`
`162`	`164`
`163`	`165`	`// ESP_GATTS_CONNECT_EVT`

0 commit comments

Comments

(0)

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit 9eea85f

File tree

1 file changed

1 file changed

`‎libraries/BLE/src/BLEServer.cpp`

0 commit comments