8012923: [parfait] File Descriptor Leak in jdk/src/windows/demo/jvmti/hprof/hprof_md.c

Reviewed-by: chegar, sla, sspitsyn, mgronlun
This commit is contained in:
Peter Allwin 2013-09-30 14:28:53 +02:00
parent 938247e8f4
commit 2f5f83e38c

View File

@ -82,9 +82,6 @@ md_connect(char *hostname, unsigned short port)
struct sockaddr_in s;
int fd;
/* create a socket */
fd = (int)socket(AF_INET, SOCK_STREAM, 0);
/* find remote host's addr from name */
if ((hentry = gethostbyname(hostname)) == NULL) {
return -1;
@ -97,8 +94,15 @@ md_connect(char *hostname, unsigned short port)
s.sin_port = htons(port);
s.sin_family = AF_INET;
/* create a socket */
fd = (int)socket(AF_INET, SOCK_STREAM, 0);
if (INVALID_SOCKET == fd) {
return 0;
}
/* now try connecting */
if (-1 == connect(fd, (struct sockaddr*)&s, sizeof(s))) {
if (SOCKET_ERROR == connect(fd, (struct sockaddr*)&s, sizeof(s))) {
closesocket(fd);
return 0;
}
return fd;