Bluetooth: Refactor loop in l2cap_retransmit_one_frame
authorSzymon Janc <szymon.janc@tieto.com>
Wed, 16 Nov 2011 08:32:20 +0000 (09:32 +0100)
committerGustavo F. Padovan <padovan@profusion.mobi>
Wed, 16 Nov 2011 20:29:28 +0000 (18:29 -0200)
This make it easier to see what is the real reason for loop to exit.
skb_queue_next return valid skb or garbage, not NULL.

Signed-off-by: Szymon Janc <szymon.janc@tieto.com>
Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
net/bluetooth/l2cap_core.c

index bd65b3e8a1df4d72e8e3b6ebca7468f91368c9ce..26925a8f8ead311a9e34808d687a93e1d9f65318 100644 (file)
@@ -1318,14 +1318,12 @@ static void l2cap_retransmit_one_frame(struct l2cap_chan *chan, u16 tx_seq)
        if (!skb)
                return;
 
-       do {
-               if (bt_cb(skb)->tx_seq == tx_seq)
-                       break;
-
+       while (bt_cb(skb)->tx_seq != tx_seq) {
                if (skb_queue_is_last(&chan->tx_q, skb))
                        return;
 
-       } while ((skb = skb_queue_next(&chan->tx_q, skb)));
+               skb = skb_queue_next(&chan->tx_q, skb);
+       }
 
        if (chan->remote_max_tx &&
                        bt_cb(skb)->retries == chan->remote_max_tx) {