handle return value of krb5_storage_{read,write}

git-svn-id: svn://svn.h5l.se/heimdal/trunk/heimdal@24011 ec53bebd-3082-4978-b11e-865c3cabbd6b

diff --git a/kcm/protocol.c b/kcm/protocol.c
index 0d6d2625c9e19cf8647102c05f52dfda26989b4a..db3d0d2112186f0eef664c96df38dab8e00fd46c 100644 (file)
--- a/kcm/protocol.c
+++ b/kcm/protocol.c
@@ -440,9 +440,12 @@ kcm_op_get_first(krb5_context context,
        return ret;
 
     for (creds = ccache->creds ; creds ; creds = creds->next) {
-       ret = krb5_storage_write(response, &creds->uuid, sizeof(creds->uuid));
-       if (ret)
+       ssize_t sret;
+       sret = krb5_storage_write(response, &creds->uuid, sizeof(creds->uuid));
+       if (sret != sizeof(creds->uuid)) {
+           ret = ENOMEM;
            break;
+       }
     }
 
     kcm_release_ccache(context, &ccache);
@@ -470,6 +473,7 @@ kcm_op_get_next(krb5_context context,
     char *name;
     struct kcm_creds *c;
     uuid_t uuid;
+    ssize_t sret;
 
     ret = krb5_ret_stringz(request, &name);
     if (ret)
@@ -483,9 +487,12 @@ kcm_op_get_next(krb5_context context,
     if (ret)
        return ret;
 
-    ret = krb5_storage_read(request, &uuid, sizeof(uuid));
-    if (ret)
-       return ret;
+    sret = krb5_storage_read(request, &uuid, sizeof(uuid));
+    if (sret != sizeof(uuid)) {
+       kcm_release_ccache(context, &ccache);
+       krb5_clear_error_message(context);
+       return KRB5_CC_IO;
+    }
 
     c = kcm_ccache_find_cred_uuid(context, ccache, uuid);
     if (c == NULL) {