New Coverity run

2009-03-17 Thread Paul Vriens
Hi, Just wanted to mention that there is a new Coverity run. There were some issues the last weeks that made sure the source wasn't updated (most likely started with the disk issue on winehq.org early February). -- Cheers, Paul.

Re: New coverity run Feb 7, 14 new warnings

2009-02-08 Thread Jacek Caban
Dan Kegel wrote: > Is it a bug? I don't know those flags well enough. It's not a bug, a constant 0 value is passed as flags to set_nsstyle_attr. I've marked it as FALSE in coverity. Jacek

Re: New coverity run Feb 7, 14 new warnings

2009-02-08 Thread Dan Kegel
Alistair wrote: >> 873 FORWARD_NULLHTMLStyle_put_fontWeight dlls/mshtml/htmlstyle.c > > NULL is a valid value to be passed though to gecko. Here's more of the error: dlls/mshtml/htmlstyle.c Event var_deref_model: Variable "v" tracked as NULL was passed to a function that dereferences i

Re: New coverity run Feb 7, 14 new warnings

2009-02-08 Thread Alistair Leslie-Hughes
Hi Dan, > 873 FORWARD_NULLHTMLStyle_put_fontWeight > dlls/mshtml/htmlstyle.c NULL is a valid value to be passed though to gecko. Best Regards Alistair Leslie-Hughes

New coverity run Feb 7, 14 new warnings

2009-02-08 Thread Dan Kegel
Congrats to everyone who's been cleaning up Coverity warnings, especially Marcus (http://marcusmeissner.livejournal.com/24466.html). Here are the yet-uninspected warnings that are in the Feb 7 run but not in the Jan 30 run: 884 REVERSE_INULL compile_file_regex dlls/dbghelp/symbol.c 883

Re: New Coverity run

2009-01-31 Thread Paul Vriens
Marcus Meissner wrote: > On Fri, Jan 23, 2009 at 09:44:45PM +0100, Paul Vriens wrote: >> Hi, >> >> I've been in touch with David Maxwell and he looked into the 'lack' of >> reports since the end of October. > > And there is now Run 278 :) > > Ciao, Marcus Hi, We even have a run 280 now but it

Re: New Coverity run

2009-01-24 Thread Dan Kegel
On Sat, Jan 24, 2009 at 8:29 AM, Alexandre Julliard wrote: >> http://bugs.winehq.org/show_bug.cgi?id=17112 >> Should we make that a release criterion? > > I don't think so, getting access to that info is troublesome, and we > can't easily trigger new test runs. We don't want to have the release >

Re: New Coverity run

2009-01-24 Thread Alexandre Julliard
Dan Kegel writes: > On Sat, Jan 24, 2009 at 3:24 AM, Marcus Meissner > wrote: >>> David promised me we get even more issues reported once we get to rung2 :). >>> The engine they use for rung2 is new and better apparently. >> >> I see it with gphoto2 for instance. >> >> However, what was the cri

Re: New Coverity run

2009-01-24 Thread Dan Kegel
On Sat, Jan 24, 2009 at 3:24 AM, Marcus Meissner wrote: >> David promised me we get even more issues reported once we get to rung2 :). >> The engine they use for rung2 is new and better apparently. > > I see it with gphoto2 for instance. > > However, what was the criteria again for rung2? 0 unrevi

Re: New Coverity run

2009-01-24 Thread Kai Blin
On Saturday 24 January 2009 12:24:29 Marcus Meissner wrote: > On Sat, Jan 24, 2009 at 12:22:33PM +0100, Paul Vriens wrote: > > Dan Kegel wrote: > >> On Fri, Jan 23, 2009 at 2:47 PM, Marcus Meissner wrote: > >>> There are actually more ... 815 - 870 (ca 57), and other types too. > >> > >> Whoops,

Re: New Coverity run

2009-01-24 Thread Marcus Meissner
On Sat, Jan 24, 2009 at 12:22:33PM +0100, Paul Vriens wrote: > Dan Kegel wrote: >> On Fri, Jan 23, 2009 at 2:47 PM, Marcus Meissner >> wrote: >>> There are actually more ... 815 - 870 (ca 57), and other types too. >> >> Whoops, right, I only got the first page, I guess. Here's a more complete >

Re: New Coverity run

2009-01-24 Thread Paul Vriens
Dan Kegel wrote: > On Fri, Jan 23, 2009 at 2:47 PM, Marcus Meissner > wrote: >> There are actually more ... 815 - 870 (ca 57), and other types too. > > Whoops, right, I only got the first page, I guess. Here's a more complete > list. David promised me we get even more issues reported once we

Re: New Coverity run

2009-01-23 Thread Dan Kegel
On Fri, Jan 23, 2009 at 2:47 PM, Marcus Meissner wrote: > There are actually more ... 815 - 870 (ca 57), and other types too. Whoops, right, I only got the first page, I guess. Here's a more complete list. For more info, get a coverity account at http://scan.coverity.com/devfaq.html#account or a

Re: New Coverity run

2009-01-23 Thread Marcus Meissner
On Fri, Jan 23, 2009 at 01:42:38PM -0800, Dan Kegel wrote: > ... and here are the new issues since the last run, all of type FORWARD_NULL. > If you don't have coverity access, see > http://scan.coverity.com/devfaq.html#account > for how to get it. > > 595 CryptDecodeObjectEx dlls/crypt32/d

re: New Coverity run

2009-01-23 Thread Dan Kegel
... and here are the new issues since the last run, all of type FORWARD_NULL. If you don't have coverity access, see http://scan.coverity.com/devfaq.html#account for how to get it. 595 CryptDecodeObjectEx dlls/crypt32/decode.c 596 CryptEncodeObjectEx dlls/crypt32/encode.c 549 S

Re: New Coverity run

2009-01-23 Thread Marcus Meissner
On Fri, Jan 23, 2009 at 09:44:45PM +0100, Paul Vriens wrote: > Hi, > > I've been in touch with David Maxwell and he looked into the 'lack' of > reports since the end of October. And there is now Run 278 :) Ciao, Marcus

New Coverity run

2009-01-23 Thread Paul Vriens
Hi, I've been in touch with David Maxwell and he looked into the 'lack' of reports since the end of October. -- Cheers, Paul.

Re: New Coverity run Oct 1st. 58 fixed, 41 new warnings in cabinet, jscript, mshtml, wined3d, and code generated by widl

2008-10-07 Thread Aric Stewart
Do you have coverity access? if so we can make all these ignore. If not then tell me and I will go in and mark them ignore. -aric Rob Shearman wrote: > 2008/10/6 Dan Kegel <[EMAIL PROTECTED]>: >> Thanks to the half-dozen Wine developers attacking the Coverity >> warnings, we are slowly making p

Re: New Coverity run Oct 1st. 58 fixed, 41 new warnings in cabinet, jscript, mshtml, wined3d, and code generated by widl

2008-10-06 Thread Rob Shearman
2008/10/6 Dan Kegel <[EMAIL PROTECTED]>: > Thanks to the half-dozen Wine developers attacking the Coverity > warnings, we are slowly making progress. > > Here are the warnings that were new as of October 1st > and which have not yet been classified. > They fall into five areas (four DLLs and one ki

New Coverity run Oct 1st. 58 fixed, 41 new warnings in cabinet, jscript, mshtml, wined3d, and code generated by widl

2008-10-06 Thread Dan Kegel
Thanks to the half-dozen Wine developers attacking the Coverity warnings, we are slowly making progress. Here are the warnings that were new as of October 1st and which have not yet been classified. They fall into five areas (four DLLs and one kind of generated code): CID Checker