From 10f1fe5fd8dfaa321621a1063d252940bb65029e Mon Sep 17 00:00:00 2001
From: Guy Harris <guy@alum.mit.edu>
Date: Sun, 14 Jan 2018 14:20:30 -0800
Subject: [PATCH] Check against various modes explicitly.

This avoids a "comparison is always true" warning when checking whether
mode is >= MODE_UNSPEC.
---
 print-ntp.c | 20 ++++++++++++++++----
 1 file changed, 16 insertions(+), 4 deletions(-)

diff --git a/print-ntp.c b/print-ntp.c
index 99cf72b2..b4fa6129 100644
--- a/print-ntp.c
+++ b/print-ntp.c
@@ -479,12 +479,24 @@ ntp_print(netdissect_options *ndo,
 	          tok2str(ntp_leapind_values, "Unknown", leapind),
 	          leapind);
 
-	if (mode >= MODE_UNSPEC && mode <= MODE_BROADCAST)
+	switch (mode) {
+
+	case MODE_UNSPEC:
+	case MODE_SYM_ACT:
+	case MODE_SYM_PAS:
+	case MODE_CLIENT:
+	case MODE_SERVER:
+	case MODE_BROADCAST:
 		ntp_time_print(ndo, &bp->td, length);
-	else if (mode == MODE_CONTROL)
+		break;
+
+	case MODE_CONTROL:
 		ntp_control_print(ndo, &bp->cd, length);
-	else
-		{;}			/* XXX: not implemented! */
+		break;
+
+	default:
+		break;			/* XXX: not implemented! */
+	}
 	return;
 
 trunc:
-- 
2.39.5