diff options
author | Florian Fainelli <florian@openwrt.org> | 2013-03-14 14:45:56 +0000 |
---|---|---|
committer | Florian Fainelli <florian@openwrt.org> | 2013-03-14 14:45:56 +0000 |
commit | ddc8718f7f4ef9723e026efac811d3338e6bcddb (patch) | |
tree | 08d6be7813310be182259388518215a17afcc18c /target/linux/generic/patches-3.9/993-mpcore_wdt_fix_wdioc_setoptions_handling.patch | |
parent | 6cb1f067dc42c4bffa8862448167486333d0227e (diff) | |
download | upstream-ddc8718f7f4ef9723e026efac811d3338e6bcddb.tar.gz upstream-ddc8718f7f4ef9723e026efac811d3338e6bcddb.tar.bz2 upstream-ddc8718f7f4ef9723e026efac811d3338e6bcddb.zip |
kernel: add support for 3.9-rc2
Signed-off-by: Florian Fainelli <florian@openwrt.org>
git-svn-id: svn://svn.openwrt.org/openwrt/trunk@36008 3c298f89-4303-0410-b956-a3cf2f4a3e73
Diffstat (limited to 'target/linux/generic/patches-3.9/993-mpcore_wdt_fix_wdioc_setoptions_handling.patch')
-rw-r--r-- | target/linux/generic/patches-3.9/993-mpcore_wdt_fix_wdioc_setoptions_handling.patch | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/target/linux/generic/patches-3.9/993-mpcore_wdt_fix_wdioc_setoptions_handling.patch b/target/linux/generic/patches-3.9/993-mpcore_wdt_fix_wdioc_setoptions_handling.patch new file mode 100644 index 0000000000..c986ac8f21 --- /dev/null +++ b/target/linux/generic/patches-3.9/993-mpcore_wdt_fix_wdioc_setoptions_handling.patch @@ -0,0 +1,29 @@ +According to the include/linux/watchdog.h WDIOC_SETOPTIONS is +classified as 'read from device' ioctl call: + #define WDIOC_SETOPTIONS _IOR(WATCHDOG_IOCTL_BASE, 4, int) + +However, the driver 'mpcore_wdt' performs 'copy_from_user' only if +_IOC_WRITE is set, thus the local variable 'uarg' which is used in +WDIOC_SETOPTIONS handling remains uninitialized. + +The proper way to fix this is to bind WDIOC_SETOPTIONS to _IOW, +but this will break compatibility. +So adding additional condition for performing 'copy_from_user'. + +Signed-off-by: Vitaly Kuzmichev <vkuzmichev@mvista.com> +--- + drivers/watchdog/mpcore_wdt.c | 3 ++- + 1 files changed, 2 insertions(+), 1 deletions(-) + +--- a/drivers/watchdog/mpcore_wdt.c ++++ b/drivers/watchdog/mpcore_wdt.c +@@ -235,7 +235,8 @@ static long mpcore_wdt_ioctl(struct file + if (_IOC_DIR(cmd) && _IOC_SIZE(cmd) > sizeof(uarg)) + return -ENOTTY; + +- if (_IOC_DIR(cmd) & _IOC_WRITE) { ++ if ((_IOC_DIR(cmd) & _IOC_WRITE) ++ || cmd == WDIOC_SETOPTIONS) { + ret = copy_from_user(&uarg, (void __user *)arg, _IOC_SIZE(cmd)); + if (ret) + return -EFAULT; |