Merge pull request #10889 from ARMmbed/release-candidate

Release candidate for mbed-os-5.13.0-rc4
ARMmbed · Jun 25, 2019 · 7482462 · 7482462
2 parents 92a58df + 1533d24
commit 7482462
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 4 deletions.
diff --git a/features/cellular/framework/targets/QUECTEL/BG96/QUECTEL_BG96_CellularStack.cpp b/features/cellular/framework/targets/QUECTEL/BG96/QUECTEL_BG96_CellularStack.cpp
@@ -50,9 +50,13 @@ nsapi_error_t QUECTEL_BG96_CellularStack::socket_connect(nsapi_socket_t handle,
     CellularSocket *socket = (CellularSocket *)handle;
 
     int modem_connect_id = -1;
-    int request_connect_id = socket->id;
     int err = -1;
 
+    int request_connect_id = find_socket_index(socket);
+    // assert here as its a programming error if the socket container doesn't contain
+    // specified handle
+    MBED_ASSERT(request_connect_id != -1);
+
     _at.lock();
     if (socket->proto == NSAPI_TCP) {
         _at.cmd_start("AT+QIOPEN=");

diff --git a/features/cellular/framework/targets/QUECTEL/M26/QUECTEL_M26_CellularStack.cpp b/features/cellular/framework/targets/QUECTEL/M26/QUECTEL_M26_CellularStack.cpp
@@ -302,12 +302,14 @@ nsapi_error_t QUECTEL_M26_CellularStack::socket_connect(nsapi_socket_t handle, c
 {
     CellularSocket *socket = (CellularSocket *)handle;
 
-    MBED_ASSERT(socket->id != -1);
-
     int modem_connect_id = -1;
-    int request_connect_id = socket->id;
     int err = -1;
 
+    int request_connect_id = find_socket_index(socket);
+    // assert here as its a programming error if the socket container doesn't contain
+    // specified handle
+    MBED_ASSERT(request_connect_id != -1);
+
     _at.lock();
     if (socket->proto == NSAPI_TCP) {
         _at.cmd_start("AT+QIOPEN=");