app: Use sm_at_host_get_current() instead of sock->pipe

For most cases, where we are going to send something to AT pipe,
the in_datamode() should be called with the current AT context,
not with the pipe reference where the socket was originally created.

This is because if DLC1 creates a sockets and If DLC2 is used
to AT#RECV, then in_datamode(sock->pipe)
would return wrong data, because sock->pipe refers to DLC1 not the
context where AT command is coming.

Signed-off-by: Seppo Takalo <seppo.takalo@nordicsemi.no>

Author: SeppoTakalo

app/src/sm_at_socket.c

@@ -246,7 +246,7 @@ static void auto_reception(struct sm_socket *sock)
 		LOG_ERR("auto_reception() error: %d", err);
 		return;
 	}
-	if (!in_datamode(sock->pipe)) {
+	if (!in_datamode(sm_at_host_get_current())) {
 		/* <CR><LF> after the data. */
 		sm_at_send_str("\r\n");
 	}
@@ -459,12 +459,8 @@ static void send_cb(const struct nrf_modem_sendcb_params *params)
 	sock->send_ntf.bytes_sent = params->bytes_sent;
 	atomic_set(&sock->send_ntf.ready, 1);
 
-	if (sock->pipe) {
-		/* Schedule work to execute in AT context */
-		sm_at_host_queue_idle_work(sock->pipe, &work);
-	} else {
-		k_work_submit_to_queue(&sm_work_q, &work);
-	}
+	/* Schedule work to execute in AT context */
+	sm_at_host_queue_idle_work(sock->pipe, &work);
 }
 
 static int set_so_send_cb(struct sm_socket *socket)
@@ -1052,7 +1048,7 @@ static int do_send(struct sm_socket *sock, const uint8_t *data, int len, int fla
 		sent += ret;
 	}
 
-	if (!in_datamode(sock->pipe)) {
+	if (!in_datamode(sm_at_host_get_current())) {
 		rsp_send("\r\n#XSEND: %d,%d,%d\r\n", sock->fd,
 			 send_ntf ? AT_SOCKET_SEND_RESULT_NW_ACK_URC
 				  : AT_SOCKET_SEND_RESULT_DEFAULT,
@@ -1116,7 +1112,7 @@ static int do_recv(struct sm_socket *sock, int timeout, int flags,
 	if (ret == 0) {
 		LOG_WRN("nrf_recv() return 0");
 	} else {
-		if (!in_datamode(sock->pipe)) {
+		if (!in_datamode(sm_at_host_get_current())) {
 			rsp_send("\r\n#XRECV: %d,%d,%d\r\n", sock->fd, mode, ret);
 		}
 
@@ -1188,7 +1184,7 @@ static int do_sendto(struct sm_socket *sock, const char *url, uint16_t port, con
 		LOG_ERR("Sent %u out of %u bytes. (%d)", sent, len, ret);
 	}
 
-	if (!in_datamode(sock->pipe)) {
+	if (!in_datamode(sm_at_host_get_current())) {
 		rsp_send("\r\n#XSENDTO: %d,%d,%d\r\n", sock->fd,
 			 send_ntf ? AT_SOCKET_SEND_RESULT_NW_ACK_URC
 				  : AT_SOCKET_SEND_RESULT_DEFAULT,
@@ -1228,7 +1224,7 @@ static int do_recvfrom(struct sm_socket *sock, int timeout, int flags,
 	if (ret == 0) {
 		LOG_WRN("nrf_recvfrom() return 0");
 	} else {
-		if (!in_datamode(sock->pipe)) {
+		if (!in_datamode(sm_at_host_get_current())) {
 			char peer_addr[NRF_INET6_ADDRSTRLEN] = {0};
 			uint16_t peer_port = 0;