Merge pull request #486 from KostyaSha/fixCompletion

KostyaSha · KostyaSha · commit 6b95338c24d7 · 2016-03-02T01:25:19.000+03:00
Fix NegativeArraySizeException in awaitCompletion()
diff --git a/src/main/java/com/github/dockerjava/core/command/FrameReader.java b/src/main/java/com/github/dockerjava/core/command/FrameReader.java
@@ -7,6 +7,8 @@
 import com.github.dockerjava.api.model.Frame;
 import com.github.dockerjava.api.model.StreamType;
 
+import javax.annotation.CheckForNull;
+
 /**
  * Breaks the input into frame. Similar to how a buffered reader would readLies.
  * <p/>
@@ -42,14 +44,16 @@ private static StreamType streamType(byte streamType) {
     /**
      * @return A frame, or null if no more frames.
      */
+    @CheckForNull
     public Frame readFrame() throws IOException {
 
         if (rawStreamDetected) {
-
             int read = inputStream.read(rawBuffer);
+            if (read == -1) {
+                return null;
+            }
 
             return new Frame(StreamType.RAW, Arrays.copyOf(rawBuffer, read));
-
         } else {
 
             byte[] header = new byte[HEADER_SIZE];
