From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Mon, 27 Oct 2003 19:59:22 +0000 (+0000)
Subject:     * sendto_kdc.c: Sockets must be closed with closesocket() instead
X-Git-Tag: krb5-1.4-beta1~730
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=46c581318f28954b76b4721fdbdfc286ca8aa2bf;p=krb5.git

* sendto_kdc.c: Sockets must be closed with closesocket() instead
      of close() for proper socket deallocation on systems which do
      not use file descriptors for sockets.  i.e., Windows.

ticket: 1973
target_version: 1.3.2
tags: pullup
status: resolved

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@15844 dc483132-0cff-0310-8789-dd5450dbe970
---

diff --git a/src/lib/krb5/os/ChangeLog b/src/lib/krb5/os/ChangeLog
index e0bc0cd90..017b9ae7e 100644
--- a/src/lib/krb5/os/ChangeLog
+++ b/src/lib/krb5/os/ChangeLog
@@ -1,3 +1,9 @@
+2003-10-27  Jeffrey Altman <jaltman@mit.edu>
+
+    * sendto_kdc.c: sockets must be closed with closesocket() and 
+      and not close() in order to ensure portability among different
+      operating systems.
+
 2003-08-21  Ken Raeburn  <raeburn@mit.edu>
 
 	* dnssrv.c: New file; split out DNS SRV RR query support...
diff --git a/src/lib/krb5/os/sendto_kdc.c b/src/lib/krb5/os/sendto_kdc.c
index 0f5b9f275..8e0f719a4 100644
--- a/src/lib/krb5/os/sendto_kdc.c
+++ b/src/lib/krb5/os/sendto_kdc.c
@@ -1073,7 +1073,7 @@ krb5int_sendto (krb5_context context, const krb5_data *message,
 egress:
     for (i = 0; i < n_conns; i++) {
 	if (conns[i].fd != INVALID_SOCKET)
-	    close(conns[i].fd);
+	    closesocket(conns[i].fd);
 	if (conns[i].state == READING
 	    && conns[i].x.in.buf != 0
 	    && conns[i].x.in.buf != udpbuf)