diff options
author | Patrick Delaunay <patrick.delaunay@foss.st.com> | 2022-01-31 17:21:37 +0100 |
---|---|---|
committer | Sean Anderson <seanga2@gmail.com> | 2022-02-25 01:41:04 -0500 |
commit | 3386fb1e485da7f206488ed206a61ae811885d90 (patch) | |
tree | 06213f8aca5fb1ce9192d4842c8b6012e9f023fd /cmd | |
parent | b0cdd8287add19d1af6668b6e9e968046506c08d (diff) | |
download | u-boot-3386fb1e485da7f206488ed206a61ae811885d90.tar.gz |
cmd: clk: test the number of argument in setfreq command
Test the number of argument in setfreq command to avoid a crash when
the command setfreq is called without argument:
STM32MP> clk setfreq
data abort
pc : [<ddba3f18>] lr : [<ddba3f89>]
reloc pc : [<c018ff18>] lr : [<c018ff89>]
sp : dbaf45b8 ip : ddb1d859 fp : 00000002
r10: dbb3fd80 r9 : dbb11e90 r8 : ddbf38cc
r7 : ddb39725 r6 : 00000000 r5 : 00000000 r4 : dbb3fd84
r3 : dbb3fd84 r2 : 0000000a r1 : dbaf45bc r0 : 00000011
Flags: nzCv IRQs off FIQs off Mode SVC_32 (T)
Code: 4dd3 1062 85a3 ddbd (7803) 2b30
Resetting CPU ...
Fixes: 7ab418fbe612 ("clk: add support for setting clk rate from cmdline")
Signed-off-by: Patrick Delaunay <patrick.delaunay@foss.st.com>
Reviewed-by: Sean Anderson <seanga2@gmail.com>
Link: https://lore.kernel.org/r/20220131172131.1.I32a8f213d330dccd922f7aafc60d3d63fcbe8615@changeid
Diffstat (limited to 'cmd')
-rw-r--r-- | cmd/clk.c | 3 |
1 files changed, 3 insertions, 0 deletions
@@ -120,6 +120,9 @@ static int do_clk_setfreq(struct cmd_tbl *cmdtp, int flag, int argc, s32 freq; struct udevice *dev; + if (argc != 3) + return CMD_RET_USAGE; + freq = dectoul(argv[2], NULL); dev = clk_lookup(argv[1]); |