]> arthur.barton.de Git - netatalk.git/commitdiff
Untabify and reindent
authorfranklahm <franklahm>
Mon, 28 Sep 2009 09:21:09 +0000 (09:21 +0000)
committerfranklahm <franklahm>
Mon, 28 Sep 2009 09:21:09 +0000 (09:21 +0000)
etc/afpd/auth.c

index c12fbba5878f34a7b6a37e16e3a61b5674254ff1..9923ac4ee0a2c0ec204f32c946d43864be48def2 100644 (file)
@@ -1,5 +1,5 @@
 /*
- * $Id: auth.c,v 1.63 2009-03-17 14:30:57 franklahm Exp $
+ * $Id: auth.c,v 1.64 2009-09-28 09:21:09 franklahm Exp $
  *
  * Copyright (c) 1990,1993 Regents of The University of Michigan.
  * All Rights Reserved.  See COPYRIGHT.
@@ -54,54 +54,54 @@ extern void afp_get_cmdline( int *ac, char ***av );
 #include "extattrs.h"
 #endif
 
-int    afp_version = 11;
+int afp_version = 11;
 static int afp_version_index;
 
-uid_t  uuid;
+uid_t   uuid;
 
 #if defined( sun ) && !defined( __svr4__ ) || defined( ultrix )
 
-int    *groups;
+int *groups;
 #define GROUPS_SIZE sizeof(int)
 
 #else /* sun __svr4__ ultrix */
 
-gid_t  *groups;
+gid_t   *groups;
 #define GROUPS_SIZE sizeof(gid_t)
 #endif /* sun ultrix */
 
-int    ngroups;
+int ngroups;
 
 /*
  * These numbers are scattered throughout the code.
  */
-static struct afp_versions     afp_versions[] = {
-            { "AFPVersion 1.1",        11 },
-            { "AFPVersion 2.0",        20 },
-            { "AFPVersion 2.1",        21 },
-            { "AFP2.2",        22 },
+static struct afp_versions  afp_versions[] = {
+    { "AFPVersion 1.1", 11 },
+    { "AFPVersion 2.0", 20 },
+    { "AFPVersion 2.1", 21 },
+    { "AFP2.2", 22 },
 #ifdef AFP3x
-            { "AFPX03", 30 },
-            { "AFP3.1", 31 },
-            { "AFP3.2", 32 }
-#endif            
-        };
+    { "AFPX03", 30 },
+    { "AFP3.1", 31 },
+    { "AFP3.2", 32 }
+#endif
+};
 
 static struct uam_mod uam_modules = {NULL, NULL, &uam_modules, &uam_modules};
 static struct uam_obj uam_login = {"", "", 0, {{NULL, NULL, NULL, NULL }}, &uam_login,
-                                      &uam_login};
+                                   &uam_login};
 static struct uam_obj uam_changepw = {"", "", 0, {{NULL, NULL, NULL, NULL}}, &uam_changepw,
-                                         &uam_changepw};
+                                      &uam_changepw};
 
 static struct uam_obj *afp_uam = NULL;
 
 
 void status_versions( data )
