Removed NOP line of code from krb5_fcc_next_cred()
authorAlexandra Ellwood <lxs@mit.edu>
Wed, 3 Oct 2007 20:13:55 +0000 (20:13 +0000)
committerAlexandra Ellwood <lxs@mit.edu>
Wed, 3 Oct 2007 20:13:55 +0000 (20:13 +0000)
cursor = (krb5_cc_cursor *) fcursor;

does nothing for a function with a prototype

krb5_fcc_next_cred(krb5_context context, krb5_ccache id,
krb5_cc_cursor *cursor, krb5_creds *creds)

The correct line of code would be:

*cursor = (krb5_cc_cursor) fcursor;

Except it isn't necessary because fcursor isn't modified
by krb5_fcc_next_cred().  Looks like code was copied from
krb5_fcc_start_seq_get() and since the line was a NOP
no one noticed it.

ticket: new

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

src/lib/krb5/ccache/cc_file.c

index 49ba479434438175dd9d9f7d1e2c229e91a2daae..9abf93f3de962255c496029d15e415b60c5a36d9 100644 (file)
@@ -1903,7 +1903,6 @@ krb5_fcc_next_cred(krb5_context context, krb5_ccache id, krb5_cc_cursor *cursor,
      TCHECK(kret);
      
      fcursor->pos = fcc_lseek(d, (off_t) 0, SEEK_CUR);
-     cursor = (krb5_cc_cursor *) fcursor;
 
 lose:
      MAYBE_CLOSE (context, id, kret);