ypxfr(8): Ansify, raise WARNS to 3, fix warnings.
authorSascha Wildner <saw@online.de>
Thu, 19 Feb 2009 11:00:30 +0000 (12:00 +0100)
committerSascha Wildner <saw@online.de>
Thu, 19 Feb 2009 11:00:46 +0000 (12:00 +0100)
libexec/ypxfr/Makefile
libexec/ypxfr/yp_dbwrite.c
libexec/ypxfr/ypxfr_extern.h
libexec/ypxfr/ypxfr_getmap.c
libexec/ypxfr/ypxfr_main.c
libexec/ypxfr/ypxfr_misc.c
libexec/ypxfr/ypxfrd_getmap.c

index 8d6bf64..dd090c0 100644 (file)
@@ -10,6 +10,7 @@ GENSRCS=yp.h yp_clnt.c ypxfr_clnt.c
 
 .PATH: ${.CURDIR}/../../usr.sbin/ypserv
 
+WARNS?=        3
 MAN=   ypxfr.8
 
 CFLAGS+= -I.
index e858467..da1a55b 100644 (file)
 /*
  * Open a DB database read/write
  */
-DB *yp_open_db_rw(domain, map, flags)
-       const char *domain;
-       const char *map;
-       const int flags;
+DB *
+yp_open_db_rw(const char *domain, const char *map, const int flags)
 {
        DB *dbp;
        char buf[1025];
@@ -89,11 +87,8 @@ DB *yp_open_db_rw(domain, map, flags)
        return (dbp);
 }
 
