diff options
-rw-r--r-- | usr.bin/rdist/Makefile | 3 | ||||
-rw-r--r-- | usr.bin/rdist/client.c | 36 | ||||
-rw-r--r-- | usr.bin/rdist/defs.h | 12 | ||||
-rw-r--r-- | usr.bin/rdist/docmd.c | 12 | ||||
-rw-r--r-- | usr.bin/rdist/expand.c | 4 | ||||
-rw-r--r-- | usr.bin/rdist/gram.y | 4 | ||||
-rw-r--r-- | usr.bin/rdist/lookup.c | 4 | ||||
-rw-r--r-- | usr.bin/rdistd/Makefile | 3 | ||||
-rw-r--r-- | usr.bin/rdistd/server.c | 21 |
9 files changed, 51 insertions, 48 deletions
diff --git a/usr.bin/rdist/Makefile b/usr.bin/rdist/Makefile index 98f0126822b..94e53be7a8c 100644 --- a/usr.bin/rdist/Makefile +++ b/usr.bin/rdist/Makefile @@ -1,4 +1,4 @@ -# $OpenBSD: Makefile,v 1.15 2004/01/05 02:55:28 espie Exp $ +# $OpenBSD: Makefile,v 1.16 2012/11/12 01:14:41 guenther Exp $ PROG= rdist @@ -6,6 +6,7 @@ CFLAGS+=-I. -I${.CURDIR} -DOS_H=\"os-openbsd.h\" SRCS= gram.y child.c client.c common.c distopt.c docmd.c expand.c \ isexec.c lookup.c message.c rdist.c CLEANFILES+=gram.c y.tab.h +WARNINGS=yes .include <bsd.prog.mk> diff --git a/usr.bin/rdist/client.c b/usr.bin/rdist/client.c index 82844eaa8c1..076391dfa84 100644 --- a/usr.bin/rdist/client.c +++ b/usr.bin/rdist/client.c @@ -1,4 +1,4 @@ -/* $OpenBSD: client.c,v 1.23 2011/04/10 15:47:28 krw Exp $ */ +/* $OpenBSD: client.c,v 1.24 2012/11/12 01:14:41 guenther Exp $ */ /* * Copyright (c) 1983 Regents of the University of California. @@ -365,7 +365,7 @@ sendhardlink(opt_t opts, struct linkbuf *lp, char *rname, int destdir) } ENCODE(elname, lname); ENCODE(ername, rname); - (void) sendcmd(C_RECVHARDLINK, "%o %s %s", + (void) sendcmd(C_RECVHARDLINK, "%lo %s %s", opts, elname, ername); return(response()); @@ -399,9 +399,9 @@ sendfile(char *rname, opt_t opts, struct stat *stb, char *user, */ ENCODE(ername, rname); - (void) sendcmd(C_RECVREG, "%o %04o %lld %ld %ld %s %s %s", + (void) sendcmd(C_RECVREG, "%lo %04o %lld %lld %lld %s %s %s", opts, stb->st_mode & 07777, (long long) stb->st_size, - stb->st_mtime, stb->st_atime, + (long long)stb->st_mtime, (long long)stb->st_atime, user, group, ername); if (response() < 0) { (void) close(f); @@ -588,7 +588,7 @@ senddir(char *rname, opt_t opts, struct stat *stb, char *user, * Send recvdir command in recvit() format. */ ENCODE(ername, rname); - (void) sendcmd(C_RECVDIR, "%o %04o 0 0 0 %s %s %s", + (void) sendcmd(C_RECVDIR, "%lo %04o 0 0 0 %s %s %s", opts, stb->st_mode & 07777, user, group, ername); if (response() < 0) return(-1); @@ -597,7 +597,7 @@ senddir(char *rname, opt_t opts, struct stat *stb, char *user, * Don't descend into directory */ if (IS_ON(opts, DO_NODESCEND)) - return(0); + goto out; if (IS_ON(opts, DO_REMOVE)) if (rmchk(opts) > 0) @@ -653,7 +653,7 @@ sendlink(char *rname, opt_t opts, struct stat *stb, char *user, u_char *s; char ername[MAXPATHLEN*4]; - debugmsg(DM_CALL, "sendlink(%s, %x, stb, %d)\n", rname, opts, destdir); + debugmsg(DM_CALL, "sendlink(%s, %lx, stb, %d)\n", rname, opts, destdir); if (stb->st_nlink > 1) { struct linkbuf *lp; @@ -666,9 +666,9 @@ sendlink(char *rname, opt_t opts, struct stat *stb, char *user, * Gather and send basic link info */ ENCODE(ername, rname); - (void) sendcmd(C_RECVSYMLINK, "%o %04o %lld %ld %ld %s %s %s", + (void) sendcmd(C_RECVSYMLINK, "%lo %04o %lld %lld %lld %s %s %s", opts, stb->st_mode & 07777, (long long) stb->st_size, - stb->st_mtime, stb->st_atime, + (long long)stb->st_mtime, (long long)stb->st_atime, user, group, ername); if (response() < 0) return(-1); @@ -772,7 +772,7 @@ update(char *rname, opt_t opts, struct stat *statp) u_char *cp; char ername[MAXPATHLEN*4]; - debugmsg(DM_CALL, "update(%s, 0x%x, 0x%x)\n", rname, opts, statp); + debugmsg(DM_CALL, "update(%s, 0x%lx, %p)\n", rname, opts, statp); switch (statp->st_mode & S_IFMT) { case S_IFBLK: @@ -921,8 +921,8 @@ update(char *rname, opt_t opts, struct stat *statp) debugmsg(DM_MISC, "update(%s,) local mode %04o remote mode %04o\n", rname, lmode, rmode); - debugmsg(DM_MISC, "update(%s,) size %lld mtime %d owner '%s' grp '%s'" - "\n", rname, (long long) size, mtime, owner, group); + debugmsg(DM_MISC, "update(%s,) size %lld mtime %lld owner '%s' grp '%s'" + "\n", rname, (long long) size, (long long)mtime, owner, group); if (statp->st_mtime != mtime) { if (statp->st_mtime < mtime && IS_ON(opts, DO_YOUNGER)) { @@ -1048,7 +1048,7 @@ statupdate(int u, char *target, opt_t opts, char *rname, int destdir, message(MT_CHANGE, "%s: change to perm %04o, owner %s, group %s", target, lmode, user, group); ENCODE(ername, rname); - (void) sendcmd(C_CHMOG, "%o %04o %s %s %s", + (void) sendcmd(C_CHMOG, "%lo %04o %s %s %s", opts, lmode, user, group, ername); (void) response(); } @@ -1141,7 +1141,7 @@ sendit(char *rname, opt_t opts, int destdir) if (checkfilename(rname) != 0) return(-1); - debugmsg(DM_CALL, "sendit(%s, 0x%x) called\n", rname, opts); + debugmsg(DM_CALL, "sendit(%s, 0x%lx) called\n", rname, opts); if (except(target)) return(0); @@ -1153,7 +1153,7 @@ sendit(char *rname, opt_t opts, int destdir) * Does rname need updating? */ u = update(rname, opts, &stb); - debugmsg(DM_MISC, "sendit(%s, 0x%x): update status of %s is %d\n", + debugmsg(DM_MISC, "sendit(%s, 0x%lx): update status of %s is %d\n", rname, opts, target, u); /* @@ -1218,7 +1218,7 @@ install(char *src, char *dest, int ddir, int destdir, opt_t opts) char ername[MAXPATHLEN*4]; debugmsg(DM_CALL, - "install(src=%s,dest=%s,ddir=%d,destdir=%d,opts=%d) start\n", + "install(src=%s,dest=%s,ddir=%d,destdir=%d,opts=%ld) start\n", (src?src:"NULL"), (dest?dest:"NULL"), ddir, destdir, opts); /* * Save source name @@ -1288,9 +1288,9 @@ install(char *src, char *dest, int ddir, int destdir, opt_t opts) */ ENCODE(ername, dest); if (ddir) - (void) sendcmd(C_DIRTARGET, "%o %s", opts, ername); + (void) sendcmd(C_DIRTARGET, "%lo %s", opts, ername); else - (void) sendcmd(C_TARGET, "%o %s", opts, ername); + (void) sendcmd(C_TARGET, "%lo %s", opts, ername); if (response() < 0) return(-1); diff --git a/usr.bin/rdist/defs.h b/usr.bin/rdist/defs.h index 5dc77403c04..687f61d2799 100644 --- a/usr.bin/rdist/defs.h +++ b/usr.bin/rdist/defs.h @@ -1,4 +1,4 @@ -/* $OpenBSD: defs.h,v 1.17 2011/04/10 15:47:28 krw Exp $ */ +/* $OpenBSD: defs.h,v 1.18 2012/11/12 01:14:41 guenther Exp $ */ #ifndef __DEFS_H__ #define __DEFS_H__ @@ -368,7 +368,7 @@ void finish(void); void lostconn(void); void coredump(void); void sighandler(int); -int sendcmd(char, char *, ...); +int sendcmd(char, char *, ...) __attribute__((__format__ (printf, 2, 3))); int remline(u_char *, int, int); ssize_t readrem(char *, ssize_t); char *getusername(UID_T, char *, opt_t); @@ -449,10 +449,10 @@ void msgprusage(void); void msgprconfig(void); char *msgparseopts(char *, int); void checkhostname(void); -void message(int, char *, ...); -void debugmsg(int, char *, ...); -void error(char *, ...); -void fatalerr(char *, ...); +void message(int, char *, ...) __attribute__((__format__ (printf, 2, 3))); +void debugmsg(int, char *, ...) __attribute__((__format__ (printf, 2, 3))); +void error(char *, ...) __attribute__((__format__ (printf, 1, 2))); +void fatalerr(char *, ...) __attribute__((__format__ (printf, 1, 2))); char *getnotifyfile(void); /* rdist.c */ diff --git a/usr.bin/rdist/docmd.c b/usr.bin/rdist/docmd.c index fb14ded7830..ad0645c2b9f 100644 --- a/usr.bin/rdist/docmd.c +++ b/usr.bin/rdist/docmd.c @@ -1,4 +1,4 @@ -/* $OpenBSD: docmd.c,v 1.24 2011/04/21 02:44:15 krw Exp $ */ +/* $OpenBSD: docmd.c,v 1.25 2012/11/12 01:14:41 guenther Exp $ */ /* * Copyright (c) 1983 Regents of the University of California. @@ -470,7 +470,7 @@ doarrow(struct cmd *cmd, char **filev) return; } - debugmsg(DM_CALL, "doarrow(%x, %s, %x) start", + debugmsg(DM_CALL, "doarrow(%p, %s, %p) start", files, A(rhost), sbcmds); if (nflag) @@ -534,7 +534,7 @@ doarrow(struct cmd *cmd, char **filev) destdir = ddir; debugmsg(DM_MISC, - "Debug files->n_next= %d, destdir=%d, ddir=%d", + "Debug files->n_next= %p, destdir=%d, ddir=%d", files->n_next, destdir, ddir); if (!sc->sc_name || !*sc->sc_name) { @@ -543,7 +543,7 @@ doarrow(struct cmd *cmd, char **filev) } debugmsg(DM_MISC, - "Debug sc->sc_name=%x, destdir=%d, ddir=%d", + "Debug sc->sc_name=%p, destdir=%d, ddir=%d", sc->sc_name, destdir, ddir); for (f = files; f != NULL; f = f->n_next) { @@ -641,7 +641,7 @@ rcmptime(struct stat *st, struct subcmd *sbcmds, char **env) char *optarget; int len; - debugmsg(DM_CALL, "rcmptime(%x) start", st); + debugmsg(DM_CALL, "rcmptime(%p) start", st); if ((d = opendir((char *) target)) == NULL) { error("%s: open directory failed: %s", target, SYSERR); @@ -762,7 +762,7 @@ dodcolon(struct cmd *cmd, char **filev) return; } - debugmsg(DM_MISC, "%s: mtime %d\n", stamp, stb.st_mtime); + debugmsg(DM_MISC, "%s: mtime %lld\n", stamp, (long long)stb.st_mtime); env = NULL; for (sc = sbcmds; sc != NULL; sc = sc->sc_next) { diff --git a/usr.bin/rdist/expand.c b/usr.bin/rdist/expand.c index 4e47670778d..f9d86419034 100644 --- a/usr.bin/rdist/expand.c +++ b/usr.bin/rdist/expand.c @@ -1,4 +1,4 @@ -/* $OpenBSD: expand.c,v 1.12 2009/10/27 23:59:42 deraadt Exp $ */ +/* $OpenBSD: expand.c,v 1.13 2012/11/12 01:14:41 guenther Exp $ */ /* * Copyright (c) 1983 Regents of the University of California. @@ -112,7 +112,7 @@ expand(struct namelist *list, int wh) /* quote in list->n_name */ char pathbuf[BUFSIZ]; if (debug) - debugmsg(DM_CALL, "expand(%x, %d) start, list = %s", + debugmsg(DM_CALL, "expand(%p, %d) start, list = %s", list, wh, getnlstr(list)); if (wh == 0) diff --git a/usr.bin/rdist/gram.y b/usr.bin/rdist/gram.y index 097583d2d8c..3f321cee479 100644 --- a/usr.bin/rdist/gram.y +++ b/usr.bin/rdist/gram.y @@ -1,5 +1,5 @@ %{ -/* $OpenBSD: gram.y,v 1.9 2009/10/27 23:59:42 deraadt Exp $ */ +/* $OpenBSD: gram.y,v 1.10 2012/11/12 01:14:41 guenther Exp $ */ /* * Copyright (c) 1993 Michael A. Cooper @@ -413,7 +413,7 @@ insert(char *label, struct namelist *files, struct namelist *hosts, struct cmd *c, *prev, *nc; struct namelist *h, *lasth; - debugmsg(DM_CALL, "insert(%s, %x, %x, %x) start, files = %s", + debugmsg(DM_CALL, "insert(%s, %p, %p, %p) start, files = %s", label == NULL ? "(null)" : label, files, hosts, subcmds, getnlstr(files)); diff --git a/usr.bin/rdist/lookup.c b/usr.bin/rdist/lookup.c index 868130c9d8e..71f0d7b33e5 100644 --- a/usr.bin/rdist/lookup.c +++ b/usr.bin/rdist/lookup.c @@ -1,4 +1,4 @@ -/* $OpenBSD: lookup.c,v 1.13 2009/10/27 23:59:42 deraadt Exp $ */ +/* $OpenBSD: lookup.c,v 1.14 2012/11/12 01:14:41 guenther Exp $ */ /* * Copyright (c) 1983 Regents of the University of California. @@ -118,7 +118,7 @@ lookup(char *name, int action, struct namelist *value) struct syment *s; char ebuf[BUFSIZ]; - debugmsg(DM_CALL, "lookup(%s, %d, %x)", name, action, value); + debugmsg(DM_CALL, "lookup(%s, %d, %p)", name, action, value); n = 0; for (cp = name; *cp; ) diff --git a/usr.bin/rdistd/Makefile b/usr.bin/rdistd/Makefile index 7dcca7df29d..bac7456575d 100644 --- a/usr.bin/rdistd/Makefile +++ b/usr.bin/rdistd/Makefile @@ -1,8 +1,9 @@ -# $OpenBSD: Makefile,v 1.4 2003/05/06 22:10:11 millert Exp $ +# $OpenBSD: Makefile,v 1.5 2012/11/12 01:14:41 guenther Exp $ PROG= rdistd SRCS= common.c filesys-os.c filesys.c message.c rdistd.c server.c .PATH: ${.CURDIR}/../rdist CFLAGS+=-I${.CURDIR}/../rdist -DOS_H=\"os-openbsd.h\" +WARNINGS=yes .include <bsd.prog.mk> diff --git a/usr.bin/rdistd/server.c b/usr.bin/rdistd/server.c index ab4343583d2..59b1a775ef4 100644 --- a/usr.bin/rdistd/server.c +++ b/usr.bin/rdistd/server.c @@ -1,4 +1,4 @@ -/* $OpenBSD: server.c,v 1.22 2011/04/10 15:47:28 krw Exp $ */ +/* $OpenBSD: server.c,v 1.23 2012/11/12 01:14:41 guenther Exp $ */ /* * Copyright (c) 1983 Regents of the University of California. @@ -656,8 +656,9 @@ query(char *xname) case S_IFIFO: #endif #endif - (void) sendcmd(QC_YES, "%lld %ld %o %s %s", - (long long) stb.st_size, stb.st_mtime, + (void) sendcmd(QC_YES, "%lld %lld %o %s %s", + (long long) stb.st_size, + (long long) stb.st_mtime, stb.st_mode & 07777, getusername(stb.st_uid, target, options), getgroupname(stb.st_gid, target, options)); @@ -680,7 +681,7 @@ chkparent(char *name, opt_t opts) struct stat stb; int r = -1; - debugmsg(DM_CALL, "chkparent(%s, %o) start\n", name, opts); + debugmsg(DM_CALL, "chkparent(%s, %lo) start\n", name, opts); cp = strrchr(name, '/'); if (cp == NULL || cp == name) @@ -695,7 +696,7 @@ chkparent(char *name, opt_t opts) r = 0; } else debugmsg(DM_MISC, - "chkparent(%s, %o) mkdir fail: %s\n", + "chkparent(%s, %lo) mkdir fail: %s\n", name, opts, SYSERR); } } else /* It exists */ @@ -1428,7 +1429,7 @@ recvit(char *cmd, int type) /* * Get modification time */ - mtime = (time_t) strtol(cp, &cp, 10); + mtime = (time_t) strtoll(cp, &cp, 10); if (*cp++ != ' ') { error("recvit: mtime not delimited"); return; @@ -1437,7 +1438,7 @@ recvit(char *cmd, int type) /* * Get access time */ - atime = strtol(cp, &cp, 10); + atime = (time_t) strtoll(cp, &cp, 10); if (*cp++ != ' ') { error("recvit: atime not delimited"); return; @@ -1477,8 +1478,8 @@ recvit(char *cmd, int type) file = fileb; debugmsg(DM_MISC, - "recvit: opts = %04o mode = %04o size = %lld mtime = %d", - opts, mode, (long long) size, mtime); + "recvit: opts = %04lo mode = %04o size = %lld mtime = %lld", + opts, mode, (long long) size, (long long)mtime); debugmsg(DM_MISC, "recvit: owner = '%s' group = '%s' file = '%s' catname = %d isdir = %d", owner, group, file, catname, (type == S_IFDIR) ? 1 : 0); @@ -1633,7 +1634,7 @@ dochmog(char *cmd) file = fileb; debugmsg(DM_MISC, - "dochmog: opts = %04o mode = %04o", opts, mode); + "dochmog: opts = %04lo mode = %04o", opts, mode); debugmsg(DM_MISC, "dochmog: owner = '%s' group = '%s' file = '%s' catname = %d", owner, group, file, catname); |