DM-Crypt Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Milan Broz <mbroz@redhat.com>
To: dm-crypt@saout.de
Subject: Re: [dm-crypt] [ANNOUNCE] cryptsetup 1.3.0-rc1 (test release candidate)
Date: Tue, 15 Mar 2011 13:36:32 +0100	[thread overview]
Message-ID: <4D7F5D50.9010302@redhat.com> (raw)
In-Reply-To: <20110315120939.GA3660@tansi.org>

On 03/15/2011 01:09 PM, Arno Wagner wrote:

>> I will also limit LUKS keyfile in next version, mistake will cause problems
>> (reading the whole device in locked memory -> OOPS or something like that).
> 
> Indeed. What you could do is have it default to error instead of
> to cut-down and proceed. That would make more sense, since
> cutting it is almost always not going to work.

yup. This is common problem for all supported types, so I will probably
remove per-type limits, and add one maximal keyfile limit configurable
during compile time.

Exceeding this limit will cause fail.

Limiting keyfile read (if needed - like /dev/urandom or when stored
in the beginning of device) should be always possible using --keyfile-size
parameter (or --new-keyfile-size for new keyfile option).

(This commadline limit option should possibly override maximal compiled-in
limit if specified number is larger.)

Is it better approach? 

Milan

  reply	other threads:[~2011-03-15 12:36 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-03-14 17:44 [dm-crypt] [ANNOUNCE] cryptsetup 1.3.0-rc1 (test release candidate) Milan Broz
2011-03-14 21:13 ` Rudolf Deilmann
2011-03-14 22:10   ` Milan Broz
2011-03-14 23:45     ` Rudolf Deilmann
2011-03-15  1:02       ` Arno Wagner
2011-03-15 11:11         ` Milan Broz
2011-03-15 11:43           ` Jonas Meurer
2011-03-15 12:19             ` Milan Broz
2011-03-15 12:09           ` Arno Wagner
2011-03-15 12:36             ` Milan Broz [this message]
2011-03-15 13:49               ` Arno Wagner
2011-03-21 18:21   ` Milan Broz
2011-03-22 18:47     ` Rudolf Deilmann
2011-03-22 19:09       ` Milan Broz
2011-03-22 20:44         ` Rudolf Deilmann
2011-03-22 21:05           ` Milan Broz
2011-03-23 14:14             ` Rudolf Deilmann
2011-03-23 14:37               ` Milan Broz
2011-03-23 16:00                 ` Rudolf Deilmann
2011-03-23 16:26                   ` Milan Broz
2011-04-14 14:28                   ` [dm-crypt] [ANNOUNCE] cryptsetup 1.3.0-rc1 ( test " Ludwig Nussel
2011-03-17 13:36 ` [dm-crypt] [ANNOUNCE] cryptsetup 1.3.0-rc1 (test " Ludwig Nussel
2011-03-17 14:02   ` Milan Broz
2011-03-22 12:50 ` [dm-crypt] [ANNOUNCE] cryptsetup 1.3.0-rc2 " Milan Broz
2011-04-05 20:02 ` [dm-crypt] [ANNOUNCE] cryptsetup 1.3.0 Milan Broz
2011-05-24 15:48 ` [dm-crypt] [ANNOUNCE] cryptsetup 1.3.1 Milan Broz

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=4D7F5D50.9010302@redhat.com \
    --to=mbroz@redhat.com \
    --cc=dm-crypt@saout.de \
    /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