<div dir="ltr"><div>Looks good to me.</div><div><br></div><div>Acked-by: David Hendricks <<a href="mailto:david.hendricks@gmail.com">david.hendricks@gmail.com</a>></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 4, 2016 at 4:37 AM, Nico Huber <span dir="ltr"><<a href="mailto:nico.huber@secunet.com" target="_blank">nico.huber@secunet.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">We didn't check the total number of queued transfers in the inner most<br>
loop. Up to DEDIPROG_ASYNC_TRANSFERS - 1 invalid transfers could be<br>
queued therefore. So add another check on the total number.<br>
<br>
Signed-off-by: Nico Huber <<a href="mailto:nico.huber@secunet.com">nico.huber@secunet.com</a>><br>
---<br>
 dediprog.c | 4 +++-<br>
 1 file changed, 3 insertions(+), 1 deletion(-)<br>
<br>
diff --git a/dediprog.c b/dediprog.c<br>
index b7276e5..6f82772 100644<br>
--- a/dediprog.c<br>
+++ b/dediprog.c<br>
@@ -462,7 +462,9 @@ static int dediprog_spi_bulk_read(struct flashctx *flash, uint8_t *buf, unsigned<br>
<br>
        /* Now transfer requested chunks using libusb's asynchronous interface. */<br>
        while (!status.error && (status.queued_idx < count)) {<br>
-               while ((status.queued_idx - status.finished_idx) < DEDIPROG_ASYNC_TRANSFERS) {<br>
+               while ((status.queued_idx < count) &&<br>
+                      (status.queued_idx - status.finished_idx) < DEDIPROG_ASYNC_TRANSFERS)<br>
+               {<br>
                        transfer = transfers[status.queued_idx % DEDIPROG_ASYNC_TRANSFERS];<br>
                        libusb_fill_bulk_transfer(transfer, dediprog_handle, 0x80 | dediprog_in_endpoint,<br>
                                        (unsigned char *)buf + status.queued_idx * chunksize, chunksize,<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.7.0<br>
<br>
<br>
_______________________________________________<br>
flashrom mailing list<br>
<a href="mailto:flashrom@flashrom.org">flashrom@flashrom.org</a><br>
<a href="https://www.flashrom.org/mailman/listinfo/flashrom" rel="noreferrer" target="_blank">https://www.flashrom.org/mailman/listinfo/flashrom</a><br>
</font></span></blockquote></div><br></div>