Incremental fix for AsyncHttpClient#128. ResumableRandomAccessFileListener::onBytesReceived() was passing in the array which backed the ByteBuffer, but didn't use the array offset for the initial position.

Added additional logic to perform a copy before the file write if the ByteBuffer provided returned false when calling ByteBuffer::hasArray().

Author: rlubke

api/src/main/java/com/ning/http/client/extra/ResumableRandomAccessFileListener.java

@@ -37,7 +37,15 @@ public ResumableRandomAccessFileListener(RandomAccessFile file) {
      */
     public void onBytesReceived(ByteBuffer buffer) throws IOException {
         file.seek(file.length());
-        file.write(buffer.array());
+        if (buffer.hasArray()) {
+            file.write(buffer.array(), buffer.arrayOffset(), buffer.remaining());
+        } else { // if the buffer is direct or backed by a String...
+            byte[] b = new byte[buffer.remaining()];
+            int pos = buffer.position();
+            buffer.get(b);
+            buffer.position(pos);
+            file.write(b);
+        }
     }
 
     /**