| History log of /src/sbin/cgdconfig/cgdconfig.c |
| Revision | | Date | Author | Comments |
| 1.63 |
| 12-May-2024 |
christos | PR/58212: Malte Dehling: Add zfs verification method
|
| 1.62 |
| 29-Apr-2024 |
riastradh | cgdconfig(8): KNF in cgdconfig.c.
No functional change intended.
|
| 1.61 |
| 17-Nov-2022 |
chs | Restore backward compatibility of UFS2 with previous NetBSD releases by disabling support in UFS2 for extended attributes (including ACLs). Add a new variant of UFS2 called "UFS2ea" that does support extended attributes. Add new fsck_ffs operations "-c ea" and "-c no-ea" to convert file systems from UFS2 to UFS2ea and vice-versa (both of which delete all existing extended attributes in the process).
|
| 1.60 |
| 13-Sep-2022 |
riastradh | cgdconfig(8): Restore loop for password re-entry for non-shared keys.
Fixes mistake in previous which changed the semantics in the case where _no_ keys are shared.
|
| 1.59 |
| 30-Aug-2022 |
riastradh | cgdconfig(8): Gracefully handle failed verification with shared keys.
The first time each key is verified, if verification fails, we chuck the failed key and try again with passphrase re-entry.
But if a key has already been verified, and verification fails, assume something is wrong with the disk and fail.
|
| 1.58 |
| 12-Aug-2022 |
riastradh | cgdconfig(8): Handle -P/-S for shared keys with -G too.
This way you can convert an existing parameters file to one that is derived from a shared key, and derive other subkeys for other disks from the same shared key.
cgdconfig -G -S -o /etc/cgd/wd0.shared /etc/cgd/wd0 cgdconfig -G -S -o /etc/cgd/wd1.shared \ -P /etc/cgd/wd0.shared /etc/cgd/wd1
This way, if you enter the same new password both times, wd0.shared and wd1.shared generate the same keys as wd0 and wd1, but only need one password entry with `cgdconfig -C'.
|
| 1.57 |
| 12-Aug-2022 |
riastradh | cgdconfig(8): Add support for generating shared-key parameters files.
Usage model:
- Generate a parameters file that supports sharing its main key:
cgdconfig -g -S -o /etc/cgd/wd0e -V gpt adiantum
- Make another parameters file that uses the same shared main key but derives an independent subkey from it:
cgdconfig -g -S -P /etc/cgd/wd0e -o /etc/cgd/ld1e \ -V disklabel aes-cbc 256
|
| 1.56 |
| 12-Aug-2022 |
riastradh | cgdconfig(8): Add support for shared keys.
New clause `shared <id> algorithm <alg> subkey <info>' in a keygen block enables `cgdconfig -C' to reuse a key between different params files, so you can, e.g., use a single password for multiple disks. This is better than simply caching the password itself because:
- Hashing the password is expensive, so it should only be done once.
Suppose your budget is time t before you get bored, and you calibrate password hash parameters to unlock n disks before you get bored waiting for `cgdconfig -C'.
. With n password hashings the adversary's cost goes up only by a factor of t/n. . With one password hashing and n subkeys the adversary's cost goes up by a factor of n.
And if you ever add a disk, rehashing it will make `cgdconfig -C' go over budget, whereas another subkey adds negligible cost to you.
- Subkeys work for other types of keygen blocks, like shell_cmd, which could be used to get a key from a hardware token that needs a button press.
The <info> parameter must be different for each params file; everything else in the keygen block must be the same. With this clause, the keygen block determines a shared key used only to derive keys; the actual key used by cgdconfig is derived from the shared key by the specified algorithm.
The only supported algorithm is hkdf-hmac-sha256, which uses HKDF-Expand of RFC 5869 instantiated with SHA-256.
Example:
algorithm aes-cbc; iv-method encblkno1; keylength 128; verify_method none; keygen pkcs5_pbkdf2/sha1 { iterations 39361; salt AAAAgMoHiYonye6KogdYJAobCHE=; shared "pw" algorithm hkdf-hmac-sha256 subkey AAAAgFlw0BMQ5gY+haYkZ6JC+yY=; };
The key used for this disk will be derived by
HKDF-HMAC-SHA256_k(WXDQExDmBj6FpiRnokL7Jg==),
where k is the outcome of PBKDF2-SHA1 with the given parameters.
Note that <info> encodes a four-byte prefix giving the big-endian length in bits of the info argument to HKDF, just like all other bit strings in cgdconfig parameters files.
If you have multiple disks configured using the same keygen block except for the info parameter, `cgdconfig -C' will only prompt once for your passphrase, generate a shared key k with PBKDF2 as usual, and then reuse it for each of the disks.
|
| 1.55 |
| 12-Aug-2022 |
riastradh | cgdconfig(8): New -T operation prints all generated keys in cgd.conf.
For testing purposes.
|
| 1.54 |
| 12-Aug-2022 |
riastradh | cgdconfig(8): New -t operation just prints the derived key in base64.
For testing purposes.
|
| 1.53 |
| 22-Nov-2021 |
nia | cgdconfig(8): Add an argon2id password-based key generation method
This provides an extra level of side-channel and cracking resistance compared to the pre-existing pkcs5_pbkdf2/sha1 method used for password-based disk encryption.
Several new keygen parameters are supported:
memory (integer, in kilobytes) parallelism (integer, usually the number of CPU cores) version (integer, usually 19...)
We do our best to calibrate these automatically when the paramsfile is initially generated.
lgtm riastradh@
|
| 1.52 |
| 16-Jun-2021 |
riastradh | cgdconfig(8): Fail more gracefully than SIGSEGV if shell_cmd fails.
|
| 1.51 |
| 18-Apr-2021 |
maya | The -n flag is useless with -g, don't mention it.
From rudolf in PR misc/36243
|
| 1.50 |
| 10-Apr-2019 |
kre | KNF, 80 column police, a few other tidy ups (use if (ret == 0) rather than if (!ret) when ret is not a boolean, and test fopen result against NULL, for example).
NFCI.
|
| 1.49 |
| 10-Apr-2019 |
kre | PR bin/53999 from rudolf (eq.cz)
Fix cgdconfig to report verification failures with gpt and mbr verification methods (and not treat them as silent hard errors). This also causes the cgd to be unconfigured when one of those verification methods fails.
Add ATF tests to check that bad verification is reported, and does not leave the cgd configured.
Patches from the PR applied.
|
| 1.48 |
| 09-May-2018 |
alnsn | branches: 1.48.2; use explicit_memset(3)
|
| 1.47 |
| 09-May-2018 |
alnsn | Missed one change when doing a manual merge of my patch with kre's commit.
|
| 1.46 |
| 09-May-2018 |
alnsn | Add '-e' option (echo the passphrase) and wipe the passphrase after use.
XXX Using memset for wiping isn't a good idea because memset is likely optimised away by gcc. This should be revisited.
|
| 1.45 |
| 09-May-2018 |
kre | Fix missing -p in usage message (noted by Christoph Badura, thanks), and update -l usage as well.
While here fix man page synopses and a few other odds and ends.
|
| 1.44 |
| 09-May-2018 |
kre | Check arg count in configure() at entry, rather than later. This avoids the stupid null deref I added a couple of commits ago (on bad usage) and also simplifies the rest of the routine which no longer needs to check the arg count nearly as much.
Thanks to Alexander Nasonov for finding the null deref bug.
|
| 1.43 |
| 06-May-2018 |
kre | Fix usage for rump. Fixes test breakage caused by previous commit.
|
| 1.42 |
| 05-May-2018 |
kre | Check whether the cgd device selected is available to be configured,that is, not already in use, before requesting passwords from the user (or elsewhere).
|
| 1.41 |
| 10-Jan-2017 |
christos | branches: 1.41.6; 1.41.12; Need <sys/stat.h> for S_IFBLK.
|
| 1.40 |
| 22-Nov-2015 |
christos | branches: 1.40.2; Don't chdir to the config directory; just form the parameters file with a path if needed.
|
| 1.39 |
| 14-Dec-2014 |
christos | fix possibly uninitialized variable.
|
| 1.38 |
| 14-Dec-2014 |
christos | avoid local variable shadowing devname.
|
| 1.37 |
| 14-Dec-2014 |
mlelstv | Add validation methods "mbr" and "gpt" for disks without a valid disklabel.
|
| 1.36 |
| 14-Dec-2014 |
mlelstv | support wedge names.
The default param file for a wedge is still named after the device (dkN) which might be unpredictable. Use an explicit param file instead.
|
| 1.35 |
| 09-Jun-2013 |
christos | List all cgd's no matter if they are not contiguously allocated
|
| 1.34 |
| 05-Dec-2012 |
christos | add cgdconfig -l like vnconfig -l
|
| 1.33 |
| 29-Aug-2011 |
joerg | branches: 1.33.2; 1.33.8; Use __dead
|
| 1.32 |
| 14-Dec-2010 |
pooka | Convert from the oldstyle cgd_kops.op_open to the newstyle prog_open and define RUMPPRG. Get rid of the Makefile.inc stuff, since it was useful only for the oldstyle regime.
|
| 1.31 |
| 02-Dec-2010 |
elric | Remove a line that was intended only for my personal testing and that breaks things.
|
| 1.30 |
| 02-Dec-2010 |
elric | In -G, refuse to operate if KEYGEN_URANDOM is specified as we already do for KEYGEN_RANDOMKEY.
Print a warning if such a refusal is made---this will help the user understand why there is an error.
Patch provided by: Taylor R Campbell <campbell+netbsd@mumble.net>.
|
| 1.29 |
| 27-Nov-2010 |
elric | Remove trailing whitespace (patch provided by: Taylor R Campbell <campbell+netbsd@mumble.net>)
|
| 1.28 |
| 08-Sep-2009 |
pooka | * allow specification of kernel operations vector * make it possible to build cgdconfig as a library
|
| 1.27 |
| 24-Jul-2008 |
christos | PR/39525: Joachim Schueth, Frederik Sausmikat: cgd inadvertently encrypts blkno eight times to generate IV
Add "encblkno1" IV type to encrypt only once, rename old "encblkno" to "encblkno8" for clarity, and make "encblkno" an alias for "encblkno8" for backward compatibility.
|
| 1.26 |
| 24-Jul-2008 |
dholland | bzero the passphrase before freeing it, to discourage chill attacks.
|
| 1.25 |
| 24-Jul-2008 |
dholland | Adjust this to not need -Wno-pointer-sign.
|
| 1.24 |
| 20-Jul-2008 |
lukem | Remove the \n and tabs from the __COPYRIGHT() strings. (Tweak some to use a consistent format.)
|
| 1.23 |
| 11-May-2008 |
elric | Add a new keygen method which will execute an external command and take its output as the key. I've been meaning to do this for quite a while...
|
| 1.22 |
| 10-May-2008 |
elric | Add -p flag to retrieve passphrases from stdin. This is mainly aimed at programatic use where the caller wants to be able to use PKCS#5.
Disable looping on invalid verifications unless both a passphrase method is in use and the -p flag is not specified.
Update man page to document -p flag.
|
| 1.21 |
| 28-Apr-2008 |
martin | branches: 1.21.2; Remove clause 3 and 4 from TNF licenses
|
| 1.20 |
| 06-Nov-2007 |
martin | branches: 1.20.8; 1.20.10; Do not use %d for size_t
|
| 1.19 |
| 06-Nov-2007 |
christos | 3 month roland feedback timeout: - Pass lint - knf - convert ints to size_t's
|
| 1.18 |
| 06-Feb-2007 |
cbiere | branches: 1.18.4; Fixed format string in opendisk_werror() which was wrong since my previous commit.
|
| 1.17 |
| 06-Feb-2007 |
cbiere | * 1 -> EXIT_FAILURE. * Replaced atoi() with parse_int(). * Use an enum for ACTION_* instead of #define. * Use strdup() or asnprintf() instead of strlcpy() to avoid truncation. * pread() returns ssize_t not int. * Suppress "unused parameter" warnings. * verify_disklabel(): Pass the correct size to disklabel_scan(). * verify_ffs(): * Don't indicate success if pread() fails. * Check the amount of bytes read. * Don't cast buf to (struct fs *), it's not aligned. * eliminate_cores(): No need for a temporary variable.
|
| 1.16 |
| 27-Jun-2005 |
christos | - constify. - don't dup extern declarations on each file because they end up being inconsistent (yyerror).
|
| 1.15 |
| 30-Mar-2005 |
elric | setprogname(3) needs to come before calls to err(3).
|
| 1.14 |
| 30-Mar-2005 |
elric | Lock all memory.
|
| 1.13 |
| 30-Mar-2005 |
elric | Turn off core dumps.
|
| 1.12 |
| 13-Aug-2004 |
tv | Add "urandomkey" key generation method as described in PR kern/22766; useful for configuring a throwaway key for cgd-on-swap at boot time.
|
| 1.11 |
| 10-Aug-2004 |
rumble | In getkey(), check the return values of the various keygen functions and abort if necessary. Also, check for errors in its callers and handle them gracefully.
OK'd by elric.
|
| 1.10 |
| 17-Mar-2004 |
dan | branches: 1.10.2; Fix a longstanding algorithmic flaw in PKCS#5 key generation.
The existing pkcs5_pbdkf2 keygen method is retained functionally as-is, for compatibility with existing params files. The corrected algorithm, which is now the default for new params file generation, is called pkcs5_pbkdf2/sha1.
NB. The backwards compatibility for the miscreant keygen method will be removed at the same time as support for the previous parameters file syntax. Sometime between now and then, users should update their params files using -G, which will create a new params file including an xor value so that the resulting generated key is the same; they should also
Problem discovery and 2-char algorithm fix by Charles Blundell, messy compat goop by me, long complicated names by Roland Dowdeswell.
Update manpage accordingly and bump date.
|
| 1.9 |
| 23-Sep-2003 |
cb | add a new verification method that prompts for the pkcs#5 pbkdf2 passphrase again and checks the generated key against the original.
|
| 1.8 |
| 17-May-2003 |
itojun | use strlcpy
|
| 1.7 |
| 02-Apr-2003 |
fvdl | Add support for UFS2. UFS2 is an enhanced FFS, adding support for 64 bit block pointers, extended attribute storage, and a few other things.
This commit does not yet include the code to manipulate the extended storage (for e.g. ACLs), this will be done later.
Originally written by Kirk McKusick and Network Associates Laboratories for FreeBSD.
|
| 1.6 |
| 24-Mar-2003 |
elric | Quick bugfix: o need to do keygen_filldefaults() in generate_convert no matter what, not only if there are no existing keygen methods in the new parameters.
|
| 1.5 |
| 24-Mar-2003 |
elric | substantial rototill of the code.
o added new features: o -G: generate a new paramsfile that produces the same key as the old paramsfile, o ffs verify_method, o multiple keygen methods that are xor'ed together (for n-factor authentication), and o calibrating the iteration count of PKCS#5 PBKDF2 to the current machine's speed. o changed paramsfile format to allow for the new features. o replaced open-coded parser with yacc grammar. o lots of supporting changes. o updated documentation to reflect new features and new paramsfile format.
|
| 1.4 |
| 28-Oct-2002 |
elric | bugfix: was still pulling random bits from /dev/random when generating a paramsfile of type randomkey which does not need them. Pointed out by dan@netbsd.org.
|
| 1.3 |
| 12-Oct-2002 |
elric | Add the concept of a verification method which allows cgdconfig(8) to reprompt for the passphrase if the key does not meet certain criteria. The currently implemented methods are ``none'' and ``disklabel''. The first behaves in the original fashion, the second will scan for a disklabel on the cgd after configuration and if it does not find a disklabel then it will reprompt for the password and reconfigure the disk.
|
| 1.2 |
| 12-Oct-2002 |
elric | Small usage enhancement: open the cgd before prompting for the passphrase so that errors can be reported w/o asking for the passphrase unnec.
|
| 1.1 |
| 04-Oct-2002 |
elric | Initial checkin of cgdconfig(8), a program to configure a cgd.
|
| 1.10.2.1 |
| 13-Aug-2004 |
tv | Oy, sometimes CVS sticky tags hurt. Revert urandomkey; that should be on HEAD.
|
| 1.18.4.4 |
| 23-Mar-2008 |
matt | sync with HEAD
|
| 1.18.4.3 |
| 08-Jan-2008 |
matt | Pull in changes from HEAD to make compile
|
| 1.18.4.2 |
| 08-Nov-2007 |
matt | sync with HEAD
|
| 1.18.4.1 |
| 06-Nov-2007 |
matt | sync with HEAD
|
| 1.20.10.1 |
| 18-May-2008 |
yamt | sync with head.
|
| 1.20.8.2 |
| 28-Sep-2008 |
mjf | Sync with HEAD.
|
| 1.20.8.1 |
| 02-Jun-2008 |
mjf | Sync with HEAD.
|
| 1.21.2.2 |
| 18-Sep-2008 |
wrstuden | Sync with wrstuden-revivesa-base-2.
|
| 1.21.2.1 |
| 23-Jun-2008 |
wrstuden | Sync w/ -current. 34 merge conflicts to follow.
|
| 1.33.8.2 |
| 23-Jun-2013 |
tls | resync from head
|
| 1.33.8.1 |
| 25-Feb-2013 |
tls | resync with head
|
| 1.33.2.2 |
| 22-May-2014 |
yamt | sync with head.
for a reference, the tree before this commit was tagged as yamt-pagecache-tag8.
this commit was splitted into small chunks to avoid a limitation of cvs. ("Protocol error: too many arguments")
|
| 1.33.2.1 |
| 16-Jan-2013 |
yamt | sync with (a bit old) head
|
| 1.40.2.1 |
| 20-Mar-2017 |
pgoyette | Sync with HEAD
|
| 1.41.12.1 |
| 21-May-2018 |
pgoyette | Sync with HEAD
|
| 1.41.6.2 |
| 07-Aug-2018 |
martin | Pull up following revision(s) (requested by alnsn in ticket #958):
sbin/cgdconfig/cgdconfig.c: revision 1.46 sbin/cgdconfig/cgdconfig.c: revision 1.47 sbin/cgdconfig/cgdconfig.c: revision 1.48 sbin/cgdconfig/cgdconfig.8: revision 1.39 sbin/cgdconfig/cgdconfig.8: revision 1.40
Add '-e' option (echo the passphrase) and wipe the passphrase after use. XXX Using memset for wiping isn't a good idea because memset is likely optimised away by gcc. This should be revisited.
Missed one change when doing a manual merge of my patch with kre's commit.
use explicit_memset(3)
With the change to use getpass_r the 128 byte passphrase limit no longer applies, so update the BUGS section here to reflect that change. The limit now is 1023 whichever method is used to fetch the passphrase.
|
| 1.41.6.1 |
| 31-Jul-2018 |
martin | Pull up following revision(s) (requested by kre in ticket #946):
sbin/cgdconfig/cgdconfig.c: revision 1.42 sbin/cgdconfig/cgdconfig.c: revision 1.43 sbin/cgdconfig/cgdconfig.c: revision 1.44 sbin/cgdconfig/cgdconfig.c: revision 1.45 sbin/cgdconfig/cgdconfig.8: revision 1.37 sbin/cgdconfig/cgdconfig.8: revision 1.38
Check whether the cgd device selected is available to be configured,that is, not already in use, before requesting passwords from the user (or elsewhere).
Fix usage for rump. Fixes test breakage caused by previous commit. Check arg count in configure() at entry, rather than later.
This avoids the stupid null deref I added a couple of commits ago (on bad usage) and also simplifies the rest of the routine which no longer needs to check the arg count nearly as much. Thanks to Alexander Nasonov for finding the null deref bug.
Fix missing -p in usage message (noted by Christoph Badura, thanks), and update -l usage as well. While here fix man page synopses and a few other odds and ends. Add commas in enumeration.
|
| 1.48.2.1 |
| 10-Jun-2019 |
christos | Sync with HEAD
|