Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 9 May 2018 22:58:32 +0200
From:      Oliver Pinter <oliver.pinter@hardenedbsd.org>
To:        Mariusz Zaborski <oshogbo@freebsd.org>
Cc:        "src-committers@freebsd.org" <src-committers@freebsd.org>,  "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>,  "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r333438 - head/sbin/geom/class/eli
Message-ID:  <CAPQ4ffuziuiZ_j6yBT-pTZDEd9-Ad7RGete-VZP1qyhtbMFTAQ@mail.gmail.com>
In-Reply-To: <201805092051.w49KpGZm079239@repo.freebsd.org>
References:  <201805092051.w49KpGZm079239@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wednesday, May 9, 2018, Mariusz Zaborski <oshogbo@freebsd.org> wrote:

> Author: oshogbo
> Date: Wed May  9 20:51:16 2018
> New Revision: 333438
> URL: https://svnweb.freebsd.org/changeset/base/333438
>
> Log:
>   Change option dry-run from 'n' to 'C' in geli attach command.
>
>   'n' is used in other commands to define the key index.
>   We should be consistent with that.
>   'C' option is used by patch(1) to perform dryrun so lets use that.
>
>   Reviewed by:  allanjude
>   Differential Revision:        https://reviews.freebsd.org/D15308


Relnotes: yes


>
> Modified:
>   head/sbin/geom/class/eli/geli.8
>   head/sbin/geom/class/eli/geom_eli.c
>
> Modified: head/sbin/geom/class/eli/geli.8
> ============================================================
> ==================
> --- head/sbin/geom/class/eli/geli.8     Wed May  9 20:49:50 2018
> (r333437)
> +++ head/sbin/geom/class/eli/geli.8     Wed May  9 20:51:16 2018
> (r333438)
> @@ -24,7 +24,7 @@
>  .\"
>  .\" $FreeBSD$
>  .\"
> -.Dd April 10, 2018
> +.Dd May 9, 2018
>  .Dt GELI 8
>  .Os
>  .Sh NAME
> @@ -67,7 +67,7 @@ utility:
>  .Cm init
>  .Nm
>  .Cm attach
> -.Op Fl dnprv
> +.Op Fl Cdprv
>  .Op Fl j Ar passfile
>  .Op Fl k Ar keyfile
>  .Ar prov
> @@ -393,6 +393,9 @@ suffix.
>  .Pp
>  Additional options include:
>  .Bl -tag -width ".Fl j Ar passfile"
> +.It Fl C
> +Do a dry-run decryption.
> +This is useful to verify passphrase and keyfile without decrypting the
> device.
>  .It Fl d
>  If specified, a decrypted provider will be detached automatically on last
> close.
>  This can help with scarce memory so the user does not have to remember to
> detach the
> @@ -420,9 +423,6 @@ For more information see the description of the
>  option for the
>  .Cm init
>  subcommand.
> -.It Fl n
> -Do a dry-run decryption.
> -This is useful to verify passphrase and keyfile without decrypting the
> device.
>  .It Fl p
>  Do not use a passphrase as a component of the User Key.
>  Cannot be combined with the
>
> Modified: head/sbin/geom/class/eli/geom_eli.c
> ============================================================
> ==================
> --- head/sbin/geom/class/eli/geom_eli.c Wed May  9 20:49:50 2018
> (r333437)
> +++ head/sbin/geom/class/eli/geom_eli.c Wed May  9 20:51:16 2018
> (r333438)
> @@ -86,7 +86,7 @@ static int eli_backup_create(struct gctl_req *req, con
>   *
>   * init [-bdgPTv] [-a aalgo] [-B backupfile] [-e ealgo] [-i iterations]
> [-l keylen] [-J newpassfile] [-K newkeyfile] [-s sectorsize] [-V version]
> prov
>   * label - alias for 'init'
> - * attach [-dprv] [-j passfile] [-k keyfile] prov
> + * attach [-Cdprv] [-j passfile] [-k keyfile] prov
>   * detach [-fl] prov ...
>   * stop - alias for 'detach'
>   * onetime [-d] [-a aalgo] [-e ealgo] [-l keylen] prov
> @@ -145,15 +145,15 @@ struct g_command class_commands[] = {
>         },
>         { "attach", G_FLAG_VERBOSE | G_FLAG_LOADKLD, eli_main,
>             {
> +               { 'C', "dryrun", NULL, G_TYPE_BOOL },
>                 { 'd', "detach", NULL, G_TYPE_BOOL },
>                 { 'j', "passfile", G_VAL_OPTIONAL, G_TYPE_STRING |
> G_TYPE_MULTI },
>                 { 'k', "keyfile", G_VAL_OPTIONAL, G_TYPE_STRING |
> G_TYPE_MULTI },
> -               { 'n', "dryrun", NULL, G_TYPE_BOOL },
>                 { 'p', "nopassphrase", NULL, G_TYPE_BOOL },
>                 { 'r', "readonly", NULL, G_TYPE_BOOL },
>                 G_OPT_SENTINEL
>             },
> -           "[-dnprv] [-j passfile] [-k keyfile] prov"
> +           "[-Cdprv] [-j passfile] [-k keyfile] prov"
>         },
>         { "detach", 0, NULL,
>             {
> _______________________________________________
> svn-src-head@freebsd.org mailing list
> https://lists.freebsd.org/mailman/listinfo/svn-src-head
> To unsubscribe, send any mail to "svn-src-head-unsubscribe@freebsd.org"
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAPQ4ffuziuiZ_j6yBT-pTZDEd9-Ad7RGete-VZP1qyhtbMFTAQ>