Merge pull request #531 from NickAllmakerOpsCompass/fix_fileattr

5saviahv · web-flow · commit ca54d1ad4807 · 2024-08-18T10:02:05.000+03:00
Fixing preserving linux file permissions
diff --git a/headers/entryHeader.js b/headers/entryHeader.js
@@ -183,7 +183,7 @@ module.exports = function () {
 
         // get Unix file permissions
         get fileAttr() {
-            return uint16(_attr >> 16) & 0xfff;
+            return (_attr || 0) >> 16 & 0xfff;
         },
 
         get offset() {
diff --git a/test/header.js b/test/header.js
@@ -142,6 +142,16 @@ describe("headers", () => {
             expect(head.toJSON()).to.eql(headerdata);
         });
 
+        it("handles fileAttr when attr above 0x80000000", () => {
+            const attr = 0x81E80000;
+
+            const head = new centralHeader();
+            head.loadFromBinary(readBuf);
+            head.attr = attr;
+
+            expect(head.fileAttr).to.equal(0x01E8);
+        });
+
         it("read binary and create new binary from it, they have to be equal", () => {
             const head = new centralHeader();
             head.loadFromBinary(readBuf);
