diff options
author | Mariusz Kozlowski <m.kozlowski@tuxland.pl> | 2007-08-06 23:44:03 +0200 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2007-08-07 17:36:20 -0400 |
commit | 9dc83afdbefd184bf29f347e8fcbb6d8a2b5e6fe (patch) | |
tree | 34e295de896a8accf6ce47bfd39cb481d84b9d9e /drivers/net/ibmveth.c | |
parent | 163642a24a44d7b1d1e1b3cb8da25a142a919e24 (diff) | |
download | blackbird-op-linux-9dc83afdbefd184bf29f347e8fcbb6d8a2b5e6fe.tar.gz blackbird-op-linux-9dc83afdbefd184bf29f347e8fcbb6d8a2b5e6fe.zip |
drivers/net/ibmveth.c: memset fix
> >> Looks like memset() is zeroing wrong nr of bytes.
> >
> > Good catch, however, I think we can just remove this memset altogether
> > since the memory gets allocated via kzalloc.
>
> Correct, that memset() is superfluous.
Ok. Then this should do it.
Signed-off-by: Mariusz Kozlowski <m.kozlowski@tuxland.pl>
drivers/net/ibmveth.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
Signed-off-by: Jeff Garzik <jeff@garzik.org>
Diffstat (limited to 'drivers/net/ibmveth.c')
-rw-r--r-- | drivers/net/ibmveth.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/ibmveth.c b/drivers/net/ibmveth.c index cf4a92436aa8..acba90f1638e 100644 --- a/drivers/net/ibmveth.c +++ b/drivers/net/ibmveth.c @@ -963,7 +963,7 @@ static int __devinit ibmveth_probe(struct vio_dev *dev, const struct vio_device_ { int rc, i; struct net_device *netdev; - struct ibmveth_adapter *adapter = NULL; + struct ibmveth_adapter *adapter; unsigned char *mac_addr_p; unsigned int *mcastFilterSize_p; @@ -997,7 +997,6 @@ static int __devinit ibmveth_probe(struct vio_dev *dev, const struct vio_device_ SET_MODULE_OWNER(netdev); adapter = netdev->priv; - memset(adapter, 0, sizeof(adapter)); dev->dev.driver_data = netdev; adapter->vdev = dev; |