On Mon, Dec 5, 2016 at 12:15 PM, Silvan Jegen <s.je...@gmail.com> wrote: > Hi > > Some comments below. > > On Sun, Dec 04, 2016 at 09:55:06PM -0800, Michael Forney wrote: >> diff --git a/cksum.c b/cksum.c >> index 570ca81..b53ec17 100644 >> --- a/cksum.c >> +++ b/cksum.c >> @@ -1,7 +1,9 @@ >> /* See LICENSE file for copyright and license details. */ >> +#include <fcntl.h> >> #include <inttypes.h> >> #include <stdio.h> > > This include is not needed anymore.
It is needed for printf, putchar, fputs, and stdout. >> diff --git a/od.c b/od.c >> index b5884e7..9ae1e29 100644 >> --- a/od.c >> +++ b/od.c >> @@ -1,8 +1,10 @@ >> /* See LICENSE file for copyright and license details. */ >> +#include <fcntl.h> >> #include <stdint.h> >> #include <stdio.h> > > This include is not needed anymore. It is needed for printf, fputc, and stdout. >> for (i = 0; i < argc; i++) { >> - if (!(fps[i] = fopen(argv[i], aflag ? "a" : "w"))) { >> - weprintf("fopen %s:", argv[i]); >> + if ((fds[i] = open(argv[i], O_WRONLY|O_CREAT|aflag, 0666)) < >> 0) { > > umask will be honored when creating a file but I am wondering if just > setting mode_t to 0660 would be the safer option here. POSIX says that it has to be 0666[0]: """ When a file that does not exist is created, the following features defined in the System Interfaces volume of POSIX.1-2008 shall apply unless the utility or function description states otherwise: ... 3. If the file is a regular file, the permission bits of the file shall be set to: S_IROTH | S_IWOTH | S_IRGRP | S_IWGRP | S_IRUSR | S_IWUSR (see the description of File Modes in XBD Headers, <sys/stat.h>) except that the bits specified by the file mode creation mask of the process shall be cleared. """ [0] http://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap01.html#tag_17_01_01_04