Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 17 Apr 2023 01:05:00 GMT
From:      "Stephen J. Kiernan" <stevek@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 1c7ac0c23bc1 - main - veriexec: fix use of getopt
Message-ID:  <202304170105.33H150a7020428@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by stevek:

URL: https://cgit.FreeBSD.org/src/commit/?id=1c7ac0c23bc12cc7cd5776fa28ebe0582e1bccec

commit 1c7ac0c23bc12cc7cd5776fa28ebe0582e1bccec
Author:     Stephen J. Kiernan <stevek@FreeBSD.org>
AuthorDate: 2023-04-17 00:59:37 +0000
Commit:     Stephen J. Kiernan <stevek@FreeBSD.org>
CommitDate: 2023-04-17 01:02:04 +0000

    veriexec: fix use of getopt
    
    getopt(3) returns int type not char. Using char triggers the
    -Wtautological-constant-out-of-range-compare warning with clang.
    Change the type of the variable used for holding the return value
    of getopt(3) to int to match the prototype and eliminate the warning.
    
    MFC after:      1 week
---
 sbin/veriexec/veriexec.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/sbin/veriexec/veriexec.c b/sbin/veriexec/veriexec.c
index 407100ab48f1..815aaae5e81b 100644
--- a/sbin/veriexec/veriexec.c
+++ b/sbin/veriexec/veriexec.c
@@ -140,8 +140,7 @@ main(int argc, char *argv[])
 {
 	long long converted_int;
 	uint32_t state;
-	char c;
-	int x;
+	int c, x;
 
 	if (argc < 2)
 		return (veriexec_usage());



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202304170105.33H150a7020428>