News for hackers. Stuff that matters.
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by lioeters on Wednesday January 08, 2020 @05:15AM (#21990069)
The fact that "bin" files of globally installed dependencies can overwrite each other silently - it's an understandable oversight, and I'll be glad if npm/yarn notifies me in the future.

What surprised me was that the "bin" names can be a relative or absolute path, like:

"bin": { "/usr/bin/date": "./fake-date" }

That is scary! I'm guessing these paths will be sanitized in the newest versions of package managers, but that seems like it was a pretty big security hole.

As others have commented, I prefer not to install things globally - mostly locally linked packages of my own. I believe projects should not require or recommend globally installed commands, everything it needs for building should be in its local node_modules folder.

Re:Understand filesystem takeover vulnerabilities in NPM Jav... by tomxor on Wednesday January 08, 2020 @07:45AM (#21990874)
> What surprised me was that the "bin" names can be a relative or absolute path, like:

> "bin": { "/usr/bin/date": "./fake-date" }

This will only possibly work if you install it with root privileges... at which point you need to have a reasonable level of trust of the script you are running. It's unreasonable to give the same level of trust to npm packages that you do your OS package manager since anyone can publish an npm package without anyone elses permission.

For this reason I think npm's defaults are a really shitty choice (global prefix under root). I always change the global prefix to live in my user directory for this reason so I can npm install -g without root:

    sudo npm config -g set prefix '~/.local'
You will need to add the local bin to your PATH (if it doesn't already contain it)

    PATH="$HOME/.local/bin:$PATH"
The order above is important, local must come before existing PATHs for lower precedence so that it cannot override higher privilege bins... e.g so that if an npm package added a bin for '~/.local/bin/cat', it would never be called implicitly since '/usr/bin/cat' exists.
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by ryukafalz on Wednesday January 08, 2020 @08:34AM (#21991248)
>The order above is important, local must come before existing PATHs for lower precedence so that it cannot override higher privilege bins

You've got the order reversed there - the first item found in your PATH will be used, and therefore earlier directories in your PATH have higher precedence:

  jfred@lambdacrypt ~$ cat dir1/pathtest 
  #!/run/current-system/profile/bin/bash
  echo "script in dir1"
  jfred@lambdacrypt ~$ cat dir2/pathtest 
  #!/run/current-system/profile/bin/bash
  echo "script in dir2"
  jfred@lambdacrypt ~$ PATH=$HOME/dir1:$HOME/dir2 pathtest
  script in dir1
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by tomxor on Wednesday January 08, 2020 @09:35PM (#21998019)
You are correct my mistake.
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by inimino on Wednesday January 08, 2020 @08:59AM (#21991452)
I don't understand the point in using ~/.local/bin and then putting it in your PATH. The point of treating /usr/local/bin and the other root-owned binary directories specially is that they are in your path, which means that if you type "ls" you want to run "ls" and not something untrusted. For that reason, anything you put in your PATH needs to be treated exactly with as much care as anything you install in /bin. The only difference might be that you don't affect other users on the same system, in a multi-user environment (which is increasingly rare). It might be convenient if you wanted to be able to clean up an entire installation by deleting one directory, but it's not a security feature.

Don't install binaries anywhere in your path if you don't trust them or know where they came from. (Also, sibling is correct that first location found wins when searching PATH for a command name.)

Also, it's not unreasonable to trust npm packages just like OS packages if you know who released it. NPM names are first-come, first-served, and once allocated, they are permanent. If you know and trust the person who released the package, there's nothing wrong with trusting the package. This was, in fact, the norm when npm was new and the community was small and most people knew each other. The only problem is with treating npm like a vetted distro like Debian, and assuming that anything on npm is safe to install without any connection to the author (or 1001 dependencies). That's where everything started to go sideways.

Re:Understand filesystem takeover vulnerabilities in NPM Jav... by tomxor on Wednesday January 08, 2020 @09:44PM (#21998062)
> anything you put in your PATH needs to be treated exactly with as much care as anything you install in /bin. [...] it's not a security feature.

The parent was highlighting how the article points out you can overwrite root files:

> "bin": { "/usr/bin/date": "./fake-date" }

This is not possible with ~/.local

I wouldn't call this a "security feature", but I would call it prudent to not hand over root access to npm packages.

> Also, it's not unreasonable to trust npm packages just like OS packages if you know who released it. NPM names are first-come, first-served, and once allocated, they are permanent. If you know and trust the person who released the package, there's nothing wrong with trusting the package.

This is not the problem with npm, the problem is dependencies, and their dependencies and so on. I certainly will never be able to personally vet the 1000 deep nest of dependencies in an npm package I may have to use.

Re:Understand filesystem takeover vulnerabilities in NPM Jav... by WorldMaker on Wednesday January 08, 2020 @03:38PM (#21995529)
It seems another reason to prefer `npx` over `npm install -g`.
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by inimino on Wednesday January 08, 2020 @09:18AM (#21991626)
"bin": { "/usr/bin/date": "./fake-date" }

The only way that can do anything scary, is if you are running `sudo npm install -g <untrusted package>`, and at that point you should already be terrified. Don't do that.

This is no different from any other package you install from any third party. The author of the package is the relevant third party here, npm is just a host, they don't do your security homework for you. (Think "github", not "Debian".)

Re:Understand filesystem takeover vulnerabilities in NPM Jav... by inimino on Wednesday January 08, 2020 @05:35AM (#21990173)
Well yes, if you're running as a privileged user and you install a package globally, it can install global binaries.

Don't install packages you can't trust.

Re:Understand filesystem takeover vulnerabilities in NPM Jav... by imtringued on Wednesday January 08, 2020 @06:51AM (#21990558)
That eliminates almost all NPM packages.
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by tomxor on Wednesday January 08, 2020 @07:47AM (#21990886)
Yes exactly, which is why npm's default root prefix is a poor choice, see my other comment for a slightly safer alternative.
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by inimino on Wednesday January 08, 2020 @09:11AM (#21991538)
No, it's not. If you want to install a package like babel or webpack, or npm itself, and you want it to install manpages and commands like any real Unix program, you want it in /usr/local, and this is exactly what /usr/local is for.

These are the usecases that -g is for, and that's the only thing it's for: utilities that you want to add to your system, not mere runtime JS dependencies. NPM has this feature because in the beginning people (including me) wanted it to be more than just a manager for JS files under a build directory, because we were using it to build shared useful utilities, not just runtime dependencies.

This is a problem that has played out again and again with every programming language that has its own package installer, because setting the boundaries and interacting with the host system correctly while satisfying everyone's needs is challenging.

Re:Understand filesystem takeover vulnerabilities in NPM Jav... by tomxor on Wednesday January 08, 2020 @09:26PM (#21997978)
Using ~/.local gives you all those things without giving root access, there is no good reason to stick everything in /usr/local especially with something like npm which is wide open for anyone to publish anything instantly.
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by inimino on Friday January 10, 2020 @12:27AM (#22008539)
Yes, and then the question is, should you be putting programs you don't trust at all in your PATH?
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by tomxor on Friday January 10, 2020 @04:52PM (#22015996)
Yes, if i trust them enough to add and not replace binaries.
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by inimino on Sunday January 12, 2020 @10:25PM (#22031579)
But once you run any of those binaries, it's free to replace any of your other (non-root-owned) binaries in `~/.local`. So the trust you're talking about seems to be a preference, but I don't think there's any particularly plausible threat model that makes one of these safer than the other.
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by inimino on Wednesday January 08, 2020 @08:49AM (#21991347)
Indeed it does.
Re:Understand filesystem takeover vulnerabilities in NPM Jav... by SamHasler on Wednesday January 08, 2020 @09:07AM (#21991497)
> I'll be glad if npm/yarn notifies me in the future.

Will they actually warn the user if this happens? The npm blog post was unclear on this point:

https://blog.npmjs.org/post/189618601100/binary-planting-wit...