[lkml]   [2010]   [Dec]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    Patch in this message
    Subject[29/44] net: clear heap allocations for privileged ethtool actions
    2.6.27-stable review patch.  If anyone has any objections, please let us know.


    From: Kees Cook <>

    [ Upstream commit b00916b189d13a615ff05c9242201135992fcda3 ]

    Several other ethtool functions leave heap uncleared (potentially) by
    drivers. Some interfaces appear safe (eeprom, etc), in that the sizes
    are well controlled. In some situations (e.g. unchecked error conditions),
    the heap will remain unchanged in areas before copying back to userspace.
    Note that these are less of an issue since these all require CAP_NET_ADMIN.

    Signed-off-by: Kees Cook <>
    Acked-by: Ben Hutchings <>
    Signed-off-by: David S. Miller <>
    Signed-off-by: Greg Kroah-Hartman <>
    net/core/ethtool.c | 2 +-
    1 file changed, 1 insertion(+), 1 deletion(-)

    --- a/net/core/ethtool.c
    +++ b/net/core/ethtool.c
    @@ -256,7 +256,7 @@ static int ethtool_get_regs(struct net_d
    if (regs.len > reglen)
    regs.len = reglen;

    - regbuf = kmalloc(reglen, GFP_USER);
    + regbuf = kzalloc(reglen, GFP_USER);
    if (!regbuf)
    return -ENOMEM;

     \ /
      Last update: 2010-12-08 01:41    [W:0.020 / U:6.156 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site