rt2x00: Don't use unitialized rxdesc->size
authorMattias Nissler <mattias.nissler@gmx.de>
Sun, 9 Mar 2008 21:41:22 +0000 (22:41 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Thu, 13 Mar 2008 20:02:34 +0000 (16:02 -0400)
rxdesc->size is unitialized before the desriptor has been read.
Move the truncation of the sk buffer to the moment all variables
have been initialized.

Signed-off-by: Mattias Nissler <mattias.nissler@gmx.de>
Signed-off-by: Ivo van Doorn <IvDoorn@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
drivers/net/wireless/rt2x00/rt2500usb.c
drivers/net/wireless/rt2x00/rt73usb.c

index 86cd9a5eee2b349d5563119305d1e1f156180a72..559131fc6d53c7da4c3c668e3043424258d8031c 100644 (file)
@@ -1123,13 +1123,10 @@ static void rt2500usb_fill_rxdone(struct queue_entry *entry,
 
        /*
         * Copy descriptor to the available headroom inside the skbuffer.
-        * Remove the original copy by trimming the skbuffer.
         */
        skb_push(entry->skb, offset);
        memcpy(entry->skb->data, rxd, entry->queue->desc_size);
        rxd = (__le32 *)entry->skb->data;
-       skb_pull(entry->skb, offset);
-       skb_trim(entry->skb, rxdesc->size);
 
        /*
         * The descriptor is now aligned to 4 bytes and thus it is
@@ -1154,12 +1151,18 @@ static void rt2500usb_fill_rxdone(struct queue_entry *entry,
        rxdesc->size = rt2x00_get_field32(word0, RXD_W0_DATABYTE_COUNT);
        rxdesc->my_bss = !!rt2x00_get_field32(word0, RXD_W0_MY_BSS);
 
+       /*
+        * Adjust the skb memory window to the frame boundaries.
+        */
+       skb_pull(entry->skb, offset);
+       skb_trim(entry->skb, rxdesc->size);
+
        /*
         * Set descriptor and data pointer.
         */
        skbdesc->data = entry->skb->data;
        skbdesc->data_len = rxdesc->size;
-       skbdesc->desc = entry->skb->data - offset;
+       skbdesc->desc = rxd;
        skbdesc->desc_len = entry->queue->desc_size;
 }
 
index a48c82f09216fec9641fb5d0abf0370639bc1064..468039f1bff90e852ef3c1e3eb7139b986c76597 100644 (file)
@@ -1376,13 +1376,10 @@ static void rt73usb_fill_rxdone(struct queue_entry *entry,
 
        /*
         * Copy descriptor to the available headroom inside the skbuffer.
-        * Remove the original copy by pulling the skbuffer.
         */
        skb_push(entry->skb, offset);
        memcpy(entry->skb->data, rxd, entry->queue->desc_size);
        rxd = (__le32 *)entry->skb->data;
-       skb_pull(entry->skb, offset + entry->queue->desc_size);
-       skb_trim(entry->skb, rxdesc->size);
 
        /*
         * The descriptor is now aligned to 4 bytes and thus it is
@@ -1404,12 +1401,18 @@ static void rt73usb_fill_rxdone(struct queue_entry *entry,
        rxdesc->size = rt2x00_get_field32(word0, RXD_W0_DATABYTE_COUNT);
        rxdesc->my_bss = !!rt2x00_get_field32(word0, RXD_W0_MY_BSS);
 
+       /*
+        * Adjust the skb memory window to the frame boundaries.
+        */
+       skb_pull(entry->skb, offset + entry->queue->desc_size);
+       skb_trim(entry->skb, rxdesc->size);
+
        /*
         * Set descriptor and data pointer.
         */
        skbdesc->data = entry->skb->data;
        skbdesc->data_len = rxdesc->size;
-       skbdesc->desc = entry->skb->data - offset;
+       skbdesc->desc = rxd;
        skbdesc->desc_len = entry->queue->desc_size;
 }