watchdog: w83977f_wdt: underflow in wdt_set_timeout()
authorDan Carpenter <dan.carpenter@oracle.com>
Fri, 6 Nov 2015 09:56:31 +0000 (12:56 +0300)
committerWim Van Sebroeck <wim@iguana.be>
Mon, 23 Nov 2015 07:59:30 +0000 (08:59 +0100)
commit62ed853c7de7a77bdf01421fb6836269c9b1aa1f
tree3064fea9ba4badc85af7233ddbe7dfdd216969bc
parent4c30737ce16612b0f3b840a7c9928385cbb67555
watchdog: w83977f_wdt: underflow in wdt_set_timeout()

"t" is controlled by the user.  If "t" is a very large integer then it
could lead to a negative "tmrval".  We cap the upper bound of "tmrval"
but, in the current code, we allow negatives.  This is a bug and it
causes a static checker warning.  Let's make "tmrval" unsigned to avoid
this problem.

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>
drivers/watchdog/w83977f_wdt.c