issue: 4398221 Fix connection closure on SYN_RCVD

When closing a connection in SYN_RCVD state, we should transition
directly to CLOSED without sending FIN or RST packets, as specified
in the RFC. This patch properly handles this case by immediately
changing the state to CLOSED instead of going through the normal
shutdown procedure.

This prevents unnecessary control packets when a connection receives
a SYN and is immediately closed without exchanging data.

Signed-off-by: Tomer Cabouly <[email protected]>

Author: tomerdbz

src/core/lwip/tcp.c

@@ -258,8 +258,15 @@ err_t tcp_close(struct tcp_pcb *pcb)
         /* Set a flag not to receive any more data... */
         pcb->flags |= TF_RXCLOSED;
     }
-    /* ... and close */
-    return tcp_close_shutdown(pcb, 1);
+
+    if (get_tcp_state(pcb) == SYN_RCVD) {
+        // according to the RFC, in case we get a SYN and no more data
+        // we should just close w/o FIN or RST
+        set_tcp_state(pcb, CLOSED);
+        return ERR_OK;
+    } else {
+        return tcp_close_shutdown(pcb, 1);
+    }
 }
 
 /**