-char   *data;
+    char    *data;
 {
     char                *start = data;
     u_int16_t           status;
-    int                        len, num, i;
+    int         len, num, i;
 
     memcpy(&status, start + AFPSTATUS_VERSOFF, sizeof(status));
     num = sizeof( afp_versions ) / sizeof( afp_versions[ 0 ] );
@@ -122,7 +122,7 @@ void status_uams(char *data, const char *authlist)
     char                *start = data;
     u_int16_t           status;
     struct uam_obj      *uams;
-    int                        len, num = 0;
+    int         len, num = 0;
 
     memcpy(&status, start + AFPSTATUS_UAMSOFF, sizeof(status));
     uams = &uam_login;
@@ -162,18 +162,18 @@ static int send_reply(const AFPObj *obj, const int err)
 }
 
 static int afp_errpwdexpired(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj  *obj _U_;
-char   *ibuf _U_, *rbuf _U_;
-int    ibuflen _U_, *rbuflen;
+    AFPObj  *obj _U_;
+    char    *ibuf _U_, *rbuf _U_;
+    int ibuflen _U_, *rbuflen;
 {
     *rbuflen = 0;
     return AFPERR_PWDEXPR;
 }
 
 static int afp_null_nolog(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj  *obj _U_;
-char   *ibuf _U_, *rbuf _U_;
-int    ibuflen _U_, *rbuflen;
+    AFPObj  *obj _U_;
+    char    *ibuf _U_, *rbuf _U_;
+    int ibuflen _U_, *rbuflen;
 {
     *rbuflen = 0;
     return( AFPERR_NOOP );
@@ -184,58 +184,58 @@ static int set_auth_switch(int expired)
     int i;
 
     if (expired) {
-       /* 
-        * BF: expired password handling
-        * to allow the user to change his/her password we have to allow login
-        * but every following call except for FPChangePassword will be thrown
-        * away with an AFPERR_PWDEXPR error. (thanks to Leland Wallace from Apple
-        * for clarifying this)
+        /*
+         * BF: expired password handling
+         * to allow the user to change his/her password we have to allow login
+         * but every following call except for FPChangePassword will be thrown
+         * away with an AFPERR_PWDEXPR error. (thanks to Leland Wallace from Apple
+         * for clarifying this)
          */
 
-       for (i=0; i<=0xff; i++) {
-           uam_afpserver_action(i, UAM_AFPSERVER_PREAUTH, afp_errpwdexpired, NULL); 
-       }
-        uam_afpserver_action(AFP_LOGOUT, UAM_AFPSERVER_PREAUTH, afp_logout, NULL); 
-       uam_afpserver_action(AFP_CHANGEPW, UAM_AFPSERVER_PREAUTH, afp_changepw, NULL);
+        for (i=0; i<=0xff; i++) {
+            uam_afpserver_action(i, UAM_AFPSERVER_PREAUTH, afp_errpwdexpired, NULL);
+        }
+        uam_afpserver_action(AFP_LOGOUT, UAM_AFPSERVER_PREAUTH, afp_logout, NULL);
+        uam_afpserver_action(AFP_CHANGEPW, UAM_AFPSERVER_PREAUTH, afp_changepw, NULL);
     }
     else {
         afp_switch = postauth_switch;
         switch (afp_version) {
         case 32:
-           uam_afpserver_action(AFP_SPOTLIGHT_PRIVATE, UAM_AFPSERVER_POSTAUTH, afp_null_nolog, NULL);
-           uam_afpserver_action(AFP_SYNCDIR, UAM_AFPSERVER_POSTAUTH, afp_syncdir, NULL);
-           uam_afpserver_action(AFP_SYNCFORK, UAM_AFPSERVER_POSTAUTH, afp_syncfork, NULL);
+            uam_afpserver_action(AFP_SPOTLIGHT_PRIVATE, UAM_AFPSERVER_POSTAUTH, afp_null_nolog, NULL);
+            uam_afpserver_action(AFP_SYNCDIR, UAM_AFPSERVER_POSTAUTH, afp_syncdir, NULL);
+            uam_afpserver_action(AFP_SYNCFORK, UAM_AFPSERVER_POSTAUTH, afp_syncfork, NULL);
 #ifdef HAVE_NFSv4_ACLS
-           uam_afpserver_action(AFP_GETACL, UAM_AFPSERVER_POSTAUTH, afp_getacl, NULL);
-           uam_afpserver_action(AFP_SETACL, UAM_AFPSERVER_POSTAUTH, afp_setacl, NULL);
-           uam_afpserver_action(AFP_ACCESS, UAM_AFPSERVER_POSTAUTH, afp_access, NULL);
+            uam_afpserver_action(AFP_GETACL, UAM_AFPSERVER_POSTAUTH, afp_getacl, NULL);
+            uam_afpserver_action(AFP_SETACL, UAM_AFPSERVER_POSTAUTH, afp_setacl, NULL);
+            uam_afpserver_action(AFP_ACCESS, UAM_AFPSERVER_POSTAUTH, afp_access, NULL);
 #endif
 #ifdef HAVE_EXT_ATTRS
-           uam_afpserver_action(AFP_GETEXTATTR, UAM_AFPSERVER_POSTAUTH, afp_getextattr, NULL);
-           uam_afpserver_action(AFP_SETEXTATTR, UAM_AFPSERVER_POSTAUTH, afp_setextattr, NULL);
-           uam_afpserver_action(AFP_REMOVEATTR, UAM_AFPSERVER_POSTAUTH, afp_remextattr, NULL);
-           uam_afpserver_action(AFP_LISTEXTATTR, UAM_AFPSERVER_POSTAUTH, afp_listextattr, NULL);
+            uam_afpserver_action(AFP_GETEXTATTR, UAM_AFPSERVER_POSTAUTH, afp_getextattr, NULL);
+            uam_afpserver_action(AFP_SETEXTATTR, UAM_AFPSERVER_POSTAUTH, afp_setextattr, NULL);
+            uam_afpserver_action(AFP_REMOVEATTR, UAM_AFPSERVER_POSTAUTH, afp_remextattr, NULL);
+            uam_afpserver_action(AFP_LISTEXTATTR, UAM_AFPSERVER_POSTAUTH, afp_listextattr, NULL);
 #endif
         case 31:
-           uam_afpserver_action(AFP_ENUMERATE_EXT2, UAM_AFPSERVER_POSTAUTH, afp_enumerate_ext2, NULL);
+            uam_afpserver_action(AFP_ENUMERATE_EXT2, UAM_AFPSERVER_POSTAUTH, afp_enumerate_ext2, NULL);
         case 30:
-           uam_afpserver_action(AFP_ENUMERATE_EXT, UAM_AFPSERVER_POSTAUTH, afp_enumerate_ext, NULL); 
-           uam_afpserver_action(AFP_BYTELOCK_EXT,  UAM_AFPSERVER_POSTAUTH, afp_bytelock_ext, NULL); 
+            uam_afpserver_action(AFP_ENUMERATE_EXT, UAM_AFPSERVER_POSTAUTH, afp_enumerate_ext, NULL);
+            uam_afpserver_action(AFP_BYTELOCK_EXT,  UAM_AFPSERVER_POSTAUTH, afp_bytelock_ext, NULL);
             /* catsearch_ext uses the same packet as catsearch FIXME double check this, it wasn't true for enue
                enumerate_ext */
-           uam_afpserver_action(AFP_CATSEARCH_EXT, UAM_AFPSERVER_POSTAUTH, afp_catsearch_ext, NULL); 
-           uam_afpserver_action(AFP_GETSESSTOKEN,  UAM_AFPSERVER_POSTAUTH, afp_getsession, NULL); 
-           uam_afpserver_action(AFP_READ_EXT,      UAM_AFPSERVER_POSTAUTH, afp_read_ext, NULL); 
-           uam_afpserver_action(AFP_WRITE_EXT,     UAM_AFPSERVER_POSTAUTH, afp_write_ext, NULL); 
-           uam_afpserver_action(AFP_DISCTOLDSESS,  UAM_AFPSERVER_POSTAUTH, afp_disconnect, NULL); 
+            uam_afpserver_action(AFP_CATSEARCH_EXT, UAM_AFPSERVER_POSTAUTH, afp_catsearch_ext, NULL);
+            uam_afpserver_action(AFP_GETSESSTOKEN,  UAM_AFPSERVER_POSTAUTH, afp_getsession, NULL);
+            uam_afpserver_action(AFP_READ_EXT,      UAM_AFPSERVER_POSTAUTH, afp_read_ext, NULL);
+            uam_afpserver_action(AFP_WRITE_EXT,     UAM_AFPSERVER_POSTAUTH, afp_write_ext, NULL);
+            uam_afpserver_action(AFP_DISCTOLDSESS,  UAM_AFPSERVER_POSTAUTH, afp_disconnect, NULL);
 
         case 22:
             /*
              * If first connection to a server is done in classic AFP2.2 version is used
              * but OSX uses AFP3.x FPzzz command !
-            */
-           uam_afpserver_action(AFP_ZZZ,  UAM_AFPSERVER_POSTAUTH, afp_zzz, NULL); 
-           break;
+             */
+            uam_afpserver_action(AFP_ZZZ,  UAM_AFPSERVER_POSTAUTH, afp_zzz, NULL);
+            break;
         }
     }
 
@@ -252,7 +252,7 @@ static int login(AFPObj *obj, struct passwd *pwd, void (*logout)(void), int expi
     set_processname("afpd");
 #endif
 
-    if ( pwd->pw_uid == 0 ) {  /* don't allow root login */
+    if ( pwd->pw_uid == 0 ) {   /* don't allow root login */
         LOG(log_error, logtype_afpd, "login: root login denied!" );
         return AFPERR_NOTAUTH;
     }
@@ -270,7 +270,7 @@ static int login(AFPObj *obj, struct passwd *pwd, void (*logout)(void), int expi
             if(addr_net && addr_node) { /* Do we have a valid Appletalk address? */
                 char nodename[256];
                 FILE *fp;
-               int mypid = getpid();
+                int mypid = getpid();
                 struct stat stat_buf;
 
                 sprintf(nodename, "%s/net%d.%dnode%d", obj->options.authprintdir,
@@ -315,7 +315,7 @@ static int login(AFPObj *obj, struct passwd *pwd, void (*logout)(void), int expi
         LOG(log_error, logtype_afpd, "login: %s getgroups: %s", pwd->pw_name, strerror(errno) );
         return AFPERR_BADUAM;
     }
-    
+
     if ( NULL == (groups = calloc(ngroups, GROUPS_SIZE)) ) {
         LOG(log_error, logtype_afpd, "login: %s calloc: %d", ngroups);
         return AFPERR_BADUAM;
@@ -385,7 +385,7 @@ static int login(AFPObj *obj, struct passwd *pwd, void (*logout)(void), int expi
 #endif /* TRU64 */
 
     /* There's probably a better way to do this, but for now, we just
-    play root */
+       play root */
 
 #ifdef ADMIN_GRP
     if (admin) uuid = 0;
@@ -400,24 +400,24 @@ static int login(AFPObj *obj, struct passwd *pwd, void (*logout)(void), int expi
 #ifdef FORCE_UIDGID
     obj->force_uid = 1;
     save_uidgid ( &obj->uidgid );
-#endif 
-       
+#endif
+
     return( AFP_OK );
 }
 
 /* ---------------------- */
 int afp_zzz (obj, ibuf, ibuflen, rbuf, rbuflen ) /* Function 122 */
-AFPObj       *obj;
-char         *ibuf  _U_, *rbuf;
-unsigned int ibuflen  _U_, *rbuflen;
+    AFPObj       *obj;
+    char         *ibuf  _U_, *rbuf;
+    unsigned int ibuflen  _U_, *rbuflen;
 {
-    u_int32_t  retdata;
+    u_int32_t   retdata;
 
     *rbuflen = 0;
 
     retdata = obj->options.sleep /120;
     if (!retdata) {
-       retdata = 1;
+        retdata = 1;
     }
     *rbuflen = sizeof(retdata);
     retdata = htonl(retdata);
@@ -435,12 +435,12 @@ static int create_session_token(AFPObj *obj)
 
     /* use 8 bytes for token as OSX, don't know if it helps */
     if ( sizeof(pid_t) > SESSIONTOKEN_LEN) {
-       LOG(log_error, logtype_afpd, "sizeof(pid_t) > %u", SESSIONTOKEN_LEN );
-       return AFPERR_MISC;
+        LOG(log_error, logtype_afpd, "sizeof(pid_t) > %u", SESSIONTOKEN_LEN );
+        return AFPERR_MISC;
     }
 
     if ( NULL == (obj->sinfo.sessiontoken = malloc(SESSIONTOKEN_LEN)) )
-       return AFPERR_MISC;
+        return AFPERR_MISC;
 
     memset(obj->sinfo.sessiontoken, 0, SESSIONTOKEN_LEN);
     obj->sinfo.sessiontoken_len = SESSIONTOKEN_LEN;
@@ -455,25 +455,25 @@ static int create_session_key(AFPObj *obj)
     /* create session key */
     if (obj->sinfo.sessionkey == NULL) {
         if (NULL == (obj->sinfo.sessionkey = malloc(SESSIONKEY_LEN)) )
-            return AFPERR_MISC;   
+            return AFPERR_MISC;
         uam_random_string(obj, obj->sinfo.sessionkey, SESSIONKEY_LEN);
         obj->sinfo.sessionkey_len = SESSIONKEY_LEN;
     }
     return AFP_OK;
 }
 
-   
+
 /* ---------------------- */
 int afp_getsession(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj       *obj;
-char        *ibuf, *rbuf;
-unsigned int ibuflen, *rbuflen;
+    AFPObj       *obj;
+    char         *ibuf, *rbuf;
+    unsigned int ibuflen, *rbuflen;
 {
     u_int16_t           type;
     u_int32_t           idlen = 0;
-    u_int32_t          boottime;
+    u_int32_t       boottime;
     u_int32_t           tklen, tp;
-    char                *token; 
+    char                *token;
     char                *p;
 
     *rbuflen = 0;
@@ -493,12 +493,12 @@ unsigned int ibuflen, *rbuflen;
     }
 
     /*
-     * 
-    */
+     *
+     */
     switch (type) {
     case 0: /* old version ?*/
-            tklen = obj->sinfo.sessiontoken_len;
-            token = obj->sinfo.sessiontoken;
+        tklen = obj->sinfo.sessiontoken_len;
+        token = obj->sinfo.sessiontoken;
         break;
     case 1: /* disconnect */
     case 2: /* reconnect update id */
@@ -517,28 +517,28 @@ unsigned int ibuflen, *rbuflen;
         break;
     case 3: /* Jaguar */
     case 4:
-       if (ibuflen >= 8 ) {
-           p = ibuf;
-           memcpy( &idlen, ibuf, sizeof(idlen));
-           idlen = ntohl(idlen);
-           ibuf += sizeof(idlen);
-           ibuflen -= sizeof(idlen);
-           ibuf += sizeof(boottime);
-           ibuflen -= sizeof(boottime);
-           if (ibuflen < idlen || idlen > (90-10)) {
-               return AFPERR_PARAM;
-           }
-           server_ipc_write(IPC_GETSESSION, idlen+8, p );
-           tklen = obj->sinfo.sessiontoken_len;
-           token = obj->sinfo.sessiontoken;
-       }
-       break;
-     case 8: /* Panther Kerberos Token */
-            tklen = obj->sinfo.cryptedkey_len;
-            token = obj->sinfo.cryptedkey;
+        if (ibuflen >= 8 ) {
+            p = ibuf;
+            memcpy( &idlen, ibuf, sizeof(idlen));
+            idlen = ntohl(idlen);
+            ibuf += sizeof(idlen);
+            ibuflen -= sizeof(idlen);
+            ibuf += sizeof(boottime);
+            ibuflen -= sizeof(boottime);
+            if (ibuflen < idlen || idlen > (90-10)) {
+                return AFPERR_PARAM;
+            }
+            server_ipc_write(IPC_GETSESSION, idlen+8, p );
+            tklen = obj->sinfo.sessiontoken_len;
+            token = obj->sinfo.sessiontoken;
+        }
         break;
-     default:
-            return AFPERR_NOOP;
+    case 8: /* Panther Kerberos Token */
+        tklen = obj->sinfo.cryptedkey_len;
+        token = obj->sinfo.cryptedkey;
+        break;
+    default:
+        return AFPERR_NOOP;
         break;
 
     }
@@ -559,9 +559,9 @@ unsigned int ibuflen, *rbuflen;
 
 /* ---------------------- */
 int afp_disconnect(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj  *obj  _U_;
-char   *ibuf, *rbuf  _U_;
-int    ibuflen  _U_, *rbuflen;
+    AFPObj  *obj  _U_;
+    char    *ibuf, *rbuf  _U_;
+    int ibuflen  _U_, *rbuflen;
 {
     u_int16_t           type;
 
@@ -588,36 +588,36 @@ int       ibuflen  _U_, *rbuflen;
     ibuf += sizeof(tklen);
 
     if ( sizeof(pid_t) > SESSIONTOKEN_LEN) {
-         LOG(log_error, logtype_afpd, "sizeof(pid_t) > %u", SESSIONTOKEN_LEN );
-         return AFPERR_MISC;
+        LOG(log_error, logtype_afpd, "sizeof(pid_t) > %u", SESSIONTOKEN_LEN );
+        return AFPERR_MISC;
     }
     if (tklen != SESSIONTOKEN_LEN) {
         return AFPERR_MISC;
-    }   
+    }
     tklen = sizeof(pid_t);
     memcpy(&token, ibuf, tklen);
 
     /* our stuff is pid + zero pad */
     ibuf += tklen;
     for (i = tklen; i < SESSIONTOKEN_LEN; i++, ibuf++) {
-         if (*ibuf != 0) {
-             return AFPERR_MISC;
-         }
+        if (*ibuf != 0) {
+            return AFPERR_MISC;
+        }
     }
-    
+
     /* killed old session, not easy */
     server_ipc_write(IPC_KILLTOKEN, tklen, &token);
     sleep(1);
-    
+
     return AFPERR_SESSCLOS;   /* was AFP_OK */
 }
 
 /* ---------------------- */
 static int get_version(obj, ibuf, ibuflen, len)
-AFPObj  *obj;
-char    *ibuf;
-int    ibuflen;
-int    len;
+    AFPObj  *obj;
+    char    *ibuf;
+    int     ibuflen;
+    int     len;
 {
     int num,i;
 
@@ -632,7 +632,7 @@ int         len;
             break;
         }
     }
-    if ( i == num )                            /* An inappropo version */
+    if ( i == num )                 /* An inappropo version */
         return AFPERR_BADVERS ;
 
     if (afp_version >= 30 && obj->proto != AFPPROTO_DSI)
@@ -649,12 +649,12 @@ int       len;
 
 /* ---------------------- */
 int afp_login(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
-char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+    AFPObj      *obj;
+    char    *ibuf, *rbuf;
+    int     ibuflen, *rbuflen;
 {
     struct passwd *pwd = NULL;
-    int                len, i;
+    int     len, i;
 
     *rbuflen = 0;
 
@@ -669,7 +669,7 @@ int         ibuflen, *rbuflen;
     ibuflen -= 2;
 
     i = get_version(obj, ibuf, ibuflen, len);
-    if (i) 
+    if (i)
         return send_reply(obj, i );
 
     ibuf += len;
@@ -702,17 +702,17 @@ int               ibuflen, *rbuflen;
 
 /* ---------------------- */
 int afp_login_ext(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj  *obj;
-char   *ibuf, *rbuf;
-unsigned int   ibuflen, *rbuflen;
+    AFPObj  *obj;
+    char    *ibuf, *rbuf;
+    unsigned int    ibuflen, *rbuflen;
 {
     struct passwd *pwd = NULL;
     unsigned int  len;
-    int                i;
+    int     i;
     char        type;
     u_int16_t   len16;
     char        *username;
-    
+
     *rbuflen = 0;
 
     if ( nologin & 1)
@@ -721,16 +721,16 @@ unsigned int      ibuflen, *rbuflen;
     if (ibuflen <= 4)
         return send_reply(obj, AFPERR_BADVERS );
 
-    ibuf++; 
+    ibuf++;
     ibuf++;     /* pad  */
     ibuf +=2;   /* flag */
 
     len = (unsigned char) *ibuf;
     ibuf++;
     ibuflen -= 5;
-    
+
     i = get_version(obj, ibuf, ibuflen, len);
-    if (i) 
+    if (i)
         return send_reply(obj, i );
 
     ibuf    += len;
@@ -756,31 +756,31 @@ unsigned int      ibuflen, *rbuflen;
         return send_reply(obj, AFPERR_BADUAM);
     }
     /* user name */
-    if (len <= 1 +sizeof(len16)) 
+    if (len <= 1 +sizeof(len16))
         return send_reply(obj, AFPERR_PARAM);
     type = *ibuf;
     username = ibuf;
     ibuf++;
     ibuflen--;
-    if (type != 3) 
+    if (type != 3)
         return send_reply(obj, AFPERR_PARAM);
 
     memcpy(&len16, ibuf, sizeof(len16));
     ibuf += sizeof(len16);
     ibuflen -= sizeof(len16);
     len = ntohs(len16);
-    if (len > ibuflen) 
+    if (len > ibuflen)
         return send_reply(obj, AFPERR_PARAM);
     ibuf += len;
-    ibuflen -= len;    
+    ibuflen -= len;
 
     /* directory service name */
-    if (!ibuflen) 
+    if (!ibuflen)
         return send_reply(obj, AFPERR_PARAM);
     type = *ibuf;
     ibuf++;
     ibuflen--;
-    
+
     switch(type) {
     case 1:
     case 2:
@@ -792,7 +792,7 @@ unsigned int        ibuflen, *rbuflen;
         break;
     case 3:
         /* With "No User Authen" it is equal */
-        if (ibuflen < sizeof(len16)) 
+        if (ibuflen < sizeof(len16))
             return send_reply(obj, AFPERR_PARAM);
         memcpy(&len16, ibuf, sizeof(len16));
         ibuf += sizeof(len16);
@@ -801,7 +801,7 @@ unsigned int        ibuflen, *rbuflen;
         break;
     default:
         return send_reply(obj, AFPERR_PARAM);
-    }    
+    }
 #if 0
     if (len != 0) {
         LOG(log_error, logtype_afpd, "login_ext: directory service path not null!" );
@@ -810,7 +810,7 @@ unsigned int        ibuflen, *rbuflen;
 #endif
     ibuf += len;
     ibuflen -= len;
-    
+
     /* Pad */
     if (ibuflen && ((unsigned long) ibuf & 1)) { /* pad character */
         ibuf++;
@@ -821,7 +821,7 @@ unsigned int        ibuflen, *rbuflen;
         return send_reply(obj, i);
     }
 
-    /* FIXME user name are in UTF8 */    
+    /* FIXME user name are in UTF8 */
     i = afp_uam->u.uam_login.login_ext(obj, username, &pwd, ibuf, ibuflen, rbuf, rbuflen);
 
     if (!pwd || ( i != AFP_OK && i != AFPERR_PWDEXPR))
@@ -832,9 +832,9 @@ unsigned int        ibuflen, *rbuflen;
 
 /* ---------------------- */
 int afp_logincont(obj, ibuf, ibuflen, rbuf, rbuflen)
-AFPObj      *obj;
-char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+    AFPObj      *obj;
+    char    *ibuf, *rbuf;
+    int     ibuflen, *rbuflen;
 {
     struct passwd *pwd = NULL;
     int err;
@@ -855,9 +855,9 @@ int         ibuflen, *rbuflen;
 
 
 int afp_logout(obj, ibuf, ibuflen, rbuf, rbuflen)
-AFPObj     *obj;
-char       *ibuf _U_, *rbuf  _U_;
-int        ibuflen  _U_, *rbuflen  _U_;
+    AFPObj     *obj;
+    char       *ibuf _U_, *rbuf  _U_;
+    int        ibuflen  _U_, *rbuflen  _U_;
 {
     LOG(log_info, logtype_afpd, "logout %s", obj->username);
     close_all_vol();
@@ -870,12 +870,12 @@ int        ibuflen  _U_, *rbuflen  _U_;
 /* change password  --
  * NOTE: an FPLogin must already have completed successfully for this
  *       to work. this also does a little pre-processing before it hands
- *       it off to the uam. 
+ *       it off to the uam.
  */
 int afp_changepw(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj      *obj;
-char   *ibuf, *rbuf;
-int            ibuflen, *rbuflen;
+    AFPObj      *obj;
+    char    *ibuf, *rbuf;
+    int     ibuflen, *rbuflen;
 {
     char username[MACFILELEN + 1], *start = ibuf;
     struct uam_obj *uam;
@@ -888,9 +888,9 @@ int         ibuflen, *rbuflen;
 
     /* check if password change is allowed, OS-X ignores the flag.
      * we shouldn't trust the client on this anyway.
-     * not sure about the "right" error code, NOOP for now */ 
+     * not sure about the "right" error code, NOOP for now */
     if (!(obj->options.passwdbits & PASSWD_SET))
-         return AFPERR_NOOP;
+        return AFPERR_NOOP;
 
     /* make sure we can deal w/ this uam */
     len = (unsigned char) *ibuf++;
@@ -912,15 +912,15 @@ int               ibuflen, *rbuflen;
         if ((len + 1) & 1) /* pad byte */
             ibuf++;
     } else {
-       /* AFP > 3.0 doesn't pass the username, APF 3.1 specs page 124 */
-       if ( ibuf[0] != '\0' || ibuf[1] != '\0')
-           return AFPERR_PARAM;
+        /* AFP > 3.0 doesn't pass the username, APF 3.1 specs page 124 */
+        if ( ibuf[0] != '\0' || ibuf[1] != '\0')
+            return AFPERR_PARAM;
         ibuf += 2;
-       len = MIN(sizeof(username), strlen(obj->username));
+        len = MIN(sizeof(username), strlen(obj->username));
         memcpy(username, obj->username, len);
-       username[ len ] = '\0';
+        username[ len ] = '\0';
     }
-        
+
 
     LOG(log_info, logtype_afpd, "changing password for <%s>", username);
 
@@ -935,17 +935,17 @@ int               ibuflen, *rbuflen;
         (ret == AFPERR_AUTHCONT) ? "continued" :
         (ret ? "failed" : "succeeded"));
     if ( ret == AFP_OK )
-       set_auth_switch(0);
-       
+        set_auth_switch(0);
+
     return ret;
 }
 
 
 /* FPGetUserInfo */
 int afp_getuserinfo(obj, ibuf, ibuflen, rbuf, rbuflen )
-AFPObj  *obj _U_;
-char   *ibuf, *rbuf;
-int    ibuflen _U_, *rbuflen;
+    AFPObj  *obj _U_;
+    char    *ibuf, *rbuf;
+    int ibuflen _U_, *rbuflen;
 {
     u_int8_t  thisuser;
     u_int32_t id;
@@ -993,7 +993,7 @@ int ibuflen _U_, *rbuflen;
         int ret;
         uuid_t uuid;
         char *uuidstring;
-        
+
         if ( ! (obj->options.flags & OPTION_UUID))
             return AFPERR_BITMAP;
         LOG(log_debug, logtype_afpd, "afp_getuserinfo: get UUID for \'%s\'", obj->username);
@@ -1017,13 +1017,13 @@ int     ibuflen _U_, *rbuflen;
 }
 
 #define UAM_LIST(type) (((type) == UAM_SERVER_LOGIN || (type) == UAM_SERVER_LOGIN_EXT) ? &uam_login : \
-                       (((type) == UAM_SERVER_CHANGEPW) ? \
-                        &uam_changepw : NULL))
+                        (((type) == UAM_SERVER_CHANGEPW) ?              \
+                         &uam_changepw : NULL))
 
 /* just do a linked list search. this could be sped up with a hashed
  * list, but i doubt anyone's going to have enough uams to matter. */
 struct uam_obj *auth_uamfind(const int type, const char *name,
-                                         const int len)
+                             const int len)
 {
     struct uam_obj *prev, *start;
 
@@ -1077,7 +1077,7 @@ int auth_load(const char *path, const char *list)
         strlcpy(name + len, p, sizeof(name) - len);
         LOG(log_debug, logtype_afpd, "uam: loading (%s)", name);
         /*
-        if ((stat(name, &st) == 0) && (mod = uam_load(name, p))) {
+          if ((stat(name, &st) == 0) && (mod = uam_load(name, p))) {
         */
         if (stat(name, &st) == 0) {
             if ((mod = uam_load(name, p))) {