Fix two more gcc80 -Wsizeof-pointer-memaccess warnings.
authorSascha Wildner <saw@online.de>
Wed, 24 Jan 2018 18:00:19 +0000 (19:00 +0100)
committerSascha Wildner <saw@online.de>
Wed, 24 Jan 2018 18:00:19 +0000 (19:00 +0100)
contrib/opie/libopie/insecure.c
libexec/comsat/comsat.c

index 12c09ac..f76661b 100644 (file)
@@ -135,8 +135,7 @@ int opieinsecure FUNCTION_NOARGS
        char host[sizeof(utmp.ut_host) + 1];
        insecure = 1;
 
-       strncpy(host, utmp.ut_host, sizeof(utmp.ut_host));
-       host[sizeof(utmp.ut_host)] = 0;
+       strncpy(host, utmp.ut_host, sizeof(host));
 
        if ((s = strchr(host, ':')) != NULL) {
          int n = s - host;
index 0d05e3e..706135c 100644 (file)
@@ -28,7 +28,6 @@
  *
  * @(#)comsat.c        8.1 (Berkeley) 6/4/93
  * $FreeBSD: src/libexec/comsat/comsat.c,v 1.17 2005/02/14 17:42:56 stefanf Exp $
- * $DragonFly: src/libexec/comsat/comsat.c,v 1.6 2005/05/03 17:39:03 liamfoy Exp $
  */
 
 #include <sys/param.h>
@@ -213,8 +212,7 @@ notify(struct utmp *utp, char *file, off_t offset, int folder)
        }
        tcgetattr(fileno(tp), &tio);
        cr = ((tio.c_oflag & (OPOST|ONLCR)) == (OPOST|ONLCR)) ?  "\n" : "\n\r";
-       strncpy(name, utp->ut_name, sizeof(utp->ut_name));
-       name[sizeof(name) - 1] = '\0';
+       strncpy(name, utp->ut_name, sizeof(name));
        switch (stb.st_mode & (S_IXUSR | S_IXGRP)) {
        case S_IXUSR:
        case (S_IXUSR | S_IXGRP):