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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 02 Jul 2007 08:20:50 -0400 From: Jeff Garzik <jeff@...zik.org> To: Veena Parat <Veena.Parat@...erion.com> CC: netdev@...r.kernel.org, Leonid.Grossman@...erion.com, ramkrishna.vepa@...erion.com, santosh.rastapur@...erion.com, Sivakumar.Subramani@...erion.com, sreenivasa.honnur@...erion.com, alicia.pena@...erion.com, sriram.rapuru@...erion.com Subject: Re: [PATCH 2.6.22 1/4]S2IO: Adding checks to check the return value of pci mapping function Veena Parat wrote: > Adding checks to check the return value of pci mapping function > > Signed-off-by: Veena Parat <veena.parat@...erion.com> > --- > diff -urpN org/drivers/net/s2io.c patch_1/drivers/net/s2io.c > --- org/drivers/net/s2io.c 2007-05-17 20:35:39.000000000 +0530 > +++ patch_1/drivers/net/s2io.c 2007-05-17 20:35:55.000000000 +0530 > @@ -282,6 +282,7 @@ static char ethtool_driver_stats_keys[][ > ("lro_flush_due_to_max_pkts"), > ("lro_avg_aggr_pkts"), > ("mem_alloc_fail_cnt"), > + {"pci_map_fail_cnt"}, > ("watchdog_timer_cnt"), > ("mem_allocated"), > ("mem_freed"), > @@ -2247,10 +2248,19 @@ static int fill_rxd_3buf(struct s2io_nic > (nic->pdev, skb->data, l3l4hdr_size + 4, > PCI_DMA_FROMDEVICE); > > + if ((((struct RxD3*)rxdp)->Buffer1_ptr == 0) || > + (((struct RxD3*)rxdp)->Buffer1_ptr == DMA_ERROR_CODE)) { > + nic->mac_control.stats_info->sw_stat.pci_map_fail_cnt++; > + return -ENOMEM; > + } > + > /* skb_shinfo(skb)->frag_list will have L4 data payload */ > skb_shinfo(skb)->frag_list = dev_alloc_skb(dev->mtu + ALIGN_SIZE); > if (skb_shinfo(skb)->frag_list == NULL) { > nic->mac_control.stats_info->sw_stat.mem_alloc_fail_cnt++; > + pci_unmap_single > + (nic->pdev, (dma_addr_t)skb->data, l3l4hdr_size + 4, > + PCI_DMA_FROMDEVICE); > DBG_PRINT(INFO_DBG, "%s: dev_alloc_skb failed\n ", dev->name); > return -ENOMEM ; > } > @@ -2267,6 +2277,11 @@ static int fill_rxd_3buf(struct s2io_nic > ((struct RxD3*)rxdp)->Buffer2_ptr = pci_map_single(nic->pdev, > frag_list->data, dev->mtu, > PCI_DMA_FROMDEVICE); > + if ((((struct RxD3*)rxdp)->Buffer2_ptr == 0) || > + (((struct RxD3*)rxdp)->Buffer2_ptr == DMA_ERROR_CODE)) { > + nic->mac_control.stats_info->sw_stat.pci_map_fail_cnt++; > + return -ENOMEM; > + } > rxdp->Control_2 |= SET_BUFFER1_SIZE_3(l3l4hdr_size + 4); > rxdp->Control_2 |= SET_BUFFER2_SIZE_3(dev->mtu); > > @@ -2399,6 +2414,16 @@ static int fill_rx_buffers(struct s2io_n > ((struct RxD1*)rxdp)->Buffer0_ptr = pci_map_single > (nic->pdev, skb->data, size - NET_IP_ALIGN, > PCI_DMA_FROMDEVICE); > + if ((((struct RxD1*)rxdp)->Buffer0_ptr == 0) || > + (((struct RxD1*)rxdp)->Buffer0_ptr == > + DMA_ERROR_CODE)) { > + nic->mac_control.stats_info->sw_stat. > + pci_map_fail_cnt++; > + nic->mac_control.stats_info->sw_stat.mem_freed > + += skb->truesize; > + dev_kfree_skb_irq(skb); > + return -ENOMEM; > + } > rxdp->Control_2 = > SET_BUFFER0_SIZE_1(size - NET_IP_ALIGN); > please provide a prep patch that eliminates all these inline casts. Create temporary variables or whatever you need to do to convert the messy and unreadable "(((struct RxD1*)rxdp)->Buffer0_ptr" into "desc->Buffer0_ptr". otherwise looks OK - 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
Powered by blists - more mailing lists