Use the text to integer helper that has error handling and doesn't modify
the input pointer.

Signed-off-by: Christoph Hellwig <h...@lst.de>
---
 block/blk-integrity.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/block/blk-integrity.c b/block/blk-integrity.c
index 686b6adf0ea5c7..bb3cd1e0eeb58e 100644
--- a/block/blk-integrity.c
+++ b/block/blk-integrity.c
@@ -245,8 +245,12 @@ static ssize_t flag_store(struct device *dev, struct 
device_attribute *attr,
                const char *page, size_t count, unsigned char flag)
 {
        struct blk_integrity *bi = dev_to_bi(dev);
-       char *p = (char *) page;
-       unsigned long val = simple_strtoul(p, &p, 10);
+       unsigned long val;
+       int err;
+
+       err = kstrtoul(page, 10, &val);
+       if (err)
+               return err;
 
        if (val)
                bi->flags |= flag;
-- 
2.43.0


Reply via email to