mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Eric Paris <eparis@redhat.com>
To: Valdis.Kletnieks@vt.edu
Cc: linux-kernel@vger.kernel.org, malware-list@dmesg.printk.net
Subject: Re: fanotify: the fscking all notification system
Date: Tue, 07 Jul 2009 16:57:11 -0400	[thread overview]
Message-ID: <1247000231.2866.130.camel@dhcp235-23.rdu.redhat.com> (raw)
In-Reply-To: <63598.1246995702@turing-police.cc.vt.edu>

On Tue, 2009-07-07 at 15:41 -0400, Valdis.Kletnieks@vt.edu wrote:
> On Tue, 30 Jun 2009 13:26:37 EDT, Eric Paris said:

> > 1) the fd the fanotify listener gets is O_RDONLY.  I think I'll add an
> > "f_flags" option which if 0 defaults to O_RDONLY|O_LARGEFILE like we
> > have today but which you could use to indicate O_RDWR or O_WRONLY.  We
> > currently give O_RDONLY since you can't request O_WR* on files/libraries
> > mapped exec, so this won't work for executables.....
> 
> Having only O_RDONLY is a show-stopper, because then we can't replace the file
> contents before continuing.  For many places, the executables and shared
> libraries aren't the problem, so "You can't HSM an executable" as a
> semi-permanent restriction isn't too bad.

My current devel work allows you to set to open flags.  So an HSM would
use O_RDWR|O_LARGEFILE whereas a file scanner/indexer which cares about
executables/libraries would use O_RDONLY|O_LARGEFILE.

> > 2) Right now you have 5 seconds to answer an fanotify permissions
> > request, if you don't get it in 5 seconds you are done and the original
> > process gets an allow.  But I have a half finished patch which would
> > allow you to delay them infinitely.  As long as you keep them delayed
> > you can modify the file they are about to access however you like.
> 
> That would work fine.

I've added the ability to delay indefinitely.  Maybe your tape
robot/intern is slow/busy.  I'll point you to code when I want you to
see it   :)

> Yes, it's sounding like it. If that's in mainstream, then I can deploy an HSM
> without needing kernel hackery like most do currently.

I'm trying to be useful!

-Eric


  reply	other threads:[~2009-07-07 20:57 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-06-29 20:08 Eric Paris
2009-06-30 13:22 ` Valdis.Kletnieks
2009-06-30 17:13   ` Bryan Donlan
2009-06-30 17:26   ` Eric Paris
2009-07-07 19:41     ` Valdis.Kletnieks
2009-07-07 20:57       ` Eric Paris [this message]
2009-07-01  4:10 ` Matt Helsley
2009-07-11 19:18   ` Sukadev Bhattiprolu
2009-07-02 15:02 ` Jan Engelhardt
2009-07-02 15:22   ` Eric Paris

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1247000231.2866.130.camel@dhcp235-23.rdu.redhat.com \
    --to=eparis@redhat.com \
    --cc=Valdis.Kletnieks@vt.edu \
    --cc=linux-kernel@vger.kernel.org \
    --cc=malware-list@dmesg.printk.net \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®