Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 16 Jan 2021 11:58:24 GMT
From:      Mariusz Zaborski <oshogbo@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 6e8062c855d1 - main - cat: persistent errno
Message-ID:  <202101161158.10GBwO66093099@gitrepo.freebsd.org>

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

URL: https://cgit.FreeBSD.org/src/commit/?id=6e8062c855d19d1cbbd65b6843449f22791c3a0a

commit 6e8062c855d19d1cbbd65b6843449f22791c3a0a
Author:     Mariusz Zaborski <oshogbo@FreeBSD.org>
AuthorDate: 2021-01-16 11:55:42 +0000
Commit:     Mariusz Zaborski <oshogbo@FreeBSD.org>
CommitDate: 2021-01-16 11:55:42 +0000

    cat: persistent errno
    
    There is no guarantee that after close(2)/free the errno will remain
    persistent. The caller of the udom_open function depends on the errno
    for reporting errors.
    
    Reviewed by:    markj
    Differential Revision:  https://reviews.freebsd.org/D28185
---
 bin/cat/cat.c | 11 ++++++++++-
 1 file changed, 10 insertions(+), 1 deletion(-)

diff --git a/bin/cat/cat.c b/bin/cat/cat.c
index cdfc983610de..8ba6f6261ad4 100644
--- a/bin/cat/cat.c
+++ b/bin/cat/cat.c
@@ -429,7 +429,7 @@ udom_open(const char *path, int flags)
 	struct addrinfo hints, *res, *res0;
 	char rpath[PATH_MAX];
 	int fd = -1;
-	int error;
+	int error, serrno;
 	cap_rights_t rights;
 
 	/*
@@ -453,18 +453,23 @@ udom_open(const char *path, int flags)
 		fd = socket(res->ai_family, res->ai_socktype,
 		    res->ai_protocol);
 		if (fd < 0) {
+			serrno = errno;
 			freeaddrinfo(res0);
+			errno = serrno;
 			return (-1);
 		}
 		if (caph_rights_limit(fd, &rights) < 0) {
+			serrno = errno;
 			close(fd);
 			freeaddrinfo(res0);
+			errno = serrno;
 			return (-1);
 		}
 		error = cap_connect(capnet, fd, res->ai_addr, res->ai_addrlen);
 		if (error == 0)
 			break;
 		else {
+			serrno = errno;
 			close(fd);
 			fd = -1;
 		}
@@ -492,9 +497,13 @@ udom_open(const char *path, int flags)
 
 		cap_rights_clear(&rights, CAP_CONNECT, CAP_SHUTDOWN);
 		if (caph_rights_limit(fd, &rights) < 0) {
+			serrno = errno;
 			close(fd);
+			errno = serrno;
 			return (-1);
 		}
+	} else {
+		errno = serrno;
 	}
 	return (fd);
 }



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