* cc_stdio.c (krb5_fcc_next_cred): Use a krb5_error_code to hol the return

value from krb5_fcc_interpret.
(krb5_fcc_get_principal): Initialize return-value variable.
(krb5_fcc_initialize): Likewise.
(krb5_fcc_interpret): Change retval to a krb5_error_code.  Reorder cases for
consistency with cc_file.c.

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

diff --git a/src/lib/krb5/ccache/ChangeLog b/src/lib/krb5/ccache/ChangeLog
index 7ea9a9b12512edd4eaea798dcebf4806f8737fb0..64a7da580219c6c82c5713ebb659ae5596d5058f 100644 (file)
--- a/src/lib/krb5/ccache/ChangeLog
+++ b/src/lib/krb5/ccache/ChangeLog
@@ -1,3 +1,12 @@
+2002-08-14  Ken Raeburn  <raeburn@mit.edu>
+
+       * cc_stdio.c (krb5_fcc_next_cred): Use a krb5_error_code to hold
+       the return value from krb5_fcc_interpret.
+       (krb5_fcc_get_principal): Initialize return-value variable.
+       (krb5_fcc_initialize): Likewise.
+       (krb5_fcc_interpret): Change retval to a krb5_error_code.  Reorder
+       cases for consistency with cc_file.c.
+
 2002-08-09  Ken Raeburn  <raeburn@mit.edu>
 
        * cc_file.c: All functions except krb5_change_cache and

diff --git a/src/lib/krb5/ccache/cc_stdio.c b/src/lib/krb5/ccache/cc_stdio.c
index b2716cd4222db726d32f8d233c8a88d50dbe3a15..35531711bdd535fbcced6200aa8c2f01c1b0497a 100644 (file)
--- a/src/lib/krb5/ccache/cc_stdio.c
+++ b/src/lib/krb5/ccache/cc_stdio.c
@@ -708,7 +708,6 @@ krb5_fcc_read_authdatum(context, id, a)
      return kret;
     
 }
-/* end of former stdio/scc_read.c */
 #undef CHECK
 
 #define CHECK(ret) if (ret != KRB5_OK) return ret;
@@ -1312,6 +1311,7 @@ krb5_fcc_skip_principal(context, id)
      return KRB5_OK;
 }
 
+
 /*
  * Modifies:
  * id
@@ -1330,7 +1330,7 @@ krb5_fcc_initialize(context, id, princ)
    krb5_ccache id;
    krb5_principal princ;
 {
-     krb5_error_code kret;
+     krb5_error_code kret = 0;
 
      kret = krb5_fcc_open_file (context, id, FCC_OPEN_AND_ERASE);
      if (kret < 0)
@@ -1580,9 +1580,9 @@ krb5_fcc_next_cred(context, id, cursor, creds)
      fcursor = (krb5_fcc_cursor *) *cursor;
      ret = fseek(((krb5_fcc_data *) id->data)->file, fcursor->pos, 0);
      if (ret < 0) {
-        ret = krb5_fcc_interpret(context, errno);
-        MAYBE_CLOSE (context, id, ret);
-        return ret;
+        kret = krb5_fcc_interpret(context, errno);
+        MAYBE_CLOSE (context, id, kret);
+        return kret;
      }
 
      kret = krb5_fcc_read_principal(context, id, &creds->client);
@@ -1612,9 +1612,8 @@ krb5_fcc_next_cred(context, id, cursor, creds)
      cursor = (krb5_cc_cursor *) fcursor;
 
 lose:
-     if (kret != KRB5_OK) {
+     if (kret != KRB5_OK)
         krb5_free_cred_contents(context, creds);
-     }
      MAYBE_CLOSE (context, id, kret);
      return kret;
 }
@@ -1638,8 +1637,8 @@ krb5_fcc_end_seq_get(context, id, cursor)
    krb5_ccache id;
    krb5_cc_cursor *cursor;
 {
-/*    MAYBE_CLOSE (context, id, ret); */
-
+/*
+    MAYBE_CLOSE (context, id, kret); */
     krb5_xfree((krb5_fcc_cursor *) *cursor);
 
     return 0;
@@ -1785,13 +1784,12 @@ krb5_fcc_get_principal(context, id, princ)
    krb5_ccache id;
    krb5_principal *princ;
 {
-     krb5_error_code kret;
+     krb5_error_code kret = KRB5_OK;
 
      MAYBE_OPEN (context, id, FCC_OPEN_RDONLY);
 
      kret = krb5_fcc_skip_header(context, id);
      if (kret) goto done;
-
      kret = krb5_fcc_read_principal(context, id, princ);
 
 done:
@@ -1812,6 +1810,7 @@ krb5_fcc_retrieve(context, id, whichfields, mcreds, creds)
                                          mcreds, creds);
 }
 
+
 /*
  * Modifies:
  * the file cache
@@ -1836,8 +1835,9 @@ krb5_fcc_store(context, id, creds)
      MAYBE_OPEN (context, id, FCC_OPEN_RDWR);
 
      ret = fseek(((krb5_fcc_data *) id->data)->file, 0, 2);
-     if (ret < 0)
+     if (ret < 0) {
          return krb5_fcc_interpret(context, errno);
+     }
 
      ret = krb5_fcc_store_principal(context, id, creds->client);
      TCHECK(ret);
@@ -1861,12 +1861,12 @@ krb5_fcc_store(context, id, creds)
      TCHECK(ret);
 
 lose:
-
      MAYBE_CLOSE (context, id, ret);
      return ret;
 #undef TCHECK
 }
 
+
 /*
  * Requires:
  * id is a cred cache returned by krb5_fcc_resolve or
@@ -1909,11 +1909,8 @@ krb5_fcc_interpret(context, errnum)
     krb5_context context;
     int errnum;
 {
-    register int retval;
+    register krb5_error_code retval;
     switch (errnum) {
-#ifdef ELOOP
-    case ELOOP:                                /* Bad symlink is like no file. */
-#endif
     case ENOENT:
        retval = KRB5_FCC_NOFILE;
        break;
@@ -1923,6 +1920,9 @@ krb5_fcc_interpret(context, errnum)
     case EISDIR:                       /* Mac doesn't have EISDIR */
 #endif
     case ENOTDIR:
+#ifdef ELOOP
+    case ELOOP:                                /* Bad symlink is like no file. */
+#endif
 #ifdef ETXTBSY
     case ETXTBSY:
 #endif