Merge pull request #1628 from michaeljbishop/pread-fix2

Fixes in pread()

1 files changed, 3 insertions, 0 deletions

diff --git a/src/library_memfs.js b/src/library_memfs.js
index 354f5e95..28178d9f 100644
--- a/src/library_memfs.js
+++ b/src/library_memfs.js
@@ -177,7 +177,10 @@ mergeInto(LibraryManager.library, {
     stream_ops: {
       read: function(stream, buffer, offset, length, position) {
         var contents = stream.node.contents;
+        if (position >= contents.length)
+          return 0;
         var size = Math.min(contents.length - position, length);
+        assert(size >= 0);
 #if USE_TYPED_ARRAYS == 2
         if (size > 8 && contents.subarray) { // non-trivial, and typed array
           buffer.set(contents.subarray(position, position + size), offset);