[PATCH] ole32: fixed 3 handle leaks (Coverity)

Marcus Meissner marcus at jet.franken.de
Mon Jun 10 01:41:47 CDT 2013


On Mon, Jun 10, 2013 at 08:25:42AM +0200, Marcus Meissner wrote:
> On Mon, Jun 10, 2013 at 10:43:58AM +0900, Dmitry Timoshkov wrote:
> > Marcus Meissner <marcus at jet.franken.de> wrote:
> > 
> > >  1030106 Resource leak
> > >  1030105 Resource leak
> > >  1030104 Resource leak
> > 
> > These cases are of the kind 'freeing resources at process exit is not useful'.
> 
> Well, it is code that is run within a library.
> 
> The usage pattern is unclear and might cover repeated calls.

ah, that was ole_server.c ... of course this is not a library.

But please ... good practice? :(

Ciao, Marcus



More information about the wine-devel mailing list