-int yp_put_record(dbp,key,data,allow_overwrite)
-       DB *dbp;
-       DBT *key;
-       DBT *data;
-       int allow_overwrite;
+int
+yp_put_record(DB *dbp, DBT *key, DBT *data, int allow_overwrite)
 {
        int rval;
 
index adef646..2fa8eb3 100644 (file)
@@ -45,7 +45,7 @@ extern BTREEINFO openinfo_b;
 #define _PATH_YP "/var/yp/"
 #endif
 
-extern char    *yp_dir;
+extern const char *yp_dir;
 extern int     debug;
 extern enum ypstat     yp_errno;
 extern void    yp_error(const char *, ...);
@@ -55,7 +55,7 @@ extern DB     *yp_open_db_rw(const char *, const char *, const int);
 extern void    yp_init_dbs(void);
 extern int     yp_put_record(DB *, DBT *, DBT *, int);
 extern int     yp_get_record(const char *, const char *, const DBT *, DBT *, int);
-extern int     ypxfr_get_map(char *, char *, char *, int (*)());
+extern int     ypxfr_get_map(char *, char *, char *, int (*)(int, char *, int, char *, int, char *));
 extern char    *ypxfr_get_master(char *, char *, char *, const int);
 extern unsigned        long ypxfr_get_order(char *, char *, char *, const int);
 extern int     ypxfr_match(char *, char *, char *, char *, unsigned long);
index f6be911..29cad5d 100644 (file)
@@ -43,7 +43,7 @@
 
 extern bool_t xdr_ypresp_all_seq(XDR *, unsigned long *);
 
-int (*ypresp_allfn)();
+int (*ypresp_allfn)(int, char *, int, char *, int, char *);
 void *ypresp_data;
 extern DB *specdbp;
 extern enum ypstat yp_errno;
@@ -57,11 +57,9 @@ extern enum ypstat yp_errno;
  * talking to one of the slaves instead. We do need to dig into libc
  * a little though, since it contains the magic XDR function we need.
  */
-int ypxfr_get_map(map, domain, host, callback)
-       char *map;
-       char *domain;
-       char *host;
-       int (*callback)();
+int
+ypxfr_get_map(char *map, char *domain, char *host,
+    int (*callback)(int, char *, int, char *, int, char *))
 {
        CLIENT *clnt;
        ypreq_nokey req;
@@ -84,8 +82,8 @@ create tcp handle"));
        ypresp_allfn = callback;
        ypresp_data = NULL;
 
-       (void)clnt_call(clnt, YPPROC_ALL, xdr_ypreq_nokey, &req,
-               xdr_ypresp_all_seq, &status, timeout);
+       (void)clnt_call(clnt, YPPROC_ALL, (xdrproc_t)xdr_ypreq_nokey, &req,
+           (xdrproc_t)xdr_ypresp_all_seq, &status, timeout);
 
        clnt_destroy(clnt);
 
index 5584990..ec9091e 100644 (file)
@@ -52,8 +52,8 @@ struct dom_binding {};
 #include <rpcsvc/ypxfrd.h>
 #include "ypxfr_extern.h"
 
-char *progname = "ypxfr";
-char *yp_dir = _PATH_YP;
+const char *progname = "ypxfr";
+const char *yp_dir = _PATH_YP;
 int _rpcpmstart = 0;
 int ypxfr_use_yplib = 0; /* Assume the worst. */
 int ypxfr_clear = 1;
@@ -62,9 +62,8 @@ struct sockaddr_in ypxfr_callback_addr;
 struct yppushresp_xfr ypxfr_resp;
 DB *dbp;
 
-static void ypxfr_exit(retval, temp)
-       ypxfrstat retval;
-       char *temp;
+static void
+ypxfr_exit(ypxfrstat retval, char *temp)
 {
        CLIENT *clnt;
        int sock = RPC_ANYSOCK;
@@ -105,7 +104,8 @@ static void ypxfr_exit(retval, temp)
        exit(0);
 }
 
-static void usage()
+static void
+usage(void)
 {
        if (_rpcpmstart) {
                ypxfr_exit(YPXFR_BADARGS,NULL);
@@ -118,13 +118,9 @@ static void usage()
        }
 }
 
-int ypxfr_foreach(status, key, keylen, val, vallen, data)
-       int status;
-       char *key;
-       int keylen;
-       char *val;
-       int vallen;
-       char *data;
+static int
+ypxfr_foreach(int status, char *key, int keylen, char *val, int vallen,
+    char *data __unused)
 {
        DBT dbkey, dbval;
 
@@ -159,9 +155,7 @@ int ypxfr_foreach(status, key, keylen, val, vallen, data)
 }
 
 int
-main(argc,argv)
-       int argc;
-       char *argv[];
+main(int argc, char *argv[])
 {
        int ch;
        int ypxfr_force = 0;
@@ -402,7 +396,7 @@ the local domain name isn't set");
                                break;
                        }
                }
-               if (!ignore && ypxfr_order <= atoi(data.data))
+               if (!ignore && ypxfr_order <= (unsigned)atoi(data.data))
                        ypxfr_exit(YPXFR_AGE, NULL);
 
        }
@@ -445,7 +439,7 @@ the local domain name isn't set");
 
        if (yp_put_record(dbp, &key, &data, 0) != YP_TRUE) {
                yp_error("failed to write order number to database");
-               ypxfr_exit(YPXFR_DBM,&ypxfr_temp_map);
+               ypxfr_exit(YPXFR_DBM, ypxfr_temp_map);
        }
 
        key.data = "YP_MASTER_NAME";
@@ -455,7 +449,7 @@ the local domain name isn't set");
 
        if (yp_put_record(dbp, &key, &data, 0) != YP_TRUE) {
                yp_error("failed to write master name to database");
-               ypxfr_exit(YPXFR_DBM,&ypxfr_temp_map);
+               ypxfr_exit(YPXFR_DBM, ypxfr_temp_map);
        }
 
        key.data = "YP_DOMAIN_NAME";
@@ -465,7 +459,7 @@ the local domain name isn't set");
 
        if (yp_put_record(dbp, &key, &data, 0) != YP_TRUE) {
                yp_error("failed to write domain name to database");
-               ypxfr_exit(YPXFR_DBM,&ypxfr_temp_map);
+               ypxfr_exit(YPXFR_DBM, ypxfr_temp_map);
        }
 
        snprintf (buf, sizeof(buf), "%s:%s", ypxfr_source_host, ypxfr_mapname);
