You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by co...@apache.org on 2017/07/21 11:27:33 UTC

[46/50] [abbrv] directory-kerby git commit: Use readFully instead of read for being more robust, according to Steve review

Use readFully instead of read for being more robust, according to Steve review


Project: http://git-wip-us.apache.org/repos/asf/directory-kerby/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-kerby/commit/f904cdab
Tree: http://git-wip-us.apache.org/repos/asf/directory-kerby/tree/f904cdab
Diff: http://git-wip-us.apache.org/repos/asf/directory-kerby/diff/f904cdab

Branch: refs/heads/gssapi
Commit: f904cdab36f64191911de54dac9735a1027e3351
Parents: 4f50e85
Author: Kai Zheng <ka...@intel.com>
Authored: Mon Jun 13 20:22:26 2016 +0800
Committer: Colm O hEigeartaigh <co...@apache.org>
Committed: Fri Jul 21 12:25:51 2017 +0100

----------------------------------------------------------------------
 .../main/java/org/apache/kerby/kerberos/kerb/KrbInputStream.java | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/directory-kerby/blob/f904cdab/kerby-kerb/kerb-util/src/main/java/org/apache/kerby/kerberos/kerb/KrbInputStream.java
----------------------------------------------------------------------
diff --git a/kerby-kerb/kerb-util/src/main/java/org/apache/kerby/kerberos/kerb/KrbInputStream.java b/kerby-kerb/kerb-util/src/main/java/org/apache/kerby/kerberos/kerb/KrbInputStream.java
index 9611fe0..1e0729d 100644
--- a/kerby-kerb/kerb-util/src/main/java/org/apache/kerby/kerberos/kerb/KrbInputStream.java
+++ b/kerby-kerb/kerb-util/src/main/java/org/apache/kerby/kerberos/kerb/KrbInputStream.java
@@ -73,9 +73,7 @@ public abstract class KrbInputStream extends DataInputStream {
         }
 
         byte[] data = new byte[len];
-        if (read(data) == -1) {
-            throw new IOException("Unexpected end of stream");
-        }
+        readFully(data);
 
         return data;
     }