diff options
| author | Al Viro <viro@ftp.linux.org.uk> | 2007-12-21 06:21:03 +0000 |
|---|---|---|
| committer | Jeff Garzik <jeff@garzik.org> | 2007-12-22 22:53:06 -0500 |
| commit | 8cc085c7aceb78d26d0a5355e111b2330f089161 (patch) | |
| tree | 3754b076639725dbb1d324d74d155f091b3f889d /drivers/net/typhoon.c | |
| parent | 8a5ed9efe661dd42bc140e522c2635e0d7b26141 (diff) | |
| download | lwn-8cc085c7aceb78d26d0a5355e111b2330f089161.tar.gz lwn-8cc085c7aceb78d26d0a5355e111b2330f089161.zip | |
typhoon: memory corruptor on big-endian if TSO is enabled
txlo_dma_addr should be host-endian; we pass it to typhoon_tso_fill(),
which does arithmetics on it, converts to l-e and passes it to card.
Unfortunately, we forgot le32_to_cpu() when initializing it from
face->txLoAddr, which sits in shared memory and is little-endian.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/net/typhoon.c')
| -rw-r--r-- | drivers/net/typhoon.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/net/typhoon.c b/drivers/net/typhoon.c index 501abe379047..2550d4cf9609 100644 --- a/drivers/net/typhoon.c +++ b/drivers/net/typhoon.c @@ -1320,7 +1320,7 @@ typhoon_init_interface(struct typhoon *tp) tp->txLoRing.writeRegister = TYPHOON_REG_TX_LO_READY; tp->txHiRing.writeRegister = TYPHOON_REG_TX_HI_READY; - tp->txlo_dma_addr = iface->txLoAddr; + tp->txlo_dma_addr = le32_to_cpu(iface->txLoAddr); tp->card_state = Sleeping; smp_wmb(); |