diff options
| author | Jiyu Yang <jiyu.yang@nxp.com> | 2020-11-01 10:09:19 +0800 |
|---|---|---|
| committer | Xianzhong <xianzhong.li@nxp.com> | 2020-11-19 11:54:17 +0800 |
| commit | 20e75a90635cfe8e45ae2cfc70918be3cff04fea (patch) | |
| tree | c9b9b29302b79e832a75f8852fa66b62a231fdab /drivers/mxc | |
| parent | 269106c4da676c8eecceee9d7bffd13f5165848a (diff) | |
LF-2492 [#ccc] check sscanf return value
Signed-off-by: Jiyu Yang <jiyu.yang@nxp.com>
Diffstat (limited to 'drivers/mxc')
| -rw-r--r-- | drivers/mxc/gpu-viv/hal/os/linux/kernel/gc_hal_kernel_device.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/drivers/mxc/gpu-viv/hal/os/linux/kernel/gc_hal_kernel_device.c b/drivers/mxc/gpu-viv/hal/os/linux/kernel/gc_hal_kernel_device.c index 608f51f56c8e..0f845c439f5c 100644 --- a/drivers/mxc/gpu-viv/hal/os/linux/kernel/gc_hal_kernel_device.c +++ b/drivers/mxc/gpu-viv/hal/os/linux/kernel/gc_hal_kernel_device.c @@ -1425,9 +1425,13 @@ static int _set_clk(const char* buf) } } - sscanf(data, "%d %d %d", &dumpCore, &clkScale[0], &clkScale[1]); - - printk("Change core:%d MC scale:%d SH scale:%d\n", dumpCore, clkScale[0], clkScale[1]); + if (3 == sscanf(data, "%d %d %d", &dumpCore, &clkScale[0], &clkScale[1])) { + printk("Change core:%d MC scale:%d SH scale:%d\n", + dumpCore, clkScale[0], clkScale[1]); + } else { + printk("usage: echo \"0 32 32\" > clk\n"); + return 0; + } if (device->kernels[dumpCore]) { |