All of lore.kernel.org
 help / color / mirror / Atom feed
From: Arno Wagner <arno@wagner.name>
To: dm-crypt@saout.de
Subject: Re: [dm-crypt] Option "validate passphrase" for command cryptsetup
Date: Wed, 20 Jun 2012 03:13:23 +0200	[thread overview]
Message-ID: <20120620011323.GA29688@tansi.org> (raw)
In-Reply-To: <4FE0A548.7030007@gmail.com>

Good discussion here, I like it! 

I prefer --test-passphrase, as it does not have the
general ring of --dry-run. With --dry-run, people
would rightfully expect to be able to use it everywhere.

Just let me know what the final decision is and I will
add it to the man-page.

Arno

On Tue, Jun 19, 2012 at 06:14:00PM +0200, Milan Broz wrote:
> On 06/19/2012 05:04 PM, jonas wrote:
> 
> > if I'm not wrong, one difference between Louis' suggestion and the way 
> > you implemented it is, that the former works with active devices, and 
> > the latter doesn't, right?
> 
> No, it is exactly the same. It works even for active devices.
> (Check for active device is later.)
> 
> > I like the idea of a --dry-run option which works for all commands, 
> > just like a simulation mode. But as well I like the idea of a command 
> > for key validation, which takes the same commandline options as 
> > luksOpen, and simply verifies whether the given key (passphrase, 
> > keyfile, whatever) is valid.
> 
> Well, universal --dry-run is nice idea but I am not going to implement it now.
> (and I would perhaps do it differently - do everything as is except final
> on-disk metadata update or in-kernel device change.)
> 
> 
> Well, I have local commit renaming this luksOpen option to --test-passphrase.
> If there are no other suggestions for today, I'll commit it.
> 
> Milan
> (grumbling something about bikeshedding :-)
> _______________________________________________
> dm-crypt mailing list
> dm-crypt@saout.de
> http://www.saout.de/mailman/listinfo/dm-crypt
> 

-- 
Arno Wagner,    Dr. sc. techn., Dipl. Inform.,   Email: arno@wagner.name 
GnuPG:  ID: 1E25338F  FP: 0C30 5782 9D93 F785 E79C  0296 797F 6B50 1E25 338F
----
One of the painful things about our time is that those who feel certainty 
are stupid, and those with any imagination and understanding are filled 
with doubt and indecision. -- Bertrand Russell 

  parent reply	other threads:[~2012-06-20  1:13 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-06-19 10:53 [dm-crypt] Option "validate passphrase" for command cryptsetup Louis
2012-06-19 11:26 ` Milan Broz
2012-06-19 12:29   ` Arno Wagner
2012-06-19 13:41     ` Milan Broz
2012-06-19 13:54       ` Thomas Bächler
2012-06-19 14:17         ` Milan Broz
2012-06-19 14:56           ` Matthias Schniedermeyer
2012-06-19 16:20             ` ken
2012-06-19 22:02         ` Heinz Diehl
2012-06-19 15:04   ` jonas
2012-06-19 16:14     ` Milan Broz
2012-06-19 16:46       ` Jonas Meurer
2012-06-20  1:13       ` Arno Wagner [this message]
2012-06-19 16:56 ` ken
2012-06-20  7:41   ` Louis
2012-06-20  8:27     ` 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=20120620011323.GA29688@tansi.org \
    --to=arno@wagner.name \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.