Need reviewer: a couple of hprof fixes for Parfait

[Dmitry Samersoff]

Changes looks good for me.

but logic below always was cryptic for me:

 139     /* now try connecting */
 140     if (-1 == connect(fd, (struct sockaddr*)&s, sizeof(s))) {
 141         (void)close(fd);
 142         return 0;
 143     }

we returns -1 if gethostbyname fail, but 0 if we can't connect to host.

-Dmitry

On 2012-06-12 06:48, Kelly O'Hair wrote:
> 
> Need reviewer.
> 
> I was asked to look at some Parfait errors in hprof code:
> 
> 7176138: Fixes for missing close() calls and possible null pointer reference instead of fatal error
> http://cr.openjdk.java.net/~ohair/openjdk8/parfait_hprof_fixes/webrev/
> 
> -kto
> 


-- 
Dmitry Samersoff
Java Hotspot development team, SPB04
* There will come soft rains ...