Commit f6514be5 authored by Dan Carpenter's avatar Dan Carpenter Committed by Rafael J. Wysocki

PM / hibernate: Fix memory corruption in resumedelay_setup()

In the original code "resume_delay" is an int so on 64 bits, the call to
kstrtoul() will cause memory corruption.  We may as well fix a style
issue here as well and make "resume_delay" unsigned int, since that's
what we pass to ssleep().

Fixes: 317cf7e5 (PM / hibernate: convert simple_strtoul to kstrtoul)
Signed-off-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Acked-by: default avatarPavel Machek <pavel@ucw.cz>
Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent 317cf7e5
...@@ -35,7 +35,7 @@ ...@@ -35,7 +35,7 @@
static int nocompress; static int nocompress;
static int noresume; static int noresume;
static int resume_wait; static int resume_wait;
static int resume_delay; static unsigned int resume_delay;
static char resume_file[256] = CONFIG_PM_STD_PARTITION; static char resume_file[256] = CONFIG_PM_STD_PARTITION;
dev_t swsusp_resume_device; dev_t swsusp_resume_device;
sector_t swsusp_resume_block; sector_t swsusp_resume_block;
...@@ -1115,7 +1115,7 @@ static int __init resumewait_setup(char *str) ...@@ -1115,7 +1115,7 @@ static int __init resumewait_setup(char *str)
static int __init resumedelay_setup(char *str) static int __init resumedelay_setup(char *str)
{ {
int rc = kstrtoul(str, 0, (unsigned long *)&resume_delay); int rc = kstrtouint(str, 0, &resume_delay);
if (rc) if (rc)
return rc; return rc;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment