Re: [PATCH v2] Add a "nosymlinks" mount option.

2016-11-22 Thread Mattias Nissler
On Mon, Nov 21, 2016 at 6:10 PM, James Bottomley wrote: > On Wed, 2016-11-16 at 13:18 -0800, Mattias Nissler wrote: >> I understand that silence suggests there's little interest, but >> here's some new information I discovered today that may justify to >> reconsider the patch: >> >> The BSDs alrea

Re: [PATCH v2] Add a "nosymlinks" mount option.

2016-11-21 Thread James Bottomley
On Wed, 2016-11-16 at 13:18 -0800, Mattias Nissler wrote: > I understand that silence suggests there's little interest, but > here's some new information I discovered today that may justify to > reconsider the patch: > > The BSDs already have exactly what I propose, the mount option is > called "

Re: [PATCH v2] Add a "nosymlinks" mount option.

2016-11-17 Thread Mattias Nissler
On Thu, Nov 17, 2016 at 1:43 PM, Austin S. Hemmelgarn wrote: > On 2016-11-16 16:18, Mattias Nissler wrote: >> >> I understand that silence suggests there's little interest, but here's >> some new information I discovered today that may justify to reconsider >> the patch: >> >> The BSDs already hav

Re: [PATCH v2] Add a "nosymlinks" mount option.

2016-11-17 Thread Austin S. Hemmelgarn
On 2016-11-16 16:18, Mattias Nissler wrote: I understand that silence suggests there's little interest, but here's some new information I discovered today that may justify to reconsider the patch: The BSDs already have exactly what I propose, the mount option is called "nosymfollow" there: https

Re: [PATCH v2] Add a "nosymlinks" mount option.

2016-11-16 Thread Mattias Nissler
I understand that silence suggests there's little interest, but here's some new information I discovered today that may justify to reconsider the patch: The BSDs already have exactly what I propose, the mount option is called "nosymfollow" there: https://github.com/freebsd/freebsd/blob/a41f4cc9a57

Re: [PATCH v2] Add a "nosymlinks" mount option.

2016-10-24 Thread Mattias Nissler
Friendly ping - does this version of the patch have any chance on getting included in mainline? On Wed, Oct 19, 2016 at 2:31 PM, Mattias Nissler wrote: > For mounts that have the new "nosymlinks" option, don't follow > symlinks when resolving paths. The new option is similar in spirit to > the ex

[PATCH v2] Add a "nosymlinks" mount option.

2016-10-19 Thread Mattias Nissler
For mounts that have the new "nosymlinks" option, don't follow symlinks when resolving paths. The new option is similar in spirit to the existing "nodev", "noexec", and "nosuid" options. Note that symlinks may still be created on mounts where the "nosymlinks" option is present. readlink() remains