[Fix] handle SlowBuffers in node 0.10

Author: ljharb

index.js

@@ -20,6 +20,10 @@ var useFromArrayBuffer = useArrayBuffer && (Buffer.prototype instanceof Uint8Arr
 
 module.exports = function toBuffer(data, encoding) {
 	if (Buffer.isBuffer(data)) {
+		if (data.constructor && !('isBuffer' in data)) {
+			// probably a SlowBuffer
+			return Buffer.from(data);
+		}
 		return data;
 	}
 
@@ -93,9 +97,9 @@ module.exports = function toBuffer(data, encoding) {
 	if (
 		isArr || (
 			Buffer.isBuffer(data)
-				&& data.constructor
-				&& typeof data.constructor.isBuffer === 'function'
-				&& data.constructor.isBuffer(data)
+			&& data.constructor
+			&& typeof data.constructor.isBuffer === 'function'
+			&& data.constructor.isBuffer(data)
 		)
 	) {
 		return Buffer.from(data);

test/index.js

@@ -5,12 +5,31 @@ var availableTypedArrays = require('available-typed-arrays')();
 var forEach = require('for-each');
 var typedArrayBuffer = require('typed-array-buffer');
 var SafeBuffer = require('safe-buffer').Buffer;
+var SlowBuffer = require('buffer').SlowBuffer;
 
 var toBuffer = require('../');
 var fixtures = require('./fixtures.json');
 
 test('buffer returns buffer', function (t) {
-	t.deepEqual(toBuffer(new Buffer('hi')), new Buffer('hi'));
+	var result = toBuffer(new Buffer('hi'));
+
+	t.deepEqual(result, new Buffer('hi'));
+	t.ok(result instanceof Buffer, 'is a Buffer');
+	t.notOk(result instanceof SlowBuffer, 'not a SlowBuffer');
+
+	t.end();
+});
+
+test('SlowBuffer returns Buffer', { skip: SlowBuffer.name === 'deprecated' }, function (t) {
+	var slow = new SlowBuffer(2);
+	slow[0] = 7;
+	slow[1] = 8;
+	var result = toBuffer(slow);
+
+	t.deepEqual(result, new Buffer([7, 8]));
+	t.ok(result instanceof Buffer, 'is a Buffer');
+	t.notOk(result instanceof SlowBuffer, 'not a SlowBuffer');
+
 	t.end();
 });