| Date | Mon, 12 May 2014 02:32:10 +0200 | From | Willy Tarreau <> | Subject | [ 010/143] cpqarray: fix info leak in ida_locked_ioctl() |
| |
2.6.32-longterm 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: Willy Tarreau <w@1wt.eu> --- drivers/block/cpqarray.c | 1 + 1 file changed, 1 insertion(+)
diff --git a/drivers/block/cpqarray.c b/drivers/block/cpqarray.c index 6422651..f9caa45 100644 --- a/drivers/block/cpqarray.c +++ b/drivers/block/cpqarray.c @@ -1181,6 +1181,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; -- 1.7.12.2.21.g234cd45.dirty
|