remove sys/param.h that snuck in, by replacing MAXPATHLEN with PATH_MAX
authorderaadt <deraadt@openbsd.org>
Thu, 2 Sep 2021 21:06:06 +0000 (21:06 +0000)
committerderaadt <deraadt@openbsd.org>
Thu, 2 Sep 2021 21:06:06 +0000 (21:06 +0000)
usr.bin/rsync/flist.c
usr.bin/rsync/session.c
usr.bin/rsync/symlinks.c

index 86cde1d..392ba49 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: flist.c,v 1.33 2021/08/29 13:43:46 claudio Exp $ */
+/*     $OpenBSD: flist.c,v 1.34 2021/09/02 21:06:06 deraadt Exp $ */
 /*
  * Copyright (c) 2019 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2019 Florian Obser <florian@openbsd.org>
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
-#include <sys/param.h>
 #include <sys/stat.h>
 
 #include <assert.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <fts.h>
+#include <limits.h>
 #include <inttypes.h>
 #include <search.h>
 #include <stdio.h>
@@ -428,7 +428,7 @@ out:
  */
 static int
 flist_recv_name(struct sess *sess, int fd, struct flist *f, uint8_t flags,
-    char last[MAXPATHLEN])
+    char last[PATH_MAX])
 {
        uint8_t          bval;
        size_t           partial = 0;
@@ -504,7 +504,7 @@ flist_recv_name(struct sess *sess, int fd, struct flist *f, uint8_t flags,
 
        /* Record our last path and construct our filename. */
 
-       strlcpy(last, f->path, MAXPATHLEN);
+       strlcpy(last, f->path, PATH_MAX);
        f->wpath = f->path;
        return 1;
 }
@@ -593,7 +593,7 @@ flist_recv(struct sess *sess, int fd, struct flist **flp, size_t *sz)
        const struct flist *fflast = NULL;
        size_t           flsz = 0, flmax = 0, lsz, gidsz = 0, uidsz = 0;
        uint8_t          flag;
-       char             last[MAXPATHLEN];
+       char             last[PATH_MAX];
        int64_t          lval; /* temporary values... */
        int32_t          ival;
        uint32_t         uival;
index d87e45c..0b43036 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: session.c,v 1.8 2021/06/30 13:10:04 claudio Exp $ */
+/*     $OpenBSD: session.c,v 1.9 2021/09/02 21:06:06 deraadt Exp $ */
 /*
  * Copyright (c) 2019 Kristaps Dzonsons <kristaps@bsd.lv>
  *
@@ -14,7 +14,6 @@
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
-#include <sys/param.h>
 
 #include <assert.h>
 #include <stdint.h>
index 18bdf8d..cc4527c 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: symlinks.c,v 1.6 2021/06/30 13:10:04 claudio Exp $ */
+/*     $OpenBSD: symlinks.c,v 1.7 2021/09/02 21:06:06 deraadt Exp $ */
 /*
  * Copyright (c) 2019 Kristaps Dzonsons <kristaps@bsd.lv>
  *
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
-#include <sys/param.h>
 
 #include <assert.h>
 #include <stdint.h>
 #include <stdlib.h>
+#include <limits.h>
 #include <unistd.h>
 
 #include "extern.h"
@@ -36,7 +36,7 @@ symlink_read(const char *path)
        ssize_t  nsz = 0;
        void    *pp;
 
-       for (sz = MAXPATHLEN; ; sz *= 2) {
+       for (sz = PATH_MAX; ; sz *= 2) {
                if ((pp = realloc(buf, sz + 1)) == NULL) {
                        ERR("realloc");
                        free(buf);
@@ -75,7 +75,7 @@ symlinkat_read(int fd, const char *path)
        ssize_t  nsz = 0;
        void    *pp;
 
-       for (sz = MAXPATHLEN; ; sz *= 2) {
+       for (sz = PATH_MAX; ; sz *= 2) {
                if ((pp = realloc(buf, sz + 1)) == NULL) {
                        ERR("realloc");
                        free(buf);