Bug in onSymlink

[znewsham]

There is a bug when copying a directory containing a symlink onto itself (e.g., if you were making an rsync type utility):

If options.dereference is not set, then resolvedSrc and resolvedDest will always be equal - since they are the value of the symlink.

        if (stat.isSrcSubdir(resolvedSrc, resolvedDest)) {
          return cb(new Error(`Cannot copy '${resolvedSrc}' to a subdirectory of itself, '${resolvedDest}'.`))
        }

this code needs to be changed to

if (options.dereference && stat.isSrcSubdir(resolvedSrc, resolvedDest)) {

Probably a similar change is required here:

if (destStat.isDirectory() && stat.isSrcSubdir(resolvedDest, resolvedSrc)) {

[RyanZim]

Confirmed bug; but we need something more complicated than your proposed fix. We can't simply ignore these checks when we're not dereferencing; just one example case:

node-fs-extra/lib/copy/__tests__/copy-prevent-copying-into-itself.test.js

Line 351 in a277cbb

it('should error when resolved dest path is a subdir of resolved src path', done => {

[Germanen]

Hallo zusammen 👋

Ich bin gerade beim Testen von fs-extra auf Issue #1038 gestoßen (Symlinks mit relativen Pfaden beim zweiten ensureSymlink()-Aufruf) – und im Zuge dessen bin ich auch auf diesen Fehler (#1027) hier gestoßen.

Ich habe das Verhalten reproduzieren können, bei dem zwei Symlinks auf dieselbe Datei existieren, und copy() bzw. die interne isSrcSubdir()-Logik fälschlicherweise denkt, der Zielpfad sei ein Unterverzeichnis des Quellpfads. Dabei handelt es sich bei beiden um Symlinks, die auf dieselbe Datei zeigen, aber keine echten Verzeichnisse sind.

🔍 Vermutung zur Ursache:

isSrcSubdir() scheint nur den Pfadnamen zu vergleichen, aber nicht den aufgelösten echten Pfad (via fs.realpath() oder fs.promises.realpath). Das führt dazu, dass bei Links wie:

• /some/path/link1 → /real/file.txt
• /some/other/link2 → /real/file.txt

…fälschlicherweise eine „Subdirectory“-Beziehung erkannt wird, obwohl beide nur Symlinks zur selben Datei sind.

---

💡 Vorschlag zur Lösung (theoretisch):

Innerhalb von isSrcSubdir():

1. Beide Pfade mit realpath auflösen
2. Dann erst auf Subdirectory prüfen
3. Oder im Fall von zwei Symlinks auf die gleiche Datei → Skip / erlauben

---

🧪 Beispiel (Pseudocode zum Reproduzieren)

const fs = require('fs-extra');
const path = require('path');

const base = path.resolve('test-copy-links');
const realFile = path.join(base, 'real.txt');
const link1 = path.join(base, 'link1');
const link2 = path.join(base, 'link2');

await fs.outputFile(realFile, 'Hello');
await fs.ensureSymlink(realFile, link1);
await fs.ensureSymlink(realFile, link2);

await fs.copy(link1, link2); // throws in isSrcSubdir