Patch "USB: usbfs: fix potential infoleak in devio" has been added to the 4.4-stable tree
Subsystems (2)
This is a note to let you know that I've just added the patch titled
USB: usbfs: fix potential infoleak in devio
to the 4.4-stable tree which can be found at:
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
The filename of the patch is:
usb-usbfs-fix-potential-infoleak-in-devio.patch
and it can be found in the queue-4.4 subdirectory.
If you, or anyone else, feels it should not be added to the stable tree, please let <[email protected]> know about it.
Quoted 1 line
From
681fef8380eb818c0b845fca5d2ab1dcbab114eeMon Sep 17 00:00:00 2001
From: Kangjie Lu <[email protected]>
Date: Tue, 3 May 2016 16:32:16 -0400
Subject: USB: usbfs: fix potential infoleak in devio
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
From: Kangjie Lu <[email protected]>
commit 681fef8380eb818c0b845fca5d2ab1dcbab114ee upstream.
The stack object “ci” has a total size of 8 bytes. Its last 3 bytes are padding bytes which are not initialized and leaked to userland via “copy_to_user”.
--- drivers/usb/core/devio.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-)
drivers/usb/core/devio.c +5 -4
@@ -1203,10 +1203,11 @@ static int proc_getdriver(struct usb_devstatic int proc_connectinfo (struct usb_dev_state *ps,void __user *arg) { -struct usbdevfs_connectinfo ci = { - .devnum = ps->dev->devnum, - .slow = ps->dev->speed ==USB_SPEED_LOW - }; +struct usbdevfs_connectinfo ci; + +memset (&ci,0 ,sizeof (ci)); + ci.devnum = ps->dev->devnum; + ci.slow = ps->dev->speed ==USB_SPEED_LOW ;if (copy_to_user (arg, &ci,sizeof (ci)))return -EFAULT ;
Patches currently in stable-queue which might be from [email protected] are
queue-4.4/usb-usbfs-fix-potential-infoleak-in-devio.patch