[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Bug 60903] PVS-Studio: fixed weaknesses


Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> changed:

           What    |Removed                     |Added
         Resolution|---                         |DUPLICATE
             Status|NEW                         |RESOLVED

--- Comment #1 from Christophe JAILLET <christophe.jaillet@xxxxxxxxxx> ---
2 of the fixes in your patch are already fixed, see bug 60086 fir details.

The 2 other fixes in your patch won't.

The removal of the 'if (script_err)...' is not an issue.
The compiler should fix it by itself.

The 'assert()' is no more.
Moreover, moving it this way would have broken our C89 compliancy. 

Anyway, thanks for the patch and report.
Should you find other similar issues, please report them. Static code checker
can greatly help in finding bugs.

*** This bug has been marked as a duplicate of bug 60086 ***

You are receiving this mail because:
You are the assignee for the bug.
To unsubscribe, e-mail: bugs-unsubscribe@xxxxxxxxxxxxxxxx
For additional commands, e-mail: bugs-help@xxxxxxxxxxxxxxxx