netdev - Re: Corrupt Network Packets -- 3.17-rc5/rc6

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <201409221830.s8MIUbWS002990@clem.clem-digital.net>
Date:	Mon, 22 Sep 2014 14:30:36 -0400 (EDT)
From:	Pete Clements <clem@...m.clem-digital.net>
To:	davem@...emloft.net (David Miller)
Cc:	clem@...m.clem-digital.net, xiyou.wangcong@...il.com,
	linux-kernel@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: Corrupt Network Packets -- 3.17-rc5/rc6


Thanks. Does not patch. Code looks not to be in the rc6 that Linus
put out. Guess there is more to come.  (This is what I believe is
the segment in rc6 -- at line 2179.)

	vp->tx_ring[entry].frag[i+1].addr =
		cpu_to_le32(skb_frag_dma_map(
		&VORTEX_PCI(vp)->dev,
		frag,
		frag->page_offset, frag->size, DMA_TO_DEVICE));
--
Pete Clements 

Quoting David Miller
  > From: Pete Clements <clem@...m.clem-digital.net>
  > Date: Mon, 22 Sep 2014 13:24:59 -0400 (EDT)
  > 
  > > Quoting Cong Wang
  > >   > <clem@...m.clem-digital.net> wrote:
  > >   > > Heads up fyi: system is i386 UP.
  > >   > >
  > >   > > Started getting some pop3 reception errors from the above system.
  > >   > > Investigating the problem discovered scp of the mailbox failed
  > >   > > with "Corrupted MAC on input"
  > >   > >
  > >   > > Problem started with rc5, still in rc6. Running rc4 with no
  > >   > > problem.
  > >   > >
  > >   > > Sorry for lack of better definition.
  > >   > 
  > >   > 
  > >   > Which driver are you using?
  > >   > 
  > > Here's the output from boot log:
  > > 
  > >  PCI: setting IRQ 11 as level-triggered
  > >  3c59x 0000:00:0d.0: found PCI INT A -> IRQ 11
  > >  3c59x: Donald Becker and others.
  > >  0000:00:0d.0: 3Com PCI 3c900 Cyclone 10Mbps TPC at d8812000.
  > 
  > Should be fixed by:
  > 
  > commit 8400dd029e764e30361a51a7ae35900ae0032b43
  > Author: Neil Horman <nhorman@...driver.com>
  > Date:   Wed Sep 17 09:04:45 2014 -0400
  > 
  >     3c59x: Fix bad offset spec in skb_frag_dma_map
  >     
  >     Recently aded the use of skb_frag_dma_map to 3c59x, but didn't realize it
  >     automatically included the frag_offset internally, as well as provided an option
  >     to specify an extra offset in the parameter list.  We need to specify an offset
  >     of 0 in the parameter list to avoid skb corruption that results in lost
  >     connections.
  >     
  >     Signed-off-by: Neil Horman <nhorman@...driver.com>
  >     CC: Linux Kernel list <linux-kernel@...r.kernel.org>
  >     CC: "David S. Miller" <davem@...emloft.net>
  >     CC: Meelis Roos <mroos@...ux.ee>
  >     Tested-by: Meelis Roos <mroos@...ux.ee>
  > 
  > diff --git a/drivers/net/ethernet/3com/3c59x.c b/drivers/net/ethernet/3com/3c59x.c
  > index 8ab87ff..8ca49f04 100644
  > --- a/drivers/net/ethernet/3com/3c59x.c
  > +++ b/drivers/net/ethernet/3com/3c59x.c
  > @@ -2186,7 +2186,7 @@ boomerang_start_xmit(struct sk_buff *skb, struct net_device *dev)
  >  			skb_frag_t *frag = &skb_shinfo(skb)->frags[i];
  >  
  >  			dma_addr = skb_frag_dma_map(&VORTEX_PCI(vp)->dev, frag,
  > -						    frag->page_offset,
  > +						    0,
  >  						    frag->size,
  >  						    DMA_TO_DEVICE);
  >  			if (dma_mapping_error(&VORTEX_PCI(vp)->dev, dma_addr)) {
  > 
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html