Hi Sam!
    For some odd reason every time I compile GENERIC (i.e. don't try
to compile safe(4) as a standalone module, and I don't compile it with
my standard test kernel) I run into an error where the compiler
complains about the result from bswap32 in safe(4) being unused.
    This patch fixes that warning so the module compiles properly and
more importantly does the network to host conversions which may be
required for the driver under some architectures.
Thanks!
-Garrett
Index: sys/dev/safe/safe.c
===================================================================
--- sys/dev/safe/safe.c	(revision 222980)
+++ sys/dev/safe/safe.c	(working copy)
@@ -1580,9 +1580,12 @@
 				 * SHA-1 ICV's are byte-swapped; fix 'em up
 				 * before copy them to their destination.
 				 */
-				bswap32(re->re_sastate.sa_saved_indigest[0]);
-				bswap32(re->re_sastate.sa_saved_indigest[1]);
-				bswap32(re->re_sastate.sa_saved_indigest[2]);
+				re->re_sastate.sa_saved_indigest[0] =
+				    bswap32(re->re_sastate.sa_saved_indigest[0]);
+				re->re_sastate.sa_saved_indigest[1] =
+				    bswap32(re->re_sastate.sa_saved_indigest[1]);
+				re->re_sastate.sa_saved_indigest[2] =
+				    bswap32(re->re_sastate.sa_saved_indigest[2]);
 			}
 			crypto_copyback(crp->crp_flags, crp->crp_buf,
 			    crd->crd_inject,
_______________________________________________
[email protected] mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "[email protected]"

Reply via email to