All of lore.kernel.org
 help / color / mirror / Atom feed
From: Milan Broz <gmazyland@gmail.com>
To: dm-crypt@saout.de
Subject: Re: [dm-crypt] Key-Slot Checker Tool
Date: Sun, 09 Sep 2012 10:27:44 +0200	[thread overview]
Message-ID: <504C5300.5030102@gmail.com> (raw)
In-Reply-To: <20120909004109.GA6421@tansi.org>

On 09/09/2012 02:41 AM, Arno Wagner wrote:
> Hi all.
> 
> I just wrote a very simple key-slot checker. It divides all 
> active keyslots into 512 byte sectors and calculates entropy
> for each. For valid encrypted data, entropy will be close
> to 0.95 on average (would be 1, but this is sample entropy,
> calculated on a limited data set).

Yes, this is something very useful.

But 512 slots is quite small chunk of random data, there will be
some false warnings I guess.
(Adding add test for the whole keyslot combined
with separate sectors? Not sure if it helps something though...)

(Well, and it cannot obviously detect corruption with
overwriting random data :)

> No fancy output, no library usage (but verifies LUKS version), 
> support for non-default key-sizes and setting your own entropy 
> threshold. I put in 0.85 as default threshold, which should work 
> well. 
> 
> Now I am not sure where to put it. Should I put it in
> misc/ in the sources? That seems to be sort of a contrib/
> directory. Or should we add a section in the Wiki for
> tools?

Parsing header on its own is something which should
not be even in misc section (in the worst case it should
include luks.h directly).

But anyway, this could be integrated into luks
format checker directly (and run in "check" cryptsetup command).
(And the same random test perhaps should be in tests for large
enough blocks - see tests/differ.c, there is nice fixme :-)

I am just not sure introducing floating point in libcryptsetup
is good idea. But perhaps this can be compile time option,
if some ancient/embedded CPU/distro has problems here,
so it can be compiled-out.

Milan

  reply	other threads:[~2012-09-09  8:27 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-09-09  0:41 [dm-crypt] Key-Slot Checker Tool Arno Wagner
2012-09-09  8:27 ` Milan Broz [this message]
2012-09-09 13:35   ` Arno Wagner
2012-09-09 21:40     ` Arno Wagner

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=504C5300.5030102@gmail.com \
    --to=gmazyland@gmail.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 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.