[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20200617113631.GA2115184@kroah.com>
Date: Wed, 17 Jun 2020 13:36:31 +0200
From: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To: Steffen Maier <maier@...ux.ibm.com>
Cc: Minchan Kim <minchan@...nel.org>, Nitin Gupta <ngupta@...are.org>,
linux-block@...r.kernel.org, linux-kernel@...r.kernel.org,
Wade Mealing <wmealing@...hat.com>,
Sergey Senozhatsky <sergey.senozhatsky.work@...il.com>,
Jens Axboe <axboe@...nel.dk>
Subject: Re: [PATCH] Revert "zram: convert remaining CLASS_ATTR() to
CLASS_ATTR_RO()"
On Wed, Jun 17, 2020 at 01:05:54PM +0200, Steffen Maier wrote:
> On 6/17/20 12:34 PM, Greg Kroah-Hartman wrote:
> > From: Wade Mealing <wmealing@...hat.com>
> >
> > Turns out that the permissions for 0400 really are what we want here,
> > otherwise any user can write to this file.
>
> Minor confusion on my side:
> Did you mean "read from" instead of "write to"?
> I.e. only owner may read but not group nor others.
> Whereas using CLASS_ATTR_RO would resolve into 0444.
Sorry, you are correct, I'll fix the changelog text and send this out
again.
It's a "fun" file, reading from it causes something to happen, not the
normal way things work :(
greg k-h
Powered by blists - more mailing lists