[alsa-devel] [PATCHv2] aplay: Limit VUMeter progress bar to 100 for negative as well

Takashi Iwai tiwai at suse.de
Wed Nov 20 22:02:08 CET 2019


On Wed, 20 Nov 2019 20:13:55 +0100,
Rosen Penev wrote:
> 
> While the progress bar cannot be negative, GCC concludes that it can be
> and assumes tmp can be written past the end. Fixes this GCC warning:
> 
> aplay.c:1747:18: warning: '%02d' directive writing between 2 and 11 bytes
>  into a region of size 4 [-Wformat-overflow=]
>  1747 |    sprintf(tmp, "%02d%%", maxperc[c]);
> 
> Signed-off-by: Rosen Penev <rosenp at gmail.com>

Applied, thanks.


Takashi


More information about the Alsa-devel mailing list