rt2x00: Fix skbdesc->data_len initialization
authorIvo van Doorn <ivdoorn@gmail.com>
Sun, 10 Feb 2008 21:51:21 +0000 (22:51 +0100)
committerJohn W. Linville <linville@tuxdriver.com>
Fri, 29 Feb 2008 20:37:20 +0000 (15:37 -0500)
skbdesc->data_len was not initialized correctly
in rt2x00pci, rt2x00usb, rt2500usb and rt73usb.

The value was set to queue->data_size which
means that the incorrect frame size was pased
to the upper layers.

Correctly base the value on either the skb->len,
or the rx frame size passed to the driver by the
device.

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/rt2x00pci.c
drivers/net/wireless/rt2x00/rt2x00usb.c
drivers/net/wireless/rt2x00/rt73usb.c

index ab6e3f7..f9d6f53 100644 (file)
@@ -1149,7 +1149,7 @@ static void rt2500usb_fill_rxdone(struct queue_entry *entry,
         * Set descriptor pointer.
         */
        skbdesc->data = entry->skb->data;
-       skbdesc->data_len = entry->queue->data_size;
+       skbdesc->data_len = rxdesc->size;
        skbdesc->desc = entry->skb->data + rxdesc->size;
        skbdesc->desc_len = entry->queue->desc_size;
 
index b8c6a8c..8f88ca2 100644 (file)
@@ -63,7 +63,7 @@ int rt2x00pci_write_tx_data(struct rt2x00_dev *rt2x00dev,
        skbdesc = get_skb_frame_desc(skb);
        memset(skbdesc, 0, sizeof(*skbdesc));
        skbdesc->data = skb->data;
-       skbdesc->data_len = queue->data_size;
+       skbdesc->data_len = skb->len;
        skbdesc->desc = priv_tx->desc;
        skbdesc->desc_len = queue->desc_size;
        skbdesc->entry = entry;
@@ -131,7 +131,7 @@ void rt2x00pci_rxdone(struct rt2x00_dev *rt2x00dev)
                skbdesc = get_skb_frame_desc(entry->skb);
                memset(skbdesc, 0, sizeof(*skbdesc));
                skbdesc->data = entry->skb->data;
-               skbdesc->data_len = queue->data_size;
+               skbdesc->data_len = entry->skb->len;
                skbdesc->desc = priv_rx->desc;
                skbdesc->desc_len = queue->desc_size;
                skbdesc->entry = entry;
index 4219057..450043b 100644 (file)
@@ -254,7 +254,7 @@ int rt2x00usb_write_tx_data(struct rt2x00_dev *rt2x00dev,
        skbdesc = get_skb_frame_desc(skb);
        memset(skbdesc, 0, sizeof(*skbdesc));
        skbdesc->data = skb->data + queue->desc_size;
-       skbdesc->data_len = queue->data_size;
+       skbdesc->data_len = skb->len - queue->desc_size;
        skbdesc->desc = skb->data;
        skbdesc->desc_len = queue->desc_size;
        skbdesc->entry = entry;
index d2096d2..5f3f74c 100644 (file)
@@ -1399,7 +1399,7 @@ static void rt73usb_fill_rxdone(struct queue_entry *entry,
         * Set descriptor and data pointer.
         */
        skbdesc->data = entry->skb->data + entry->queue->desc_size;
-       skbdesc->data_len = entry->queue->data_size;
+       skbdesc->data_len = rxdesc->size;
        skbdesc->desc = entry->skb->data;
        skbdesc->desc_len = entry->queue->desc_size;