shlwapi.dll, Fix endless loop in StrPBrkW

Michael Stefaniuc mstefani at redhat.de
Mon Mar 3 16:14:34 CST 2003


On Mon, Mar 03, 2003 at 04:14:16PM -0500, Vincent Béron wrote:
> Michael Stefaniuc a écrit:
> > 
> > That's even easier in smatch, i have to just check for:
> > (if|for|while)_cond
> > end_(if|for|while)
> > 
> > but that's still a lot of false positives. One false positive (real
> > code) is:
> > 	while (*p++ != 0x4D && p < pend);
> > 
> > I need to check what's inside the () too. I'm looking at the moment at
> > the false positives to know what to look for.
> 
> Probably anything that changes a value (++, --, =, +=, -=, *=, /=, etc.).
That is my intention.

> But then you risk losing some real problems, as something might be 
> correctly assigned in the condition part of a while, but with a ";" as 
> loop instead of the real loop.
Hmm ... that is also easy to check
while (...);
{
translates into smatch:
end_while
cmpstmt_start

A lot of nice stuff to play with.

bye
	michael

-- 
Michael Stefaniuc               Tel.: +49-711-96437-199
System Administration           Fax.: +49-711-96437-111
Red Hat GmbH                    Email: mstefani at redhat.com
Hauptstaetterstr. 58            http://www.redhat.de/
D-70178 Stuttgart
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 232 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-devel/attachments/20030303/dc87001a/attachment.pgp


More information about the wine-devel mailing list