Author: phk
Date: Sun Nov  9 15:53:29 2014
New Revision: 274314
URL: https://svnweb.freebsd.org/changeset/base/274314

Log:
  Report the 1-based key numbers rather than the 0-based ones to be consistent.
  
  Fix documentation for destroy command.  Not sure how the wrong explanation
  happened.
  
  Spotted by:   mwlucas

Modified:
  head/sbin/gbde/gbde.8
  head/sbin/gbde/gbde.c

Modified: head/sbin/gbde/gbde.8
==============================================================================
--- head/sbin/gbde/gbde.8       Sun Nov  9 15:52:11 2014        (r274313)
+++ head/sbin/gbde/gbde.8       Sun Nov  9 15:53:29 2014        (r274314)
@@ -233,9 +233,23 @@ pass-phrase:
 .Pp
 .Dl "gbde setkey ada0s1f -n 2 -P foo -L key2.lockfile"
 .Pp
-To destroy all copies of the masterkey:
+To invalidate your own masterkey:
+.Pp
+.Dl "gbde nuke ada0s1f"
+.Pp
+This will overwrite your masterkey sector with zeros, and results in
+a diagnostic if you try to use the key again.
+You can also destroy the other three copies of the masterkey with the
+-n argument.
+.Pp
+You can also invalidate your masterkey without leaving a tell-tale sector
+full of zeros:
 .Pp
 .Dl "gbde destroy ada0s1f"
+.Pp
+This will overwrite the information fields in your masterkey sector,
+encrypt it and write it back.
+You get a (different) diagnostic if you try to use it.
 .Sh SEE ALSO
 .Xr gbde 4 ,
 .Xr geom 4

Modified: head/sbin/gbde/gbde.c
==============================================================================
--- head/sbin/gbde/gbde.c       Sun Nov  9 15:52:11 2014        (r274313)
+++ head/sbin/gbde/gbde.c       Sun Nov  9 15:53:29 2014        (r274314)
@@ -300,7 +300,6 @@ cmd_attach(const struct g_bde_softc *sc,
                gctl_ro_param(r, "key", 16, buf);
                close(ffd);
        }
-       /* gctl_dump(r, stdout); */
        errstr = gctl_issue(r);
        if (errstr != NULL)
                errx(1, "Attach to %s failed: %s", dest, errstr);
@@ -371,7 +370,7 @@ cmd_open(struct g_bde_softc *sc, int dfd
        if (error != 0)
                errx(1, "Error %d decrypting lock", error);
        if (nkey)
-               printf("Opened with key %u\n", *nkey);
+               printf("Opened with key %u\n", 1 + *nkey);
        return;
 }
 
@@ -392,7 +391,7 @@ cmd_nuke(struct g_bde_key *gl, int dfd ,
        free(sbuf);
        if (i != (int)gl->sectorsize)
                err(1, "write");
-       printf("Nuked key %d\n", key);
+       printf("Nuked key %d\n", 1 + key);
 }
 
 static void
@@ -493,7 +492,7 @@ cmd_destroy(struct g_bde_key *gl, int nk
        bzero(&gl->sector0, sizeof gl->sector0);
        bzero(&gl->sectorN, sizeof gl->sectorN);
        bzero(&gl->keyoffset, sizeof gl->keyoffset);
-       bzero(&gl->flags, sizeof gl->flags);
+       gl->flags &= GBDE_F_SECT0;
        bzero(gl->mkey, sizeof gl->mkey);
        for (i = 0; i < G_BDE_MAXKEYS; i++)
                if (i != nkey)
_______________________________________________
svn-src-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"

Reply via email to