From 2bc94b417c66333b4acae96b46d4aff34b3813cd Mon Sep 17 00:00:00 2001
From: tb <>
Date: Sun, 12 Mar 2023 17:29:02 +0000
Subject: pk7_cb() and cms_cb()

Add and fix FALLTHROUGH statement. I was confused for way too long since
I hadn't noticed that this case fell through to the next. Also add and
move some empty lines in the cms_cb() to make this resemble KNF more.
---
 src/lib/libcrypto/cms/cms_asn1.c | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

(limited to 'src/lib/libcrypto/cms/cms_asn1.c')

diff --git a/src/lib/libcrypto/cms/cms_asn1.c b/src/lib/libcrypto/cms/cms_asn1.c
index d6138f93aa..5285afd7ff 100644
--- a/src/lib/libcrypto/cms/cms_asn1.c
+++ b/src/lib/libcrypto/cms/cms_asn1.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: cms_asn1.c,v 1.20 2022/11/26 16:08:51 tb Exp $ */
+/* $OpenBSD: cms_asn1.c,v 1.21 2023/03/12 17:29:02 tb Exp $ */
 /*
  * Written by Dr Stephen N Henson (steve@openssl.org) for the OpenSSL
  * project.
@@ -1335,16 +1335,18 @@ cms_cb(int operation, ASN1_VALUE **pval, const ASN1_ITEM *it, void *exarg)
 {
 	ASN1_STREAM_ARG *sarg = exarg;
 	CMS_ContentInfo *cms = NULL;
+
 	if (pval)
 		cms = (CMS_ContentInfo *)*pval;
 	else
 		return 1;
-	switch (operation) {
 
+	switch (operation) {
 	case ASN1_OP_STREAM_PRE:
 		if (CMS_stream(&sarg->boundary, cms) <= 0)
 			return 0;
-		/* fall thru */
+		/* FALLTHROUGH */
+
 	case ASN1_OP_DETACHED_PRE:
 		sarg->ndef_bio = CMS_dataInit(cms, sarg->out);
 		if (!sarg->ndef_bio)
@@ -1356,8 +1358,8 @@ cms_cb(int operation, ASN1_VALUE **pval, const ASN1_ITEM *it, void *exarg)
 		if (CMS_dataFinal(cms, sarg->ndef_bio) <= 0)
 			return 0;
 		break;
-
 	}
+
 	return 1;
 }
 
-- 
cgit v1.2.3-55-g6feb