jimzah/openafs
1
0
Fork 0
mirror of https://git.openafs.org/openafs.git synced 2025-01-31 05:27:44 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

userok: Don't double check for expiry

Browse Source 
rxkad_GetServerInfo returns an error if you attempt to get server
information using a connection whose tokens have already expired.

The additional check for expiry in the userOK routines is therefore
not required - they'll never be reached, because GetServerInfo will
already have errored out.

Change-Id: I7b1a97249d40b12a2aae4009971a9b676700870e
Reviewed-on: http://gerrit.openafs.org/2753
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
This commit is contained in:
Simon Wilkinson 2010-09-13 08:54:55 +01:00 committed by Derrick Brashear
parent 45b9938364
commit 10bb4b3103
1 changed files with 0 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/auth/userok.c
View File

@ -466,14 +466,6 @@ rxkadSuperUser(struct afsconf_dir *adir, struct rx_call *acall, char *namep)
if (code)
return 0; /* bogus connection/other error */
/* don't bother checking anything else if tix have expired */
#ifdef AFS_PTHREAD_ENV
if (exp < clock_Sec())
#else
if (exp < FT_ApproxTime())
#endif
return 0; /* expired tix */
return kerberosSuperUser(adir, tname, tinst, tcell, namep);
}