svn commit: r311233 - head/contrib/netbsd-tests/fs/tmpfs
Ngie Cooper
ngie at FreeBSD.org
Wed Jan 4 02:46:37 UTC 2017
Author: ngie
Date: Wed Jan 4 02:46:36 2017
New Revision: 311233
URL: https://svnweb.freebsd.org/changeset/base/311233
Log:
Fix Coverity issues
- Initialize .sun_len before passing it to strlcpy and bind.
- Close fd on error
MFC after: 3 days
Reported by: Coverity
CID: 978283, 979581
Modified:
head/contrib/netbsd-tests/fs/tmpfs/h_tools.c
Modified: head/contrib/netbsd-tests/fs/tmpfs/h_tools.c
==============================================================================
--- head/contrib/netbsd-tests/fs/tmpfs/h_tools.c Wed Jan 4 02:43:33 2017 (r311232)
+++ head/contrib/netbsd-tests/fs/tmpfs/h_tools.c Wed Jan 4 02:46:36 2017 (r311233)
@@ -243,12 +243,19 @@ sockets_main(int argc, char **argv)
return EXIT_FAILURE;
}
+#ifdef __FreeBSD__
+ addr.sun_len = sizeof(addr.sun_path);
+ (void)strlcpy(addr.sun_path, argv[1], addr.sun_len);
+#else
(void)strlcpy(addr.sun_path, argv[1], sizeof(addr.sun_path));
+#endif
addr.sun_family = PF_UNIX;
-
error = bind(fd, (struct sockaddr *)&addr, sizeof(addr));
if (error == -1) {
warn("connect");
+#ifdef __FreeBSD__
+ (void)close(fd);
+#endif
return EXIT_FAILURE;
}
More information about the svn-src-all
mailing list