[src/trunk]: src/sys/opencrypto Add missing "break" for CRYPTO_CAST_CBC, and ...

[hubertf <hubertf%NetBSD.org@localhost>]

details:   https://anonhg.NetBSD.org/src/rev/831901c2585e
branches:  trunk
changeset: 751309:831901c2585e
user:      hubertf <hubertf%NetBSD.org@localhost>
date:      Sun Jan 31 14:32:56 2010 +0000

description:
Add missing "break" for CRYPTO_CAST_CBC, and some assorted comment fixes.
openssl(1) checks for CAST (and others) on ~every startup.

diffstat:

 sys/opencrypto/cryptodev.c |  8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diffs (43 lines):

diff -r aa73ff8c4dd6 -r 831901c2585e sys/opencrypto/cryptodev.c
--- a/sys/opencrypto/cryptodev.c        Sun Jan 31 14:30:22 2010 +0000
+++ b/sys/opencrypto/cryptodev.c        Sun Jan 31 14:32:56 2010 +0000
@@ -1,4 +1,4 @@
-/*     $NetBSD: cryptodev.c,v 1.51 2009/12/20 09:36:06 dsl Exp $ */
+/*     $NetBSD: cryptodev.c,v 1.52 2010/01/31 14:32:56 hubertf Exp $ */
 /*     $FreeBSD: src/sys/opencrypto/cryptodev.c,v 1.4.2.4 2003/06/03 00:09:02 sam Exp $        */
 /*     $OpenBSD: cryptodev.c,v 1.53 2002/07/10 22:21:30 mickey Exp $   */
 
@@ -64,7 +64,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: cryptodev.c,v 1.51 2009/12/20 09:36:06 dsl Exp $");
+__KERNEL_RCSID(0, "$NetBSD: cryptodev.c,v 1.52 2010/01/31 14:32:56 hubertf Exp $");
 
 #include <sys/param.h>
 #include <sys/systm.h>
@@ -598,7 +598,6 @@
         * XXX disabled on NetBSD since 1.6O due to a race condition.
         * XXX But crypto_dispatch went to splcrypto() itself!  (And
         * XXX now takes the crypto_mtx mutex itself).  We do, however,
-        *
         * XXX need to hold the mutex across the call to cv_wait().
         * XXX     (should we arrange for crypto_dispatch to return to
         * XXX      us with it held?  it seems quite ugly to do so.)
@@ -612,7 +611,7 @@
        /* 
         * If the request was going to be completed by the
         * ioctl thread then it would have been done by now.
-        * Remove the F_USER flag it so crypto_done() is not confused
+        * Remove the F_USER flag so crypto_done() is not confused
         * if the crypto device calls it after this point.
         */
        crp->crp_flags &= ~(CRYPTO_F_USER);
@@ -1498,6 +1497,7 @@
                break;
        case CRYPTO_CAST_CBC:
                txform = &enc_xform_cast5;
+               break;
        case CRYPTO_SKIPJACK_CBC:
                txform = &enc_xform_skipjack;
                break;