diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2013-08-23 04:40:59 -0400 |
|---|---|---|
| committer | Wim Van Sebroeck <wim@iguana.be> | 2013-10-13 14:01:50 -0400 |
| commit | 8612ed0d97abcf1c016d34755b7cf2060de71963 (patch) | |
| tree | ba08f8831713c10a9f9c63214185063ef294c967 /drivers | |
| parent | 9d05746e7b16d8565dddbe3200faa1e669d23bbf (diff) | |
watchdog: ts72xx_wdt: locking bug in ioctl
Calling the WDIOC_GETSTATUS & WDIOC_GETBOOTSTATUS and twice will cause a
interruptible deadlock.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@iguana.be>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/watchdog/ts72xx_wdt.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/watchdog/ts72xx_wdt.c b/drivers/watchdog/ts72xx_wdt.c index 42913f131dc2..c9b0c627fe7e 100644 --- a/drivers/watchdog/ts72xx_wdt.c +++ b/drivers/watchdog/ts72xx_wdt.c | |||
| @@ -310,7 +310,8 @@ static long ts72xx_wdt_ioctl(struct file *file, unsigned int cmd, | |||
| 310 | 310 | ||
| 311 | case WDIOC_GETSTATUS: | 311 | case WDIOC_GETSTATUS: |
| 312 | case WDIOC_GETBOOTSTATUS: | 312 | case WDIOC_GETBOOTSTATUS: |
| 313 | return put_user(0, p); | 313 | error = put_user(0, p); |
| 314 | break; | ||
| 314 | 315 | ||
| 315 | case WDIOC_KEEPALIVE: | 316 | case WDIOC_KEEPALIVE: |
| 316 | ts72xx_wdt_kick(wdt); | 317 | ts72xx_wdt_kick(wdt); |
