ESP32S2: Socket.recv_into only took len-1 bytes

Neradoc · Neradoc · commit 6be2466d6fa1 · 2021-02-07T08:57:19.000+01:00
diff --git a/ports/esp32s2/common-hal/socketpool/Socket.c b/ports/esp32s2/common-hal/socketpool/Socket.c
@@ -256,7 +256,7 @@ mp_uint_t common_hal_socketpool_socket_recv_into(socketpool_socket_obj_t* self,
                 timed_out = supervisor_ticks_ms64() - start_ticks >= self->timeout_ms;
             }
             RUN_BACKGROUND_TASKS;
-            received = lwip_recv(self->num, (void*) buf, len - 1, 0);
+            received = lwip_recv(self->num, (void*) buf, len, 0);
 
             // In non-blocking mode, fail instead of looping
             if (received == -1 && self->timeout_ms == 0) {

Original file line number	Diff line number	Diff line change
`@@ -256,7 +256,7 @@ mp_uint_t common_hal_socketpool_socket_recv_into(socketpool_socket_obj_t* self,`
`256`	`256`	`timed_out = supervisor_ticks_ms64() - start_ticks >= self->timeout_ms;`
`257`	`257`	`}`
`258`	`258`	`RUN_BACKGROUND_TASKS;`
`259`		`- received = lwip_recv(self->num, (void*) buf, len - 1, 0);`
	`259`	`+ received = lwip_recv(self->num, (void*) buf, len, 0);`
`260`	`260`
`261`	`261`	`// In non-blocking mode, fail instead of looping`
`262`	`262`	`if (received == -1 && self->timeout_ms == 0) {`