Fix Buffer.alloc bug #234

A previous pull request was not compatible with Node 4.x
cthackers · Jun 22, 2018 · ca2afb5 · ca2afb5
1 parent 80d259f
commit ca2afb5
Showing 1 changed file with 6 additions and 1 deletion.
diff --git a/zipFile.js b/zipFile.js
@@ -267,7 +267,12 @@ module.exports = function (/*String|Buffer*/input, /*Number*/inputType) {
 						entry.header.offset = dindex;
 						// data header
 						var dataHeader = entry.header.dataHeaderToBinary();
-						var postHeader = Buffer.alloc(name);
+						var postHeader;
+						try {
+							postHeader = Buffer.alloc(name.length, name);  // using alloc will work on node  5.x+
+						} catch(e){
+							postHeader = new Buffer(name); // use deprecated method if alloc fails...
+						}
 						var dataLength = dataHeader.length + postHeader.length + compressedData.length;
 
 						dindex += dataLength;