[PATCH 1/4] cmd: clk: test the number of argument in setfreq command

Sean Anderson seanga2 at gmail.com
Tue Feb 1 02:16:39 CET 2022


On 1/31/22 11:21 AM, Patrick Delaunay wrote:
> 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 at foss.st.com>
> ---
> 
>   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]);
> 

Reviewed-by: Sean Anderson <seanga2 at gmail.com>


More information about the U-Boot mailing list