uknetdev_output: Fix loop iterator

Setting q to p->next leads to an infinite loop if p has at least two
segments. The correct way is to iterate over q so we actually progress
through the list.

Signed-off-by: Florian Schmidt <florian.schmidt@neclab.eu>
Reviewed-by: Simon Kuenzer <simon.kuenzer@neclab.eu>

diff --git a/uknetdev.c b/uknetdev.c
index 34394fc5531703bfb802d3259ac1eb9bc3b28d90..47d2fb19d7133b3e565735e49ff2019b13df8998 100644 (file)
--- a/uknetdev.c
+++ b/uknetdev.c
@@ -142,7 +142,7 @@ static err_t uknetdev_output(struct netif *nf, struct pbuf *p)
         *       yet. As long as we do not have this, we have to copy.
         */
        wpos = nb->data;
-       for (q = p; q != NULL; q = p->next) {
+       for (q = p; q != NULL; q = q->next) {
                memcpy(wpos, q->payload, q->len);
                wpos += q->len;
        }