bnx2x: remove zeroing of dump data buffer
authorMichal Schmidt <mschmidt@redhat.com>
Mon, 1 Jul 2013 15:23:05 +0000 (17:23 +0200)
committerDavid S. Miller <davem@davemloft.net>
Tue, 2 Jul 2013 07:15:56 +0000 (00:15 -0700)
There is no need to initialize the dump data with zeros.
data is allocated with vzalloc, so it's already zero-filled.

More importantly, the memset is harmful, because dump->len (the length
requested by userspace) can be bigger than the allocated buffer (whose
size is determined by asking the driver's .get_dump_flag method).

Signed-off-by: Michal Schmidt <mschmidt@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c

index b8c067d..4f75ae8 100644 (file)
@@ -985,8 +985,6 @@ static int bnx2x_get_dump_data(struct net_device *dev,
        struct bnx2x *bp = netdev_priv(dev);
        struct dump_header dump_hdr = {0};
 
-       memset(p, 0, dump->len);
-
        /* Disable parity attentions as long as following dump may
         * cause false alarms by reading never written registers. We
         * will re-enable parity attentions right after the dump.