@@ -477,7 +471,7 @@ the local domain name isn't set");
 
        if (yp_put_record(dbp, &key, &data, 0) != YP_TRUE) {
                yp_error("failed to write input name to database");
-               ypxfr_exit(YPXFR_DBM,&ypxfr_temp_map);
+               ypxfr_exit(YPXFR_DBM, ypxfr_temp_map);
 
        }
 
@@ -491,7 +485,7 @@ the local domain name isn't set");
 
        if (yp_put_record(dbp, &key, &data, 0) != YP_TRUE) {
                yp_error("failed to write output name to database");
-               ypxfr_exit(YPXFR_DBM,&ypxfr_temp_map);
+               ypxfr_exit(YPXFR_DBM, ypxfr_temp_map);
        }
 
        if (interdom) {
@@ -502,7 +496,7 @@ the local domain name isn't set");
 
                if (yp_put_record(dbp, &key, &data, 0) != YP_TRUE) {
                        yp_error("failed to add interdomain flag to database");
-                       ypxfr_exit(YPXFR_DBM,&ypxfr_temp_map);
+                       ypxfr_exit(YPXFR_DBM, ypxfr_temp_map);
                }
        }
 
@@ -514,7 +508,7 @@ the local domain name isn't set");
 
                if (yp_put_record(dbp, &key, &data, 0) != YP_TRUE) {
                        yp_error("failed to add secure flag to database");
-                       ypxfr_exit(YPXFR_DBM,&ypxfr_temp_map);
+                       ypxfr_exit(YPXFR_DBM, ypxfr_temp_map);
                }
        }
 
@@ -523,7 +517,7 @@ the local domain name isn't set");
        if (ypxfr_get_map(ypxfr_mapname,ypxfr_source_domain,
                          ypxfr_source_host, ypxfr_foreach)){
                yp_error("failed to retrieve map from source host");
-               ypxfr_exit(YPXFR_YPERR,&ypxfr_temp_map);
+               ypxfr_exit(YPXFR_YPERR, ypxfr_temp_map);
        }
 
        (void)(dbp->close)(dbp);
@@ -541,11 +535,11 @@ leave:
                yp_error("failed to get order number of %s: %s",
                                ypxfr_mapname, yp_errno == YPXFR_SUCC ?
                                "map has order 0" : ypxfrerr_string(yp_errno));
-               ypxfr_exit(YPXFR_YPERR,&ypxfr_temp_map);
+               ypxfr_exit(YPXFR_YPERR, ypxfr_temp_map);
        }
 
        if (ypxfr_order != ypxfr_skew_check)
-               ypxfr_exit(YPXFR_SKEW,&ypxfr_temp_map);
+               ypxfr_exit(YPXFR_SKEW, ypxfr_temp_map);
 
        /*
         * Send a YPPROC_CLEAR to the local ypserv.
@@ -555,11 +549,11 @@ leave:
                char *out = NULL;
                int stat;
                if ((stat = callrpc("localhost",YPPROG,YPVERS,YPPROC_CLEAR,
-                       xdr_void, (void *)&in,
-                       xdr_void, (void *)out)) != RPC_SUCCESS) {
+                       (xdrproc_t)xdr_void, (void *)&in,
+                       (xdrproc_t)xdr_void, (void *)out)) != RPC_SUCCESS) {
                        yp_error("failed to send 'clear' to local ypserv: %s",
                                 clnt_sperrno((enum clnt_stat) stat));
-                       ypxfr_exit(YPXFR_CLEAR, &ypxfr_temp_map);
+                       ypxfr_exit(YPXFR_CLEAR, ypxfr_temp_map);
                }
        }
 
index b5adda3..839129f 100644 (file)
@@ -44,8 +44,8 @@ struct dom_binding {};
 #include <rpcsvc/ypclnt.h>
 #include "ypxfr_extern.h"
 
-char *ypxfrerr_string(code)
-       ypxfrstat code;
+char *
+ypxfrerr_string(ypxfrstat code)
 {
        switch (code) {
        case YPXFR_SUCC:
@@ -118,11 +118,8 @@ char *ypxfrerr_string(code)
  * allocated by the XDR routines. We have to rememver to free() or
  * xdr_free() the memory as required to avoid leaking memory.
  */
