Tweaked the sizeof syntax and some multi-statement lines.
[rsync/rsync.git] / authenticate.c
index 2b98119..8f4d7d1 100644 (file)
 /* support rsync authentication */
 #include "rsync.h"
 
+extern char *password_file;
+extern int am_root;
+
 /***************************************************************************
 encode a buffer using base64 - simple and slow algorithm. null terminates
 the result.
   ***************************************************************************/
-static void base64_encode(char *buf, int len, char *out)
+void base64_encode(char *buf, int len, char *out)
 {
        char *b64 = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789+/";
        int bit_offset, byte_offset, idx, i;
@@ -33,7 +36,7 @@ static void base64_encode(char *buf, int len, char *out)
 
        memset(out, 0, bytes+1);
 
-       for (i=0;i<bytes;i++) {
+       for (i = 0; i < bytes; i++) {
                byte_offset = (i*6)/8;
                bit_offset = (i*6)%8;
                if (bit_offset < 3) {
@@ -54,7 +57,7 @@ static void gen_challenge(char *addr, char *challenge)
        char input[32];
        struct timeval tv;
 
-       memset(input, 0, sizeof(input));
+       memset(input, 0, sizeof input);
 
        strlcpy((char *)input, addr, 17);
        sys_gettimeofday(&tv);
@@ -63,27 +66,25 @@ static void gen_challenge(char *addr, char *challenge)
        SIVAL(input, 24, getpid());
 
        sum_init();
-       sum_update(input, sizeof(input));
+       sum_update(input, sizeof input);
        sum_end(challenge);
 }
 
 
-/* return the secret for a user from the sercret file. maximum length
  is len. null terminate it */
+/* Return the secret for a user from the secret file, null terminated.
* Maximum length is len (not counting the null). */
 static int get_secret(int module, char *user, char *secret, int len)
 {
        char *fname = lp_secrets_file(module);
-       int fd, found=0;
-       char line[MAXPATHLEN];
-       char *p, *pass=NULL;
        STRUCT_STAT st;
-       int ok = 1;
-       extern int am_root;
+       int fd, ok = 1;
+       char ch, *p;
 
-       if (!fname || !*fname) return 0;
+       if (!fname || !*fname)
+               return 0;
 
-       fd = open(fname,O_RDONLY);
-       if (fd == -1) return 0;
+       if ((fd = open(fname, O_RDONLY)) < 0)
+               return 0;
 
        if (do_stat(fname, &st) == -1) {
                rsyserr(FERROR, errno, "stat(%s)", fname);
@@ -103,50 +104,61 @@ static int get_secret(int module, char *user, char *secret, int len)
                return 0;
        }
 
-       while (!found) {
-               int i = 0;
-               memset(line, 0, sizeof(line));
-               while (i<(sizeof(line)-1)) {
-                       if (read(fd, &line[i], 1) != 1) {
-                               memset(line, 0, sizeof(line));
-                               close(fd);
-                               return 0;
-                       }
-                       if (line[i] == '\r') continue;
-                       if (line[i] == '\n') break;
-                       i++;
+       if (*user == '#') {
+               /* Reject attempt to match a comment. */
+               close(fd);
+               return 0;
+       }
+
+       /* Try to find a line that starts with the user name and a ':'. */
+       p = user;
+       while (1) {
+               if (read(fd, &ch, 1) != 1) {
+                       close(fd);
+                       return 0;
+               }
+               if (ch == '\n')
+                       p = user;
+               else if (p) {
+                       if (*p == ch)
+                               p++;
+                       else if (!*p && ch == ':')
+                               break;
+                       else
+                               p = NULL;
                }
-               line[i] = 0;
-               if (line[0] == '#') continue;
-               p = strchr(line,':');
-               if (!p) continue;
-               *p = 0;
-               if (strcmp(user, line)) continue;
-               pass = p+1;
-               found = 1;
        }
 
+       /* Slurp the secret into the "secret" buffer. */
+       p = secret;
+       while (len > 0) {
+               if (read(fd, p, 1) != 1 || *p == '\n')
+                       break;
+               if (*p == '\r')
+                       continue;
+               p++;
+               len--;
+       }
+       *p = '\0';
        close(fd);
-       if (!found) return 0;
 
-       strlcpy(secret, pass, len);
        return 1;
 }
 
 static char *getpassf(char *filename)
 {
-       char buffer[100];
-       int fd=0;
        STRUCT_STAT st;
-       int ok = 1;
-       extern int am_root;
-       char *envpw=getenv("RSYNC_PASSWORD");
+       char buffer[512], *p;
+       int fd, n, ok = 1;
+       char *envpw = getenv("RSYNC_PASSWORD");
 
-       if (!filename) return NULL;
+       if (!filename)
+               return NULL;
 
-       if ( (fd=open(filename,O_RDONLY)) == -1) {
+       if ((fd = open(filename,O_RDONLY)) < 0) {
                rsyserr(FERROR, errno, "could not open password file \"%s\"",filename);
-               if (envpw) rprintf(FERROR,"falling back to RSYNC_PASSWORD environment variable.\n");    
+               if (envpw)
+                       rprintf(FERROR, "falling back to RSYNC_PASSWORD environment variable.\n");      
                return NULL;
        }
        
@@ -156,26 +168,27 @@ static char *getpassf(char *filename)
        } else if ((st.st_mode & 06) != 0) {
                rprintf(FERROR,"password file must not be other-accessible\n");
                ok = 0;
-       } else if (am_root && (st.st_uid != 0)) {
+       } else if (am_root && st.st_uid != 0) {
                rprintf(FERROR,"password file must be owned by root when running as root\n");
                ok = 0;
        }
        if (!ok) {
                rprintf(FERROR,"continuing without password file\n");
-               if (envpw) rprintf(FERROR,"using RSYNC_PASSWORD environment variable.\n");
+               if (envpw)
+                       rprintf(FERROR, "using RSYNC_PASSWORD environment variable.\n");
                close(fd);
                return NULL;
        }
 
-       if (envpw) rprintf(FERROR,"RSYNC_PASSWORD environment variable ignored\n");
+       if (envpw)
+               rprintf(FERROR, "RSYNC_PASSWORD environment variable ignored\n");
 
-       buffer[sizeof(buffer)-1]='\0';
-       if (read(fd,buffer,sizeof(buffer)-1) > 0)
-       {
-               char *p = strtok(buffer,"\n\r");
-               close(fd);
-               if (p) p = strdup(p);
-               return p;
+       n = read(fd, buffer, sizeof buffer - 1);
+       close(fd);
+       if (n > 0) {
+               buffer[n] = '\0';
+               if ((p = strtok(buffer, "\n\r")) != NULL)
+                       return strdup(p);
        }       
 
        return NULL;
@@ -203,7 +216,7 @@ static void generate_hash(char *in, char *challenge, char *out)
 
    otherwise return username
 */
-char *auth_server(int fd, int module, char *addr, char *leader)
+char *auth_server(int f_in, int f_out, int module, char *addr, char *leader)
 {
        char *users = lp_auth_users(module);
        char challenge[16];
@@ -216,45 +229,45 @@ char *auth_server(int fd, int module, char *addr, char *leader)
        char *tok;
 
        /* if no auth list then allow anyone in! */
-       if (!users || !*users) return "";
+       if (!users || !*users)
+               return "";
 
        gen_challenge(addr, challenge);
        
        base64_encode(challenge, 16, b64_challenge);
 
-       io_printf(fd,"%s%s\n", leader, b64_challenge);
+       io_printf(f_out, "%s%s\n", leader, b64_challenge);
 
-       if (!read_line(fd, line, sizeof(line)-1)) {
+       if (!read_line(f_in, line, sizeof line - 1))
                return NULL;
-       }
 
-       memset(user, 0, sizeof(user));
-       memset(pass, 0, sizeof(pass));
+       memset(user, 0, sizeof user);
+       memset(pass, 0, sizeof pass);
 
-       if (sscanf(line,"%99s %29s", user, pass) != 2) {
+       if (sscanf(line,"%99s %29s", user, pass) != 2)
                return NULL;
-       }
        
        users = strdup(users);
-       if (!users) return NULL;
+       if (!users)
+               return NULL;
 
        for (tok=strtok(users," ,\t"); tok; tok = strtok(NULL," ,\t")) {
-               if (fnmatch(tok, user, 0) == 0) break;
+               if (wildmatch(tok, user))
+                       break;
        }
        free(users);
 
-       if (!tok) {
+       if (!tok)
                return NULL;
-       }
        
-       memset(secret, 0, sizeof(secret));
-       if (!get_secret(module, user, secret, sizeof(secret)-1)) {
-               memset(secret, 0, sizeof(secret));
+       memset(secret, 0, sizeof secret);
+       if (!get_secret(module, user, secret, sizeof secret - 1)) {
+               memset(secret, 0, sizeof secret);
                return NULL;
        }
 
        generate_hash(secret, b64_challenge, pass2);
-       memset(secret, 0, sizeof(secret));
+       memset(secret, 0, sizeof secret);
        
        if (strcmp(pass, pass2) == 0)
                return user;
@@ -267,20 +280,26 @@ void auth_client(int fd, char *user, char *challenge)
 {
        char *pass;
        char pass2[30];
-       extern char *password_file;
 
-       if (!user || !*user) return;
+       if (!user || !*user)
+               user = "nobody";
 
-       if (!(pass=getpassf(password_file)) && !(pass=getenv("RSYNC_PASSWORD"))) {
+       if (!(pass = getpassf(password_file))
+        && !(pass = getenv("RSYNC_PASSWORD"))) {
                /* XXX: cyeoh says that getpass is deprecated, because
-                  it may return a truncated password on some systems,
-                  and it is not in the LSB. */
+                * it may return a truncated password on some systems,
+                * and it is not in the LSB.
+                 *
+                 * Andrew Klein says that getpassphrase() is present
+                 * on Solaris and reads up to 256 characters.
+                 *
+                 * OpenBSD has a readpassphrase() that might be more suitable.
+                 */
                pass = getpass("Password: ");
        }
 
-       if (!pass || !*pass) {
+       if (!pass)
                pass = "";
-       }
 
        generate_hash(pass, challenge, pass2);
        io_printf(fd, "%s %s\n", user, pass2);