From 3386fb1e485da7f206488ed206a61ae811885d90 Mon Sep 17 00:00:00 2001 From: Patrick Delaunay Date: Mon, 31 Jan 2022 17:21:37 +0100 Subject: [PATCH] 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 : [] lr : [] reloc pc : [] lr : [] 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 Reviewed-by: Sean Anderson Link: https://lore.kernel.org/r/20220131172131.1.I32a8f213d330dccd922f7aafc60d3d63fcbe8615@changeid --- cmd/clk.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/cmd/clk.c b/cmd/clk.c index dbbdc31b35..52237791cf 100644 --- a/cmd/clk.c +++ b/cmd/clk.c @@ -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]); -- 2.39.5