-char *ypxfr_get_master(domain,map,source,yplib)
-       char *domain;
-       char *map;
-       char *source;
-       const int yplib;
+char *
+ypxfr_get_master(char *domain, char *map, char *source, const int yplib)
 {
        static char mastername[MAXPATHLEN + 2];
 
@@ -193,11 +190,8 @@ failed"));
        }
 }
 
-unsigned long ypxfr_get_order(domain, map, source, yplib)
-       char *domain;
-       char *map;
-       char *source;
-       const int yplib;
+unsigned long
+ypxfr_get_order(char *domain, char *map, char *source, const int yplib)
 {
        if (yplib) {
                unsigned long order;
@@ -258,12 +252,9 @@ failed"));
        }
 }
 
-int ypxfr_match(server, domain, map, key, keylen)
-       char *server;
-       char *domain;
-       char *map;
-       char *key;
-       unsigned long keylen;
+int
+ypxfr_match(char *server, char *domain, char *map, char *key,
+    unsigned long keylen)
 {
        ypreq_key ypkey;
        ypresp_val *ypval;
@@ -293,11 +284,11 @@ int ypxfr_match(server, domain, map, key, keylen)
        clnt_destroy(clnt);
 
        if (ypval->stat != YP_TRUE) {
-               xdr_free(xdr_ypresp_val, (char *)ypval);
+               xdr_free((xdrproc_t)xdr_ypresp_val, ypval);
                return(0);
        }
 
-       xdr_free(xdr_ypresp_val, (char *)ypval);
+       xdr_free((xdrproc_t)xdr_ypresp_val, ypval);
 
        return(1);
 }
index 91d5c01..9ee331e 100644 (file)
@@ -49,7 +49,8 @@
 
 int fp = 0;
 
-static bool_t xdr_my_xfr(register XDR *xdrs, xfr *objp)
+static bool_t
+xdr_my_xfr(register XDR *xdrs, xfr *objp)
 {
        while (1) {
                if (!xdr_xfr(xdrs, objp))
@@ -61,7 +62,7 @@ static bool_t xdr_my_xfr(register XDR *xdrs, xfr *objp)
                                return(FALSE);
                        }
                }
-               xdr_free(xdr_xfr, (char *)objp);
+               xdr_free((xdrproc_t)xdr_xfr, objp);
                if (objp->ok == FALSE) {
                        switch (objp->xfr_u.xfrstat) {
                        case(XFR_DONE):
@@ -98,11 +99,8 @@ static bool_t xdr_my_xfr(register XDR *xdrs, xfr *objp)
 
 #define PERM_SECURE (S_IRUSR|S_IWUSR)
 
-int ypxfrd_get_map(host, map, domain, tmpname)
-       char *host;
-       char *map;
-       char *domain;
-       char *tmpname;
+int
+ypxfrd_get_map(char *host, char *map, char *domain, char *tmpname)
 {
        CLIENT *clnt;
        struct ypxfr_mapname req;
@@ -131,8 +129,10 @@ int ypxfrd_get_map(host, map, domain, tmpname)
                return(1);
        }
 
-       if (clnt_call(clnt,YPXFRD_GETMAP,xdr_ypxfr_mapname,(char *)&req,
-                       xdr_my_xfr, (char *)&resp, timeout) != RPC_SUCCESS) {
+       if (clnt_call(clnt, YPXFRD_GETMAP,
+                       (xdrproc_t)xdr_ypxfr_mapname, (char *)&req,
+                       (xdrproc_t)xdr_my_xfr, (char *)&resp,
+                       timeout) != RPC_SUCCESS) {
                yp_error("%s", clnt_sperror(clnt,"call to rpc.ypxfrd failed"));
                status++;
                unlink(tmpname);