From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Sun, 10 Jun 2007 15:22:49 +0000 (+0000)
Subject: STABLE14-rxkad-badticket-20070610
X-Git-Tag: openafs-stable-1_4_5-pre1~100
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=38733b808fa9721a73cc12b6d41e28076a450975;p=packages%2Fo%2Fopenafs.git

STABLE14-rxkad-badticket-20070610


In rxkad_CheckResponse, when checking the return value from tkt_CheckTimes,
the order is == 0, == -1, < -1, <= 0.  The <= 0 case is extraneous so
remove it. Both < -1 and <= 0 returned RXKADBADTICKET.


(cherry picked from commit f4bd2604222c1199fc0ece0eaa82c034fa50b977)
---

diff --git a/src/rxkad/rxkad_server.c b/src/rxkad/rxkad_server.c
index e2c54385b..2e91c6b5f 100644
--- a/src/rxkad/rxkad_server.c
+++ b/src/rxkad/rxkad_server.c
@@ -361,8 +361,6 @@ rxkad_CheckResponse(struct rx_securityClass *aobj,
 	return RXKADEXPIRED;
     else if (code < -1)
 	return RXKADBADTICKET;
-    else if (code <= 0)
-	return RXKADBADTICKET;
 
     code = fc_keysched(&sessionkey, sconn->keysched);
     if (code)