mirror of
https://github.com/hardkernel/linux.git
synced 2026-03-31 18:23:00 +09:00
debug: drop sscanf, use kstrto variant instead
PD#165796: debug: drop sscanf, use kstrto variant instead kstrto is kernel reconmmend function which is more safe than sscanf this will also fix the following checkpatch.pl warning WARNING: Prefer kstrto<type> to single variable sscanf + ret = sscanf(buf, "%ld", &val); Change-Id: I9a4673b9aa43c10f51a20778ba520461ee0a7984 Signed-off-by: Yixun Lan <yixun.lan@amlogic.com>
This commit is contained in:
@@ -294,7 +294,9 @@ static ssize_t x##_write(struct file *file, const char __user *userbuf,\
|
||||
if (copy_from_user(buf, userbuf, count)) \
|
||||
return -EFAULT; \
|
||||
buf[count] = 0; \
|
||||
ret = sscanf(buf, "%ld", &val); \
|
||||
ret = kstrtoul(buf, 0, &val); \
|
||||
if (ret) \
|
||||
return -1; \
|
||||
x = (typeof(x))val; \
|
||||
if (irq_check_en || isr_check_en) \
|
||||
aml_wdt_disable_dbg(); \
|
||||
|
||||
Reference in New Issue
Block a user