Skip to content

Commit a713358

Browse files
committed
Fix the fault exception issue in socket_recvfrom with the null pointer parameter as SocketAddress *addr.
1 parent 9d28c3f commit a713358

File tree

1 file changed

+12
-2
lines changed

1 file changed

+12
-2
lines changed

features/cellular/framework/AT/AT_CellularStack.cpp

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -307,7 +307,12 @@ nsapi_size_or_error_t AT_CellularStack::socket_recvfrom(nsapi_socket_t handle, S
307307

308308
_at.unlock();
309309
if (ret_val != NSAPI_ERROR_OK) {
310-
tr_error("Socket %d create %s error %d", find_socket_index(socket), addr->get_ip_address(), ret_val);
310+
if(addr){
311+
tr_error("Socket %d create %s error %d", find_socket_index(socket), addr->get_ip_address(), ret_val);
312+
}
313+
else{
314+
tr_error("Socket %d create error %d", find_socket_index(socket), ret_val);
315+
}
311316
return ret_val;
312317
}
313318
}
@@ -319,7 +324,12 @@ nsapi_size_or_error_t AT_CellularStack::socket_recvfrom(nsapi_socket_t handle, S
319324
_at.unlock();
320325

321326
if (ret_val >= 0) {
322-
tr_info("Socket %d recv %d bytes from %s port %d", find_socket_index(socket), ret_val, addr->get_ip_address(), addr->get_port());
327+
if(addr){
328+
tr_info("Socket %d recv %d bytes from %s port %d", find_socket_index(socket), ret_val, addr->get_ip_address(), addr->get_port());
329+
}
330+
else{
331+
tr_info("Socket %d recv %d bytes", find_socket_index(socket), ret_val);
332+
}
323333
} else if (ret_val != NSAPI_ERROR_WOULD_BLOCK) {
324334
tr_error("Socket %d recv error %d", find_socket_index(socket), ret_val);
325335
}

0 commit comments

Comments
 (0)