On Mon, 27 Feb 2023 at 12:55, Marek Vasut <[email protected]> wrote: > > On case 'fdt bootcpu' is invoked without parameters, argv[2] is not > valid and this command would SEGFAULT in sandbox environment. Add > missing argc test to avoid the crash and rather print usage help > message. > > Signed-off-by: Marek Vasut <[email protected]> > --- > Cc: Heinrich Schuchardt <[email protected]> > Cc: Simon Glass <[email protected]> > Cc: Tom Rini <[email protected]> > --- > cmd/fdt.c | 7 ++++++- > 1 file changed, 6 insertions(+), 1 deletion(-)
Yes sandbox is good for finding this sort of thing. Reviewed-by: Simon Glass <[email protected]>
