From: Eric Biggers <ebiggers3@gmail.com>
To: Anand Jain <anand.jain@oracle.com>
Cc: linux-fscrypt@vger.kernel.org, linux-doc@vger.kernel.org,
linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org,
"Theodore Y . Ts'o" <tytso@mit.edu>,
Jaegeuk Kim <jaegeuk@kernel.org>,
Richard Weinberger <richard@nod.at>,
Michael Halcrow <mhalcrow@google.com>,
Eric Biggers <ebiggers@google.com>
Subject: Re: [PATCH] fscrypt: add a documentation file for filesystem-level encryption
Date: Thu, 31 Aug 2017 10:50:46 -0700 [thread overview]
Message-ID: <20170831175046.GB5023@gmail.com> (raw)
In-Reply-To: <9e3f608d-5410-b8f7-cdf0-93045be7a995@oracle.com>
Hi Anand,
On Mon, Aug 28, 2017 at 08:18:46PM +0800, Anand Jain wrote:
>
>
> On 08/23/2017 01:07 AM, Eric Biggers wrote:
> >On Tue, Aug 22, 2017 at 11:33:51PM +0800, Anand Jain wrote:
> >>
> >>
> >>On 08/22/2017 10:55 AM, Eric Biggers wrote:
> >>>On Tue, Aug 22, 2017 at 10:22:30AM +0800, Anand Jain wrote:
> >>>>
> >>>>Hi Eric,
> >>>>
> >>>> How about a section on the threat model specific to the file-name ?
> >>>>
> >>>> (Sorry if I am missing something).
> >>>>
> >>>>Thanks, Anand
> >>>
> >>>It's already mentioned that filenames are encrypted: "fscrypt protects the
> >>>confidentiality of file contents and filenames in the event of a single
> >>>point-in-time permanent offline compromise of the block device content."
> >>>There's not much more to it than that; all the other points in the "Threat
> >>>model" section (offline manipulations, timing attacks, access control, key
> >>>eviction, etc.) are essentially the same between contents and filenames
> >>>encryption.
> >>
> >> Do you think if application does not keep the sensitive information
> >>in the file-name, would that remove the file-name from the list of
> >>items that should be protected ?
> >>
> >
> >If *no* applications care whether the filenames are encrypted or not, sure.
> >But are you absolutely sure that no applications care? How do you know? And what
> >is the advantage of not encrypting the filenames anyway? It is better to
> >encrypt by default.
> >
> >Eric
>
> (sorry for the delay in reply due to my vacation).
>
> It all depends on the use case, Android is one such use case. Some
> data center use a known set of application. Again it all depends on
> the use case.
>
> File-name is a kind of File-system semantic and altering based on
> the on the user key context does not guarantee the system will be
> compatible with all their legacy applications.
>
You really need to give more detail about why filename encryption specifically
is a problem. What "legacy applications" is a problem for, and why? What do
you mean by a "data center" use case? Isn't contents encryption a "problem" for
some "legacy applications" as well? Contents encryption changes filesystem
semantics as well.
> Also a section on backup and restore in this doc will be a good
> idea. As I think that will be affected IMO. And needing to have the
> user master key to restore encrypted file isn't practical in some
> data center multi tenanted solutions. Albeit it may work in some
> cases but hard to generalize.
>
The proposed documentation already mentions that there is no way to backup the
raw ciphertext currently. I'll try to make this a bit clearer, but in any case
I am documenting the current state of things; the new documentation file is
*not* a plan for things that don't exist yet. If/when someone proposes a
patchset which adds a backup+restore API they can propose a documentation update
along with it.
Eric
prev parent reply other threads:[~2017-08-31 17:50 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-08-18 19:47 [PATCH] fscrypt: add a documentation file for filesystem-level encryption Eric Biggers
2017-08-18 21:06 ` Andreas Dilger
2017-08-20 2:32 ` Theodore Ts'o
2017-08-21 22:33 ` Eric Biggers
2017-08-21 13:44 ` Anand Jain
2017-08-21 21:02 ` Theodore Ts'o
2017-08-21 23:08 ` Eric Biggers
2017-08-22 2:22 ` Anand Jain
2017-08-22 3:07 ` Eric Biggers
2017-08-22 15:35 ` Anand Jain
2017-08-22 17:36 ` Eric Biggers
2017-08-28 12:18 ` Anand Jain
2017-08-31 18:14 ` Eric Biggers
2017-08-22 3:07 ` Theodore Ts'o
2017-08-22 2:22 ` Anand Jain
2017-08-22 2:55 ` Eric Biggers
2017-08-22 15:33 ` Anand Jain
2017-08-22 17:07 ` Eric Biggers
2017-08-28 12:18 ` Anand Jain
2017-08-28 14:22 ` Theodore Ts'o
2017-08-29 3:54 ` Anand Jain
2017-08-31 18:10 ` Eric Biggers
2017-08-31 17:50 ` Eric Biggers [this message]
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=20170831175046.GB5023@gmail.com \
--to=ebiggers3@gmail.com \
--cc=anand.jain@oracle.com \
--cc=ebiggers@google.com \
--cc=jaegeuk@kernel.org \
--cc=linux-doc@vger.kernel.org \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-fscrypt@vger.kernel.org \
--cc=linux-fsdevel@vger.kernel.org \
--cc=mhalcrow@google.com \
--cc=richard@nod.at \
--cc=tytso@mit.edu \
/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;
as well as URLs for NNTP newsgroup(s).