Two Review requests
John Zavgren
john.zavgren at oracle.com
Fri Sep 28 07:11:39 PDT 2012
Greetings:
I just posted the webrev images for two networking code bugs:
File Descriptor Leak:
http://cr.openjdk.java.net/~chegar/8000203/webrev.00/
(Jira bug ID number: 8000203)
Uninitialized memory:
http://cr.openjdk.java.net/~chegar/8000206/webrev.00/
(Jira bug ID: 8000206)
This change doesn't actually fix a bug... the original code initialized "optlen" before it was referenced, however, parfait (static code analysis) believes "optlen" MAY be used before initialization. I added the assignment statement to "spoof" parfait, and it no longer flags a bug. I assume it's better to put minor harmless tweaks in our code than to add state information to parfait, that would cause it to ignore certain "situations". That option seems complicated and dangerous.
Thanks!
John Zavgren
john.zavgren at oracle.com
More information about the net-dev
mailing list