lkml.org 
[lkml]   [2013]   [Oct]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[ 002/135] cpqarray: fix info leak in ida_locked_ioctl()
    Date
    3.11-stable review patch.  If anyone has any objections, please let me know.

    ------------------

    From: Dan Carpenter <dan.carpenter@oracle.com>

    commit 627aad1c01da6f881e7f98d71fd928ca0c316b1a upstream.

    The pciinfo struct has a two byte hole after ->dev_fn so stack
    information could be leaked to the user.

    This was assigned CVE-2013-2147.

    Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
    Acked-by: Mike Miller <mike.miller@hp.com>
    Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
    Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
    Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

    ---
    drivers/block/cpqarray.c | 1 +
    1 file changed, 1 insertion(+)

    --- a/drivers/block/cpqarray.c
    +++ b/drivers/block/cpqarray.c
    @@ -1193,6 +1193,7 @@ out_passthru:
    ida_pci_info_struct pciinfo;

    if (!arg) return -EINVAL;
    + memset(&pciinfo, 0, sizeof(pciinfo));
    pciinfo.bus = host->pci_dev->bus->number;
    pciinfo.dev_fn = host->pci_dev->devfn;
    pciinfo.board_id = host->board_id;



    \
     
     \ /
      Last update: 2013-10-11 23:41    [W:4.122 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site