CVE-2020-26299 (JavaScript, published 2021-02-10T18:15Z)
Description: value: ftp-srv is an open-source FTP server designed to be simple yet configurable. In ftp-srv before version 4.4.0 there is a path-traversal vulnerability. Clients of FTP servers utilizing ftp-srv hosted on Windows machines can escape the FTP user's defined root folder using the expected FTP commands, for example, CWD and UPDR. When windows separators exist within the path (`\`), `path.resolve` leaves the upper pointers intact and allows the user to move beyond the root folder defined for that user. We did not take that into account when creating the path resolve function. The issue is patched in version 4.4.0 (commit 457b859450a37cba10ff3c431eb4aa67771122e3).
Severity: MEDIUM (CVSS v3 base score: 9.6)
Weakness: CWE-22 \u2014 Improper Limitation of a Pathname to a Restricted Directory ('Path Traversal')
The product uses external input to construct a pathname that is intended to identify a file or directory that is located underneath a restricted parent directory, but the product does not properly neutralize special elements within the pathname that can cause the pathname to resolve to a location that is outside of the restricted directory.
Repository: https://github.com/autovance/ftp-srv
Files changed:
src/fs.js
test/fs.spec.js
test/start.js
Commit message: fix(fs): check resolved path against root (#224)

* fix(fs): check resolved path against root

This should prevent paths from being resolved above the root.

Should affect all commands that utilize the FS functions.

Fixes #167

* test: use __dirname for relative certs

* fix: improve path resolution

* chore: remove unused package

* fix: normalize resolve path if absolute

Otherwise join will normalize

Co-authored-by: Tyler Stewart <tyler@autovance.com>

Vulnerable code:
    this.cwd = nodePath.normalize(cwd ? nodePath.join(nodePath.sep, cwd) : nodePath.sep);
    const clientPath = (() => {
      path = nodePath.normalize(path);
      if (nodePath.isAbsolute(path)) {
        return nodePath.join(path);
      } else {
        return nodePath.join(this.cwd, path);
      }
    })();

    const fsPath = (() => {
      const resolvedPath = nodePath.join(this.root, clientPath);
      return nodePath.resolve(nodePath.normalize(nodePath.join(resolvedPath)));
    })();
      const result = fs._resolvePath();
    it('cannot escape root', function () {
    key: fs.readFileSync(`${process.cwd()}/test/cert/server.key`),
    cert: fs.readFileSync(`${process.cwd()}/test/cert/server.crt`),
    ca: fs.readFileSync(`${process.cwd()}/test/cert/server.csr`)
    resolve({root: require('os').homedir()});

Fixed code:
const UNIX_SEP_REGEX = /\//g;
const WIN_SEP_REGEX = /\\/g;

    this.cwd = nodePath.normalize((cwd || '/').replace(WIN_SEP_REGEX, '/'));
    // Unix separators normalize nicer on both unix and win platforms
    const resolvedPath = path.replace(WIN_SEP_REGEX, '/');

    // Join cwd with new path
    const joinedPath = nodePath.isAbsolute(resolvedPath)
      ? nodePath.normalize(resolvedPath)
      : nodePath.join('/', this.cwd, resolvedPath);

    // Create local filesystem path using the platform separator
    const fsPath = nodePath.resolve(nodePath.join(this.root, joinedPath)
      .replace(UNIX_SEP_REGEX, nodePath.sep)
      .replace(WIN_SEP_REGEX, nodePath.sep));

    // Create FTP client path using unix separator
    const clientPath = joinedPath.replace(WIN_SEP_REGEX, '/');
      const result = fs._resolvePath('.');
    it('gets correct relative path', function () {
      const result = fs._resolvePath('other');
      expect(result).to.be.an('object');
      expect(result.clientPath).to.equal(
        nodePath.normalize('/file/1/2/3/other'));
      expect(result.fsPath).to.equal(
        nodePath.resolve('/tmp/ftp-srv/file/1/2/3/other'));
    });

    it('cannot escape root - unix', function () {
    it('cannot escape root - win', function () {
      const result = fs._resolvePath('.\\..\\..\\..\\..\\..\\..\\');
      expect(result).to.be.an('object');
      expect(result.clientPath).to.equal(
        nodePath.normalize('/'));
      expect(result.fsPath).to.equal(
        nodePath.resolve('/tmp/ftp-srv'));
    });

    it('cannot escape root - backslash prefix', function () {
      const result = fs._resolvePath('\\/../../../../../../');
      expect(result).to.be.an('object');
      expect(result.clientPath).to.equal(
        nodePath.normalize('/'));
      expect(result.fsPath).to.equal(
        nodePath.resolve('/tmp/ftp-srv'));
    });

    key: fs.readFileSync(`${__dirname}/cert/server.key`),
    cert: fs.readFileSync(`${__dirname}/cert/server.crt`),
    ca: fs.readFileSync(`${__dirname}/cert/server.csr`)
    resolve({root: __dirname});