(this may have caused the spin-looping I saw...)

	* krb524d.c (do_connection): if recvfrom failed, just return,
	don't try to use the (likely invalid) sockaddr in a reply message.

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

diff --git a/src/krb524/ChangeLog b/src/krb524/ChangeLog
index b361dff33d77d9c1ab5851dc6eb438c6137a809d..a46711842efc0b9a85c46bb8e764a0a0475c2880 100644 (file)
--- a/src/krb524/ChangeLog
+++ b/src/krb524/ChangeLog
@@ -1,3 +1,8 @@
+Mon Sep 25 00:15:47 1995  Mark Eichin  <eichin@cygnus.com>
+
+       * krb524d.c (do_connection): if recvfrom failed, just return,
+       don't try to use the (likely invalid) sockaddr in a reply message.
+
 Thu Sep 14 17:36:11 1995  Mark Eichin  <eichin@cygnus.com>
 
        * krb524d.c (usage): needs to get context to pass to

diff --git a/src/krb524/krb524d.c b/src/krb524/krb524d.c
index 889a11b811aa09701d0735f3ee36c1199362a973..df9a15c067c92d4301948d3a8b9b8a82042b2c9d 100644 (file)
--- a/src/krb524/krb524d.c
+++ b/src/krb524/krb524d.c
@@ -268,8 +268,9 @@ krb5_error_code do_connection(s, context)
      saddrlen = sizeof(struct sockaddr);
      ret = recvfrom(s, msgdata.data, msgdata.length, 0, &saddr, &saddrlen);
      if (ret < 0) {
-         ret = errno;
-         goto error;
+       /* if recvfrom fails, we probably don't have a valid saddr to 
+         use for the reply, so don't even try to respond. */
+       return errno;
      }
      if (debug)
          printf("message received\n");