Guard against for stream.read returning null in nodejs

Author: strobejb

src/io/load.js

@@ -28,8 +28,15 @@ jBinary.loadData = promising(function (source, callback) {
 	if (NODE && is(source, ReadableStream)) {
 		var buffers = [];
 		source
-			.on('readable', function () { buffers.push(this.read()) })
-			.on('end', function () { callback(null, Buffer.concat(buffers)) })
+			.on('readable', function () {
+				var buf = this.read();
+				if(buf) {
+					buffers.push(buf);
+				}
+			})
+			.on('end', function () {
+				callback(null, Buffer.concat(buffers));
+			})
 			.on('error', callback)
 		;
 	} else

test/test.js

@@ -204,6 +204,17 @@ suite('Loading data', function () {
 		});
 	}
 
+	if (hasNodeRequire && require('stream').Readable) {
+		test('from file-based readableStream', function (done) {
+			var stream = require('fs').createReadStream(localFileName);
+			jBinary.loadData(stream, function (err, data) {
+				assert.notOk(err, err);
+				assert.equal(data.byteLength || data.length, 512);
+				done();
+			});
+		});
+	}
+
 	test('with explicit typeset object', function (done) {
 		var typeSet = {
 			IS_CORRECT_TYPESET: true