Re: [PATCH 1/1] cmd: avoid endless loop in sound play command

On Fri, 30 Dec 2022 at 17:54, Heinrich Schuchardt heinrich.schuchardt@canonical.com wrote:

A parameter starting with a hyphen leads to an endless loop in the sound play command.

Leave it to dectoul() to handle the hyphen. It will return 0 for a negative number.

Signed-off-by: Heinrich Schuchardt heinrich.schuchardt@canonical.com

cmd/sound.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)

Reviewed-by: Simon Glass sjg@chromium.org

Could add a test for this command perhaps