Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 12 Aug 2021 02:29:31 GMT
From:      Eric van Gyzen <vangyzen@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 080c6fdc0ce8 - main - dumpon: fix build on some architectures
Message-ID:  <202108120229.17C2TVHI044819@gitrepo.freebsd.org>

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

URL: https://cgit.FreeBSD.org/src/commit/?id=080c6fdc0ce86c55f7cbbe1e86c026f3bc641bb1

commit 080c6fdc0ce86c55f7cbbe1e86c026f3bc641bb1
Author:     Eric van Gyzen <vangyzen@FreeBSD.org>
AuthorDate: 2021-08-12 02:27:24 +0000
Commit:     Eric van Gyzen <vangyzen@FreeBSD.org>
CommitDate: 2021-08-12 02:27:24 +0000

    dumpon: fix build on some architectures
    
    Fix a -Wsign-compare issue on some architectures.
    
    Fixes:          96f9bd46547d6dfbaf219ab449efacacb0dacccc
    Pointy hat to:  me
    MFC after:      1 week
    Sponsored by:   Dell EMC Isilon
---
 sbin/dumpon/dumpon.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sbin/dumpon/dumpon.c b/sbin/dumpon/dumpon.c
index 291239c4bcc8..a11f07a01200 100644
--- a/sbin/dumpon/dumpon.c
+++ b/sbin/dumpon/dumpon.c
@@ -334,7 +334,7 @@ genkey(const char *pubkeyfile, struct diocskerneldump_arg *kdap)
 			err(1, "genkey pipe write");
 		bytes = write(filedes[1], kdap->kda_encryptedkey,
 		    kdap->kda_encryptedkeysize);
-		if (bytes != kdap->kda_encryptedkeysize)
+		if (bytes != (ssize_t)kdap->kda_encryptedkeysize)
 			err(1, "genkey pipe write kda_encryptedkey");
 		_exit(0);
 	}
@@ -351,7 +351,7 @@ genkey(const char *pubkeyfile, struct diocskerneldump_arg *kdap)
 		err(1, "Unable to allocate encrypted key");
 	bytes = read(filedes[0], kdap->kda_encryptedkey,
 	    kdap->kda_encryptedkeysize);
-	if (bytes != kdap->kda_encryptedkeysize)
+	if (bytes != (ssize_t)kdap->kda_encryptedkeysize)
 		errx(1, "genkey pipe read kda_encryptedkey");
 	error = waitpid(pid, &status, WEXITED);
 	if (error == -1)



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