Change some args from "char *" to "const char *" in order to get rid of
authorWayne Davison <wayned@samba.org>
Thu, 15 Jan 2009 08:14:51 +0000 (00:14 -0800)
committerWayne Davison <wayned@samba.org>
Thu, 15 Jan 2009 08:23:07 +0000 (00:23 -0800)
a compiler warning that was just introduced.  Also avoids changing the
host string to lower-case in access.c (by using iwildmatch()).

access.c
clientserver.c

index 772ec27..9a023de 100644 (file)
--- a/access.c
+++ b/access.c
 
 #include "rsync.h"
 
-static int match_hostname(char *host, char *tok)
+static int match_hostname(const char *host, const char *tok)
 {
        if (!host || !*host)
                return 0;
-       return wildmatch(tok, host);
+       return iwildmatch(tok, host);
 }
 
-static int match_binary(char *b1, char *b2, char *mask, int addrlen)
+static int match_binary(const char *b1, const char *b2, const char *mask, int addrlen)
 {
        int i;
 
@@ -56,7 +56,7 @@ static void make_mask(char *mask, int plen, int addrlen)
        return;
 }
 
-static int match_address(char *addr, char *tok)
+static int match_address(const char *addr, const char *tok)
 {
        char *p;
        struct addrinfo hints, *resa, *rest;
@@ -210,7 +210,7 @@ static int match_address(char *addr, char *tok)
        return ret;
 }
 
-static int access_match(char *list, char *addr, char *host)
+static int access_match(const char *list, const char *addr, const char *host)
 {
        char *tok;
        char *list2 = strdup(list);
@@ -219,8 +219,6 @@ static int access_match(char *list, char *addr, char *host)
                out_of_memory("access_match");
 
        strlower(list2);
-       if (host)
-               strlower(host);
 
        for (tok = strtok(list2, " ,\t"); tok; tok = strtok(NULL, " ,\t")) {
                if (match_hostname(host, tok) || match_address(addr, tok)) {
@@ -233,7 +231,8 @@ static int access_match(char *list, char *addr, char *host)
        return 0;
 }
 
-int allow_access(char *addr, char *host, char *allow_list, char *deny_list)
+int allow_access(const char *addr, const char *host,
+                const char *allow_list, const char *deny_list)
 {
        if (allow_list && !*allow_list)
                allow_list = NULL;
index 1bc5b9a..059be75 100644 (file)
@@ -408,7 +408,7 @@ static int path_failure(int f_out, const char *dir, BOOL was_chdir)
        return -1;
 }
 
-static int rsync_module(int f_in, int f_out, int i, char *addr, char *host)
+static int rsync_module(int f_in, int f_out, int i, const char *addr, const char *host)
 {
        int argc;
        char **argv, **orig_argv, **orig_early_argv, *module_chdir;
@@ -922,7 +922,7 @@ static int load_config(int globals_only)
 int start_daemon(int f_in, int f_out)
 {
        char line[1024];
-       char *addr, *host;
+       const char *addr, *host;
        int i;
 
        io_set_sock_fds(f_in, f_out);