From: yasuoka <yasuoka@openbsd.org>
Date: Mon, 2 Sep 2024 04:45:22 +0000 (+0000)
Subject: Coding of the handling of errors for <sequence number> hadn't be completed.
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=7c8f25c75aa93df13bb762b936f1afe12721b093;p=openbsd

Coding of the handling of errors for <sequence number> hadn't be completed.
---

diff --git a/usr.sbin/radiusctl/parser.c b/usr.sbin/radiusctl/parser.c
index c0934db0a2c..5c7efbc4404 100644
--- a/usr.sbin/radiusctl/parser.c
+++ b/usr.sbin/radiusctl/parser.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: parser.c,v 1.4 2024/07/24 08:27:20 yasuoka Exp $	*/
+/*	$OpenBSD: parser.c,v 1.5 2024/09/02 04:45:22 yasuoka Exp $	*/
 
 /*
  * Copyright (c) 2010 Reyk Floeter <reyk@vantronix.net>
@@ -371,9 +371,11 @@ match_token(char *word, const struct token table[])
 				break;
 			match++;
 			res.session_seq = strtonum(word, 1, UINT_MAX, &errstr);
-			if (errstr != NULL)
+			if (errstr != NULL) {
 				printf("invalid argument: %s is %s for "
-				"\"session-id\"", word, errstr);
+				    "\"session-id\"\n", word, errstr);
+				return (NULL);
+			}
 			t = &table[i];
 		case MSGAUTH:
 			if (word != NULL &&