thunderbolt: debugfs: Fix uninitialized return in counters_write()
authorDan Carpenter <dan.carpenter@oracle.com>
Thu, 10 Sep 2020 10:08:05 +0000 (13:08 +0300)
committerMika Westerberg <mika.westerberg@linux.intel.com>
Thu, 10 Sep 2020 16:39:56 +0000 (19:39 +0300)
If the first line is in an invalid format then the "ret" value is
uninitialized.  We should return -EINVAL instead.

Fixes: 54e418106c76 ("thunderbolt: Add debugfs interface")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Mika Westerberg <mika.westerberg@linux.intel.com>
drivers/thunderbolt/debugfs.c

index fdfe6e4..3680b27 100644 (file)
@@ -231,6 +231,7 @@ static ssize_t counters_write(struct file *file, const char __user *user_buf,
                char *line = buf;
                u32 val, offset;
 
+               ret = -EINVAL;
                while (parse_line(&line, &offset, &val, 1, 4)) {
                        ret = tb_port_write(port, &val, TB_CFG_COUNTERS,
                                            offset, 1);