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

2013-06-09 Thread Dmitry Timoshkov
Marcus Meissner 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

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

2013-06-09 Thread Dmitry Timoshkov
Marcus Meissner 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. What do you mean by "library" in this con

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

2013-06-09 Thread Marcus Meissner
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 wrote: > > > > > 1030106 Resource leak > > > 1030105 Resource leak > > > 1030104 Resource leak > > > > These cases are of the kind 'freeing

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

2013-06-09 Thread Marcus Meissner
On Mon, Jun 10, 2013 at 10:43:58AM +0900, Dmitry Timoshkov wrote: > Marcus Meissner 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

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

2013-06-09 Thread Dmitry Timoshkov
Marcus Meissner wrote: > 1030106 Resource leak > 1030105 Resource leak > 1030104 Resource leak These cases are of the kind 'freeing resources at process exit is not useful'. -- Dmitry.