From e317068ce5d43237d1addd7de47c5724a60a75f1 Mon Sep 17 00:00:00 2001 From: truckman Date: Wed, 1 Jun 2016 17:35:03 +0000 Subject: [PATCH] MFC r300649 Fix Coverity CID 1019054 (String not null terminated) in setfacl. Increase the size of buf[] by one to allow room for a NUL character at the end. Reported by: Coverity CID: 1019054 git-svn-id: svn://svn.freebsd.org/base/stable/10@301149 ccf9f872-aa2e-dd11-9fc8-001c23d0bc1f --- bin/setfacl/file.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bin/setfacl/file.c b/bin/setfacl/file.c index c44f39819..e5e19a247 100644 --- a/bin/setfacl/file.c +++ b/bin/setfacl/file.c @@ -43,7 +43,7 @@ acl_t get_acl_from_file(const char *filename) { FILE *file; - char buf[BUFSIZ]; + char buf[BUFSIZ+1]; if (filename == NULL) err(1, "(null) filename in get_acl_from_file()"); @@ -61,7 +61,7 @@ get_acl_from_file(const char *filename) err(1, "fopen() %s failed", filename); } - fread(buf, sizeof(buf), (size_t)1, file); + fread(buf, sizeof(buf) - 1, (size_t)1, file); if (ferror(file) != 0) { fclose(file); err(1, "error reading from %s", filename); -- 2.42.0