[PATCH] winealsa.drv: Init *num to 0 (Coverity)

Marcus Meissner marcus at jet.franken.de
Mon Apr 16 14:22:05 CDT 2012


On Mon, Apr 16, 2012 at 11:53:00AM -0500, Andrew Eikum wrote:
> On Mon, Apr 16, 2012 at 09:09:58AM +0200, Marcus Meissner wrote:
> > Also initialize a "may be uninitialized" value the compiler sees.
> > 
> 
> Not sure what this means. *num is initialized at the beginning of
> alsa_enum_devices, which is always called from GetEndpointIDs. Seems
> like a Coverity oversight. I'd either leave it as-is, or also remove
> the *num=0 from alsa_enum_devices.

It cannot handle two variables that track the same logic easily.
So it cannot see "counter" and "content" variables that belong
together.

I marked it as FALSE now.

Ciao, Marcus



More information about the wine-devel mailing list