Merge pull request #183 from us3r64/fix/socket-recv_into

dhalbert · web-flow · commit c69b6f289d7c · 2024-01-15T09:52:04.000-05:00
Fix double subtraction of bytes_read from num_to_read in recv_into
diff --git a/adafruit_esp32spi/adafruit_esp32spi_socket.py b/adafruit_esp32spi/adafruit_esp32spi_socket.py
@@ -121,7 +121,7 @@ def recv_into(self, buffer, nbytes: int = 0):
                 )
                 buffer[num_read : num_read + len(bytes_read)] = bytes_read
                 num_read += len(bytes_read)
-                num_to_read -= num_read
+                num_to_read -= len(bytes_read)
             elif num_read > 0:
                 # We got a message, but there are no more bytes to read, so we can stop.
                 break

Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ def recv_into(self, buffer, nbytes: int = 0):`
`121`	`121`	`)`
`122`	`122`	`buffer[num_read : num_read + len(bytes_read)] = bytes_read`
`123`	`123`	`num_read += len(bytes_read)`
`124`		`- num_to_read -= num_read`
	`124`	`+ num_to_read -= len(bytes_read)`
`125`	`125`	`elif num_read > 0:`
`126`	`126`	`# We got a message, but there are no more bytes to read, so we can stop.`
`127`	`127`	`break`