Hi Anton,

Thanks for the patch.

> Report of the static analyzer:
> DIVISION_BY_ZERO.EX Variable xatou(...),
> whose possible value set allows a zero value at xatonum_template.c:118 by 
> calling function 'xatou' at beep.c:90,
> is used as a denominator at beep.c:90.
>
> Corrections explained:
> - If xatou(optarg) returns 0, freq is set to 440 instead of
>   terminating the program.
> - A warning message is printed to inform the user about the change.

Not anymore, no? ;)

> - Removed redundant second call to xatou(optarg), using freq
>
> Triggers found by static analyzer Svace.
>
> Signed-off-by: Anton Moryakov <[email protected]>

Cheers,

Xabier Oneca_,,_
_______________________________________________
busybox mailing list
[email protected]
https://lists.busybox.net/mailman/listinfo/busybox

Reply via email to