public inbox for passt-dev@passt.top
 help / color / mirror / code / Atom feed
* [PATCH] tap: Avoid bogus missingReturn cppcheck warning in tap_l2_max_len()
@ 2025-05-15 16:05 Stefano Brivio
  2025-05-16  7:25 ` David Gibson
  0 siblings, 1 reply; 5+ messages in thread
From: Stefano Brivio @ 2025-05-15 16:05 UTC (permalink / raw)
  To: passt-dev

Cppcheck 2.14.2 on Alpine Linux (musl) says that:

tap.c:111:19: error: Found an exit path from function with non-void return type that has missing return statement [missingReturn]
 switch (c->mode) {
                  ^

This exit path is not reachable because we ASSERT(0) after the switch,
but for some reason cppcheck doesn't see this with musl headers. Hide
the warning with a redundant return statement.

Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
---
 tap.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/tap.c b/tap.c
index d630f6d..6db5d88 100644
--- a/tap.c
+++ b/tap.c
@@ -118,6 +118,8 @@ unsigned long tap_l2_max_len(const struct ctx *c)
 	}
 	/* NOLINTEND(bugprone-branch-clone) */
 	ASSERT(0);
+
+	return 0; /* Unreachable, for cppcheck's sake */
 }
 
 /**
-- 
@@ -118,6 +118,8 @@ unsigned long tap_l2_max_len(const struct ctx *c)
 	}
 	/* NOLINTEND(bugprone-branch-clone) */
 	ASSERT(0);
+
+	return 0; /* Unreachable, for cppcheck's sake */
 }
 
 /**
-- 
2.43.0


^ permalink raw reply related	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2025-05-19  7:27 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-05-15 16:05 [PATCH] tap: Avoid bogus missingReturn cppcheck warning in tap_l2_max_len() Stefano Brivio
2025-05-16  7:25 ` David Gibson
2025-05-16 16:26   ` Stefano Brivio
2025-05-17 10:29     ` David Gibson
2025-05-19  7:27       ` Stefano Brivio

Code repositories for project(s) associated with this public inbox

	https://passt.top/passt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for IMAP folder(s).