https://db.gcve.eu Contains only the most 10 recent entries. http://www.rssboard.org/rss-specification python-feedgen en Thu, 22 Oct 2026 03:28:43 +0000 https://db.gcve.eu/vuln/ghsa-p634-w6r4-rjp2 <p><strong>Affected:</strong> npm: adm-zip</p> <p>### Summary</p> <p>A ZIP file can contain two entries with the identical name. adm-zip keeps both in its internal entry list, but its name-lookup table only retains the last one written. `getEntry(name)` and `extractAllTo()` walk these two different internal structures, so they can each resolve a duplicate name to a *different* entry. An application that validates a named entry&#39;s contents via `getEntry()` before trusting an archive, then extracts the whole archive, can end up approving one file&#39;s content while a different file&#39;s bytes are what actually land on disk under that name.</p> <p>### Details - `zipFile.js:58-83` retains both entries in `entryList` but overwrites `entryTable[name]` with only the last one written. - `adm-zip.js:83-95,658-663` uses `entryTable` for `getEntry()` lookups — returns the *last* duplicate. - `adm-zip.js:769-914` iterates `entryList` for extraction — writes the *first* duplicate (sync, default overwrite policy).</p> <p>### PoC ```js const AdmZip = require(&#39;adm-zip&#39;); const z = new AdmZip({ noSort: true }); z.addFile(&#39;a.txt&#39;, Buffer.from(&#39;FIRST&#39;)); z.addFile(&#39;b.txt&#39;, Buffer.from(&#39;SECOND&#39;)); const raw = Buffer.from(z.toBuffer()); // rename the a.txt entry to b.txt directly in the raw bytes for (let at = raw.indexOf(&#39;a.txt&#39;); at &gt;= 0; at = raw.indexOf(&#39;a.txt&#39;, at + 5)) { raw.write(&#39;b.txt&#39;, at); } const parsed = new AdmZip(raw, { noSort: true }); const validated = parsed.getEntry(&#39;b.txt&#39;).getData().toString(); parsed.extractAllTo(outDir, false); // validated ==…</p> <p><strong>Affected:</strong> npm: adm-zip</p> <p>### Summary</p> <p>A ZIP file can contain two entries with the identical name. adm-zip keeps both in its internal entry list, but its name-lookup table only retains the last one written. `getEntry(name)` and `extractAllTo()` walk these two different internal structures, so they can each resolve a duplicate name to a *different* entry. An application that validates a named entry&#39;s contents via `getEntry()` before trusting an archive, then extracts the whole archive, can end up approving one file&#39;s content while a different file&#39;s bytes are what actually land on disk under that name.</p> <p>### Details - `zipFile.js:58-83` retains both entries in `entryList` but overwrites `entryTable[name]` with only the last one written. - `adm-zip.js:83-95,658-663` uses `entryTable` for `getEntry()` lookups — returns the *last* duplicate. - `adm-zip.js:769-914` iterates `entryList` for extraction — writes the *first* duplicate (sync, default overwrite policy).</p> <p>### PoC ```js const AdmZip = require(&#39;adm-zip&#39;); const z = new AdmZip({ noSort: true }); z.addFile(&#39;a.txt&#39;, Buffer.from(&#39;FIRST&#39;)); z.addFile(&#39;b.txt&#39;, Buffer.from(&#39;SECOND&#39;)); const raw = Buffer.from(z.toBuffer()); // rename the a.txt entry to b.txt directly in the raw bytes for (let at = raw.indexOf(&#39;a.txt&#39;); at &gt;= 0; at = raw.indexOf(&#39;a.txt&#39;, at + 5)) { raw.write(&#39;b.txt&#39;, at); } const parsed = new AdmZip(raw, { noSort: true }); const validated = parsed.getEntry(&#39;b.txt&#39;).getData().toString(); parsed.extractAllTo(outDir, false); // validated ==…</p> https://db.gcve.eu/vuln/ghsa-p634-w6r4-rjp2