Example code: Don't create rs_error on failing context creation.
[radsecproxy.git] / radsecproxy.c
index 178d76d..2276d40 100644 (file)
@@ -9,6 +9,9 @@
 /* Code contributions from:
  *
  * Arne Schwabe <schwabe at uni-paderborn.de>
+ * Maja Wolniewicz <mgw@umk.pl>
+ * Simon Leinen <simon.leinen@switch.ch>
+ * Stefan Winter <stefan.winter@restena.lu>
  */
 
 /* For UDP there is one server instance consisting of udpserverrd and udpserverth
@@ -32,7 +35,7 @@
  *       This is only for connected peers
  * Example: With 3 UDP peers and 30 TLS peers, there will be a max of
  *          1 + (2 + 2 * 3) + (2 * 30) + (2 * 30) = 129 threads
-*/
+ */
 
 /* Bugs:
  * May segfault when dtls connections go down? More testing needed
@@ -59,6 +62,7 @@
 #include <regex.h>
 #include <libgen.h>
 #include <pthread.h>
+#include <errno.h>
 #include <openssl/ssl.h>
 #include <openssl/rand.h>
 #include <openssl/err.h>
@@ -105,10 +109,10 @@ uint8_t protoname2int(const char *name) {
            return i;
     return 255;
 }
-    
+
 /* callbacks for making OpenSSL thread safe */
 unsigned long ssl_thread_id() {
-        return (unsigned long)pthread_self();
+    return (unsigned long)pthread_self();
 }
 
 void ssl_locking_callback(int mode, int type, const char *file, int line) {
@@ -135,7 +139,7 @@ int prefixmatch(void *a1, void *a2, uint8_t len) {
 struct clsrvconf *find_conf(uint8_t type, struct sockaddr *addr, struct list *confs, struct list_node **cur, uint8_t server_p) {
     struct list_node *entry;
     struct clsrvconf *conf;
-    
+
     for (entry = (cur && *cur ? list_next(*cur) : list_first(confs)); entry; entry = list_next(entry)) {
        conf = (struct clsrvconf *)entry->data;
        if (conf->type == type && addressmatches(conf->hostports, addr, server_p)) {
@@ -143,7 +147,7 @@ struct clsrvconf *find_conf(uint8_t type, struct sockaddr *addr, struct list *co
                *cur = entry;
            return conf;
        }
-    }    
+    }
     return NULL;
 }
 
@@ -159,7 +163,7 @@ struct clsrvconf *find_srvconf(uint8_t type, struct sockaddr *addr, struct list_
 struct clsrvconf *find_clconf_type(uint8_t type, struct list_node **cur) {
     struct list_node *entry;
     struct clsrvconf *conf;
-    
+
     for (entry = (cur && *cur ? list_next(*cur) : list_first(clconfs)); entry; entry = list_next(entry)) {
        conf = (struct clsrvconf *)entry->data;
        if (conf->type == type) {
@@ -167,13 +171,13 @@ struct clsrvconf *find_clconf_type(uint8_t type, struct list_node **cur) {
                *cur = entry;
            return conf;
        }
-    }    
+    }
     return NULL;
 }
 
 struct gqueue *newqueue() {
     struct gqueue *q;
-    
+
     q = malloc(sizeof(struct gqueue));
     if (!q)
        debugx(1, DBG_ERR, "malloc failed");
@@ -202,7 +206,7 @@ void removequeue(struct gqueue *q) {
 
 void freebios(struct gqueue *q) {
     BIO *bio;
-    
+
     pthread_mutex_lock(&q->mutex);
     while ((bio = (BIO *)list_shift(q->entries)))
        BIO_free(bio);
@@ -212,7 +216,7 @@ void freebios(struct gqueue *q) {
 
 struct client *addclient(struct clsrvconf *conf, uint8_t lock) {
     struct client *new = malloc(sizeof(struct client));
-    
+
     if (!new) {
        debug(DBG_ERR, "malloc failed");
        return NULL;
@@ -229,7 +233,7 @@ struct client *addclient(struct clsrvconf *conf, uint8_t lock) {
            return NULL;
        }
     }
-    
+
     memset(new, 0, sizeof(struct client));
     new->conf = conf;
     if (conf->pdef->addclient)
@@ -250,7 +254,7 @@ void removeclientrqs_sendrq_freeserver_lock(uint8_t wantlock) {
     else
        pthread_mutex_unlock(&lock);
 }
-    
+
 void removeclientrqs(struct client *client) {
     struct request *rq;
     struct rqout *rqout;
@@ -275,7 +279,7 @@ void removeclientrqs(struct client *client) {
 
 void removelockedclient(struct client *client) {
     struct clsrvconf *conf;
-    
+
     conf = client->conf;
     if (conf->clients) {
        removeclientrqs(client);
@@ -288,7 +292,7 @@ void removelockedclient(struct client *client) {
 
 void removeclient(struct client *client) {
     struct clsrvconf *conf;
-    
+
     if (!client)
        return;
 
@@ -332,7 +336,7 @@ void freeserver(struct server *server, uint8_t destroymutex) {
 
 int addserver(struct clsrvconf *conf) {
     int i;
-    
+
     if (conf->servers) {
        debug(DBG_ERR, "addserver: currently works with just one server per conf");
        return 0;
@@ -345,7 +349,7 @@ int addserver(struct clsrvconf *conf) {
     memset(conf->servers, 0, sizeof(struct server));
     conf->servers->conf = conf;
 
-#ifdef RADPROT_DTLS    
+#ifdef RADPROT_DTLS
     if (conf->type == RAD_DTLS)
        conf->servers->rbios = newqueue();
 #endif
@@ -354,7 +358,7 @@ int addserver(struct clsrvconf *conf) {
     conf->servers->sock = -1;
     if (conf->pdef->addserverextra)
        conf->pdef->addserverextra(conf);
-    
+
     conf->servers->requests = calloc(MAX_REQUESTS, sizeof(struct rqout));
     if (!conf->servers->requests) {
        debug(DBG_ERR, "malloc failed");
@@ -367,32 +371,32 @@ int addserver(struct clsrvconf *conf) {
            goto errexit;
        }
        if (pthread_mutex_init(conf->servers->requests[i].lock, NULL)) {
-           debug(DBG_ERR, "mutex init failed");
+           debugerrno(errno, DBG_ERR, "mutex init failed");
            free(conf->servers->requests[i].lock);
            conf->servers->requests[i].lock = NULL;
            goto errexit;
        }
     }
     if (pthread_mutex_init(&conf->servers->lock, NULL)) {
-       debug(DBG_ERR, "mutex init failed");
+       debugerrno(errno, DBG_ERR, "mutex init failed");
        goto errexit;
     }
     conf->servers->newrq = 0;
     if (pthread_mutex_init(&conf->servers->newrq_mutex, NULL)) {
-       debug(DBG_ERR, "mutex init failed");
+       debugerrno(errno, DBG_ERR, "mutex init failed");
        pthread_mutex_destroy(&conf->servers->lock);
        goto errexit;
     }
     if (pthread_cond_init(&conf->servers->newrq_cond, NULL)) {
-       debug(DBG_ERR, "mutex init failed");
+       debugerrno(errno, DBG_ERR, "mutex init failed");
        pthread_mutex_destroy(&conf->servers->newrq_mutex);
        pthread_mutex_destroy(&conf->servers->lock);
        goto errexit;
     }
 
     return 1;
-    
- errexit:
+
+errexit:
     freeserver(conf->servers, 0);
     conf->servers = NULL;
     return 0;
@@ -454,7 +458,7 @@ void sendrq(struct request *rq) {
     to = rq->to;
     if (!to)
        goto errexit;
-    
+
     start = to->conf->statusserver ? 1 : 0;
     pthread_mutex_lock(&to->newrq_mutex);
     if (start && rq->msg->code == RAD_Status_Server) {
@@ -500,7 +504,7 @@ void sendrq(struct request *rq) {
        debug(DBG_ERR, "sendrq: radmsg2buf failed");
        goto errexit;
     }
-    
+
     debug(DBG_DBG, "sendrq: inserting packet with id %d in queue for %s", i, to->conf->name);
     to->requests[i].rq = rq;
     pthread_mutex_unlock(to->requests[i].lock);
@@ -517,7 +521,7 @@ void sendrq(struct request *rq) {
     removeclientrqs_sendrq_freeserver_lock(0);
     return;
 
- errexit:
+errexit:
     if (rq->from)
        rmclientrq(rq, rq->msg->id);
     freerq(rq);
@@ -528,7 +532,7 @@ void sendrq(struct request *rq) {
 void sendreply(struct request *rq) {
     uint8_t first;
     struct client *to = rq->from;
-    
+
     if (!rq->replybuf)
        rq->replybuf = radmsg2buf(rq->msg, (uint8_t *)to->conf->secret);
     radmsg_free(rq->msg);
@@ -541,14 +545,14 @@ void sendreply(struct request *rq) {
 
     pthread_mutex_lock(&to->replyq->mutex);
     first = list_first(to->replyq->entries) == NULL;
-    
+
     if (!list_push(to->replyq->entries, rq)) {
        pthread_mutex_unlock(&to->replyq->mutex);
        freerq(rq);
        debug(DBG_ERR, "sendreply: malloc failed");
        return;
     }
-    
+
     if (first) {
        debug(DBG_DBG, "signalling server writer");
        pthread_cond_signal(&to->replyq->cond);
@@ -556,50 +560,14 @@ void sendreply(struct request *rq) {
     pthread_mutex_unlock(&to->replyq->mutex);
 }
 
-int pwdencrypt(uint8_t *in, uint8_t len, char *shared, uint8_t sharedlen, uint8_t *auth) {
+int pwdcrypt(char encrypt_flag, uint8_t *in, uint8_t len, char *shared, uint8_t sharedlen, uint8_t *auth) {
     static pthread_mutex_t lock = PTHREAD_MUTEX_INITIALIZER;
     static unsigned char first = 1;
     static EVP_MD_CTX mdctx;
     unsigned char hash[EVP_MAX_MD_SIZE], *input;
     unsigned int md_len;
     uint8_t i, offset = 0, out[128];
-    
-    pthread_mutex_lock(&lock);
-    if (first) {
-       EVP_MD_CTX_init(&mdctx);
-       first = 0;
-    }
 
-    input = auth;
-    for (;;) {
-       if (!EVP_DigestInit_ex(&mdctx, EVP_md5(), NULL) ||
-           !EVP_DigestUpdate(&mdctx, (uint8_t *)shared, sharedlen) ||
-           !EVP_DigestUpdate(&mdctx, input, 16) ||
-           !EVP_DigestFinal_ex(&mdctx, hash, &md_len) ||
-           md_len != 16) {
-           pthread_mutex_unlock(&lock);
-           return 0;
-       }
-       for (i = 0; i < 16; i++)
-           out[offset + i] = hash[i] ^ in[offset + i];
-       input = out + offset - 16;
-       offset += 16;
-       if (offset == len)
-           break;
-    }
-    memcpy(in, out, len);
-    pthread_mutex_unlock(&lock);
-    return 1;
-}
-
-int pwddecrypt(uint8_t *in, uint8_t len, char *shared, uint8_t sharedlen, uint8_t *auth) {
-    static pthread_mutex_t lock = PTHREAD_MUTEX_INITIALIZER;
-    static unsigned char first = 1;
-    static EVP_MD_CTX mdctx;
-    unsigned char hash[EVP_MAX_MD_SIZE], *input;
-    unsigned int md_len;
-    uint8_t i, offset = 0, out[128];
-    
     pthread_mutex_lock(&lock);
     if (first) {
        EVP_MD_CTX_init(&mdctx);
@@ -618,7 +586,10 @@ int pwddecrypt(uint8_t *in, uint8_t len, char *shared, uint8_t sharedlen, uint8_
        }
        for (i = 0; i < 16; i++)
            out[offset + i] = hash[i] ^ in[offset + i];
-       input = in + offset;
+       if (encrypt_flag)
+           input = out + offset;
+       else
+           input = in + offset;
        offset += 16;
        if (offset == len)
            break;
@@ -635,7 +606,7 @@ int msmppencrypt(uint8_t *text, uint8_t len, uint8_t *shared, uint8_t sharedlen,
     unsigned char hash[EVP_MAX_MD_SIZE];
     unsigned int md_len;
     uint8_t i, offset;
-    
+
     pthread_mutex_lock(&lock);
     if (first) {
        EVP_MD_CTX_init(&mdctx);
@@ -647,7 +618,7 @@ int msmppencrypt(uint8_t *text, uint8_t len, uint8_t *shared, uint8_t sharedlen,
     printfchars(NULL, "msppencrypt salt in", "%02x ", salt, 2);
     printfchars(NULL, "msppencrypt in", "%02x ", text, len);
 #endif
-    
+
     if (!EVP_DigestInit_ex(&mdctx, EVP_md5(), NULL) ||
        !EVP_DigestUpdate(&mdctx, shared, sharedlen) ||
        !EVP_DigestUpdate(&mdctx, auth, 16) ||
@@ -657,15 +628,15 @@ int msmppencrypt(uint8_t *text, uint8_t len, uint8_t *shared, uint8_t sharedlen,
        return 0;
     }
 
-#if 0    
+#if 0
     printfchars(NULL, "msppencrypt hash", "%02x ", hash, 16);
 #endif
-    
+
     for (i = 0; i < 16; i++)
        text[i] ^= hash[i];
-    
+
     for (offset = 16; offset < len; offset += 16) {
-#if 0  
+#if 0
        printf("text + offset - 16 c(%d): ", offset / 16);
        printfchars(NULL, NULL, "%02x ", text + offset - 16, 16);
 #endif
@@ -679,12 +650,12 @@ int msmppencrypt(uint8_t *text, uint8_t len, uint8_t *shared, uint8_t sharedlen,
        }
 #if 0
        printfchars(NULL, "msppencrypt hash", "%02x ", hash, 16);
-#endif    
-       
+#endif
+
        for (i = 0; i < 16; i++)
            text[offset + i] ^= hash[i];
     }
-    
+
 #if 0
     printfchars(NULL, "msppencrypt out", "%02x ", text, len);
 #endif
@@ -701,7 +672,7 @@ int msmppdecrypt(uint8_t *text, uint8_t len, uint8_t *shared, uint8_t sharedlen,
     unsigned int md_len;
     uint8_t i, offset;
     char plain[255];
-    
+
     pthread_mutex_lock(&lock);
     if (first) {
        EVP_MD_CTX_init(&mdctx);
@@ -713,7 +684,7 @@ int msmppdecrypt(uint8_t *text, uint8_t len, uint8_t *shared, uint8_t sharedlen,
     printfchars(NULL, "msppdecrypt salt in", "%02x ", salt, 2);
     printfchars(NULL, "msppdecrypt in", "%02x ", text, len);
 #endif
-    
+
     if (!EVP_DigestInit_ex(&mdctx, EVP_md5(), NULL) ||
        !EVP_DigestUpdate(&mdctx, shared, sharedlen) ||
        !EVP_DigestUpdate(&mdctx, auth, 16) ||
@@ -723,15 +694,15 @@ int msmppdecrypt(uint8_t *text, uint8_t len, uint8_t *shared, uint8_t sharedlen,
        return 0;
     }
 
-#if 0    
+#if 0
     printfchars(NULL, "msppdecrypt hash", "%02x ", hash, 16);
 #endif
-    
+
     for (i = 0; i < 16; i++)
        plain[i] = text[i] ^ hash[i];
-    
+
     for (offset = 16; offset < len; offset += 16) {
-#if 0  
+#if 0
        printf("text + offset - 16 c(%d): ", offset / 16);
        printfchars(NULL, NULL, "%02x ", text + offset - 16, 16);
 #endif
@@ -745,7 +716,7 @@ int msmppdecrypt(uint8_t *text, uint8_t len, uint8_t *shared, uint8_t sharedlen,
        }
 #if 0
        printfchars(NULL, "msppdecrypt hash", "%02x ", hash, 16);
-#endif    
+#endif
 
        for (i = 0; i < 16; i++)
            plain[offset + i] = text[offset + i] ^ hash[i];
@@ -841,7 +812,7 @@ void _internal_removeserversubrealms(struct list *realmlist, struct clsrvconf *s
 void removeserversubrealms(struct list *realmlist, struct clsrvconf *srv) {
     struct list_node *entry;
     struct realm *realm;
-    
+
     for (entry = list_first(realmlist); entry; entry = list_next(entry)) {
        realm = (struct realm *)entry->data;
        pthread_mutex_lock(&realm->mutex);
@@ -855,7 +826,7 @@ void removeserversubrealms(struct list *realmlist, struct clsrvconf *srv) {
        pthread_mutex_unlock(&realm->mutex);
     }
 }
-                       
+
 int attrvalidate(unsigned char *attrs, int length) {
     while (length > 1) {
        if (ATTRLEN(attrs) < 2) {
@@ -879,15 +850,15 @@ int pwdrecrypt(uint8_t *pwd, uint8_t len, char *oldsecret, char *newsecret, uint
        debug(DBG_WARN, "pwdrecrypt: invalid password length");
        return 0;
     }
-       
-    if (!pwddecrypt(pwd, len, oldsecret, strlen(oldsecret), oldauth)) {
+
+    if (!pwdcrypt(0, pwd, len, oldsecret, strlen(oldsecret), oldauth)) {
        debug(DBG_WARN, "pwdrecrypt: cannot decrypt password");
        return 0;
     }
 #ifdef DEBUG
     printfchars(NULL, "pwdrecrypt: password", "%02x ", pwd, len);
-#endif 
-    if (!pwdencrypt(pwd, len, newsecret, strlen(newsecret), newauth)) {
+#endif
+    if (!pwdcrypt(1, pwd, len, newsecret, strlen(newsecret), newauth)) {
        debug(DBG_WARN, "pwdrecrypt: cannot encrypt password");
        return 0;
     }
@@ -911,7 +882,7 @@ int msmpprecrypt(uint8_t *msmpp, uint8_t len, char *oldsecret, char *newsecret,
 int msmppe(unsigned char *attrs, int length, uint8_t type, char *attrtxt, struct request *rq,
           char *oldsecret, char *newsecret) {
     unsigned char *attr;
-    
+
     for (attr = attrs; (attr = attrget(attr, length - (attr - attrs), type)); attr += ATTRLEN(attr)) {
        debug(DBG_DBG, "msmppe: Got %s", attrtxt);
        if (!msmpprecrypt(ATTRVAL(attr), ATTRVALLEN(attr), oldsecret, newsecret, rq->buf + 4, rq->rqauth))
@@ -923,7 +894,7 @@ int msmppe(unsigned char *attrs, int length, uint8_t type, char *attrtxt, struct
 int findvendorsubattr(uint32_t *attrs, uint32_t vendor, uint32_t subattr) {
     if (!attrs)
        return 0;
-    
+
     for (; attrs[0]; attrs += 2)
        if (attrs[0] == vendor && attrs[1] == subattr)
            return 1;
@@ -935,7 +906,7 @@ int dovendorrewriterm(struct tlv *attr, uint32_t *removevendorattrs) {
     uint8_t alen, sublen;
     uint32_t vendor;
     uint8_t *subattrs;
-    
+
     if (!removevendorattrs)
        return 0;
 
@@ -945,13 +916,13 @@ int dovendorrewriterm(struct tlv *attr, uint32_t *removevendorattrs) {
        removevendorattrs += 2;
     if (!*removevendorattrs)
        return 0;
-    
+
     if (findvendorsubattr(removevendorattrs, vendor, 256))
        return 1; /* remove entire vendor attribute */
 
     sublen = attr->l - 4;
     subattrs = attr->v + 4;
-    
+
     if (!attrvalidate(subattrs, sublen)) {
        debug(DBG_INFO, "dovendorrewrite: vendor attribute validation failed, no rewrite");
        return 0;
@@ -982,16 +953,17 @@ void dorewriterm(struct radmsg *msg, uint8_t *rmattrs, uint32_t *rmvattrs) {
            list_removedata(msg->attrs, attr);
            freetlv(attr);
            n = p ? list_next(p) : list_first(msg->attrs);
-       } else
+       } else {
            p = n;
-            n = list_next(n);
+           n = list_next(n);
+       }
     }
 }
 
 int dorewriteadd(struct radmsg *msg, struct list *addattrs) {
     struct list_node *n;
     struct tlv *a;
-    
+
     for (n = list_first(addattrs); n; n = list_next(n)) {
        a = copytlv((struct tlv *)n->data);
        if (!a)
@@ -1006,7 +978,7 @@ int dorewriteadd(struct radmsg *msg, struct list *addattrs) {
 
 int resizeattr(struct tlv *attr, uint8_t newlen) {
     uint8_t *newv;
-    
+
     if (newlen != attr->l) {
        newv = realloc(attr->v, newlen);
        if (!newv)
@@ -1026,14 +998,14 @@ int dorewritemodattr(struct tlv *attr, struct modattr *modattr) {
     in = stringcopy((char *)attr->v, attr->l);
     if (!in)
        return 0;
-    
+
     if (regexec(modattr->regex, in, nmatch, pmatch, 0)) {
        free(in);
        return 1;
     }
-    
+
     out = modattr->replacement;
-    
+
     for (i = start; out[i]; i++) {
        if (out[i] == '\\' && out[i + 1] >= '1' && out[i + 1] <= '9') {
            pfield = &pmatch[out[i + 1] - '0'];
@@ -1088,15 +1060,19 @@ int dorewritemod(struct radmsg *msg, struct list *modattrs) {
 }
 
 int dorewrite(struct radmsg *msg, struct rewrite *rewrite) {
-    if (!rewrite)
-       return 1;
-    if (rewrite->removeattrs || rewrite->removevendorattrs)
-       dorewriterm(msg, rewrite->removeattrs, rewrite->removevendorattrs);
-    if (rewrite->addattrs && !dorewriteadd(msg, rewrite->addattrs))
-       return 0;
-    if (rewrite->modattrs && !dorewritemod(msg, rewrite->modattrs))
-       return 0;
-    return 1;
+    int rv = 1;                        /* Success.  */
+
+    if (rewrite) {
+       if (rewrite->removeattrs || rewrite->removevendorattrs)
+           dorewriterm(msg, rewrite->removeattrs, rewrite->removevendorattrs);
+       if (rewrite->modattrs)
+           if (!dorewritemod(msg, rewrite->modattrs))
+               rv = 0;
+       if (rewrite->addattrs)
+           if (!dorewriteadd(msg, rewrite->addattrs))
+               rv = 0;
+    }
+    return rv;
 }
 
 int rewriteusername(struct request *rq, struct tlv *attr) {
@@ -1112,23 +1088,48 @@ int rewriteusername(struct request *rq, struct tlv *attr) {
     return 1;
 }
 
-int addvendorattr(struct radmsg *msg, uint32_t vendor, struct tlv *attr) {
-    struct tlv *vattr;
+/** Create vendor specific tlv with ATTR.  ATTR is consumed (freed) if
+ * all is well with the new tlv, i.e. if the function returns
+ * !NULL.  */
+static struct tlv *
+makevendortlv(uint32_t vendor, struct tlv *attr)
+{
+    struct tlv *newtlv = NULL;
     uint8_t l, *v;
 
+    if (!attr)
+       return NULL;
     l = attr->l + 6;
     v = malloc(l);
     if (v) {
-       vendor = htonl(vendor);
+       vendor = htonl(vendor & 0x00ffffff); /* MSB=0 according to RFC 2865. */
        memcpy(v, &vendor, 4);
        tlv2buf(v + 4, attr);
-       v[5] += 2;
-       vattr = maketlv(RAD_Attr_Vendor_Specific, l, v);
-       if (vattr && radmsg_add(msg, vattr))
-           return 1;
+       v[5] += 2; /* Vendor length increased for type and length fields. */
+       newtlv = maketlv(RAD_Attr_Vendor_Specific, l, v);
+       if (newtlv == NULL)
+           free(v);
+       else
+           freetlv(attr);
+    }
+    return newtlv;
+}
+
+/** Ad vendor attribute with VENDOR + ATTR and push it on MSG.  ATTR
+ * is consumed.  */
+int addvendorattr(struct radmsg *msg, uint32_t vendor, struct tlv *attr) {
+    struct tlv *vattr;
+
+    vattr = makevendortlv(vendor, attr);
+    if (!vattr) {
+       freetlv(attr);
+       return 0;
+    }
+    if (!radmsg_add(msg, vattr)) {
        freetlv(vattr);
+       return 0;
     }
-    return 0;
+    return 1;
 }
 
 void addttlattr(struct radmsg *msg, uint32_t *attrtype, uint8_t addttl) {
@@ -1137,17 +1138,15 @@ void addttlattr(struct radmsg *msg, uint32_t *attrtype, uint8_t addttl) {
 
     memset(ttl, 0, 4);
     ttl[3] = addttl;
-    
+
     if (attrtype[1] == 256) { /* not vendor */
        attr = maketlv(attrtype[0], 4, ttl);
        if (attr && !radmsg_add(msg, attr))
            freetlv(attr);
     } else {
        attr = maketlv(attrtype[1], 4, ttl);
-       if (attr) {
+       if (attr)
            addvendorattr(msg, attrtype[0], attr);
-           freetlv(attr);
-       }
     }
 }
 
@@ -1178,7 +1177,7 @@ int checkttl(struct radmsg *msg, uint32_t *attrtype) {
     struct list_node *node;
     uint32_t vendor;
     int sublen;
-    
+
     if (attrtype[1] == 256) { /* not vendor */
        attr = radmsg_gettype(msg, attrtype[0]);
        if (attr)
@@ -1192,7 +1191,7 @@ int checkttl(struct radmsg *msg, uint32_t *attrtype) {
            if (ntohl(vendor) != attrtype[0])
                continue;
            sublen = attr->l - 4;
-           subattrs = attr->v + 4;  
+           subattrs = attr->v + 4;
            if (!attrvalidate(subattrs, sublen))
                continue;
            while (sublen > 1) {
@@ -1205,7 +1204,7 @@ int checkttl(struct radmsg *msg, uint32_t *attrtype) {
        }
     return -1;
 }
-         
+
 const char *radmsgtype2string(uint8_t code) {
     static const char *rad_msg_names[] = {
        "", "Access-Request", "Access-Accept", "Access-Reject",
@@ -1237,7 +1236,7 @@ uint8_t *radattr2ascii(struct tlv *attr) {
            l += 2;
     if (l == attr->l)
        return (uint8_t *)stringcopy((char *)attr->v, attr->l);
-    
+
     a = malloc(l + 1);
     if (!a)
        return NULL;
@@ -1266,7 +1265,7 @@ void acclog(struct radmsg *msg, struct client *from) {
     username = radattr2ascii(attr);
     if (username) {
        debug(DBG_INFO, "acclog: accounting-request from client %s (%s) with username: %s", from->conf->name, addr2string(from->addr), username);
-             
+
        free(username);
     }
 }
@@ -1326,7 +1325,7 @@ struct server *findserver(struct realm **realm, struct tlv *username, uint8_t ac
     struct realm *subrealm;
     struct server *server = NULL;
     char *id = (char *)tlv2str(username);
-    
+
     if (!id)
        return NULL;
     /* returns with lock on realm */
@@ -1348,7 +1347,7 @@ struct server *findserver(struct realm **realm, struct tlv *username, uint8_t ac
     if (srvconf)
        server = srvconf->servers;
 
- exit:    
+exit:
     free(id);
     return server;
 }
@@ -1371,7 +1370,7 @@ struct request *newrequest() {
 int addclientrq(struct request *rq) {
     struct request *r;
     struct timeval now;
-    
+
     r = rq->from->rqs[rq->rqid];
     if (r) {
        if (rq->udpport == r->udpport && !memcmp(rq->rqauth, r->rqauth, 16)) {
@@ -1410,7 +1409,7 @@ int radsrv(struct request *rq) {
     struct server *to = NULL;
     struct client *from = rq->from;
     int ttlres;
-    
+
     msg = buf2radmsg(rq->buf, (uint8_t *)from->conf->secret, NULL);
     free(rq->buf);
     rq->buf = NULL;
@@ -1420,17 +1419,17 @@ int radsrv(struct request *rq) {
        freerq(rq);
        return 0;
     }
-    
+
     rq->msg = msg;
     rq->rqid = msg->id;
     memcpy(rq->rqauth, msg->auth, 16);
 
     debug(DBG_DBG, "radsrv: code %d, id %d", msg->code, msg->id);
     if (msg->code != RAD_Access_Request && msg->code != RAD_Status_Server && msg->code != RAD_Accounting_Request) {
-       debug(DBG_INFO, "radsrv: server currently accepts only access-requests, accounting-requests and status-server, ignoring");      
+       debug(DBG_INFO, "radsrv: server currently accepts only access-requests, accounting-requests and status-server, ignoring");
        goto exit;
     }
-    
+
     if (!addclientrq(rq))
        goto exit;
 
@@ -1449,7 +1448,7 @@ int radsrv(struct request *rq) {
        debug(DBG_INFO, "radsrv: ignoring request from client %s (%s), ttl exceeded", from->conf->name, addr2string(from->addr));
        goto exit;
     }
-       
+
     attr = radmsg_gettype(msg, RAD_Attr_User_Name);
     if (!attr) {
        if (msg->code == RAD_Accounting_Request) {
@@ -1459,12 +1458,12 @@ int radsrv(struct request *rq) {
            debug(DBG_INFO, "radsrv: ignoring access request, no username attribute");
        goto exit;
     }
-    
+
     if (from->conf->rewriteusername && !rewriteusername(rq, attr)) {
        debug(DBG_WARN, "radsrv: username malloc failed, ignoring request");
        goto rmclrqexit;
     }
-    
+
     userascii = radattr2ascii(attr);
     if (!userascii)
        goto rmclrqexit;
@@ -1487,8 +1486,10 @@ int radsrv(struct request *rq) {
        }
        goto exit;
     }
-    
-    if (options.loopprevention && !strcmp(from->conf->name, to->conf->name)) {
+
+    if ((to->conf->loopprevention == 1
+        || (to->conf->loopprevention == UCHAR_MAX && options.loopprevention == 1))
+       && !strcmp(from->conf->name, to->conf->name)) {
        debug(DBG_INFO, "radsrv: Loop prevented, not forwarding request from client %s (%s) to server %s, discarding",
              from->conf->name, addr2string(from->addr), to->conf->name);
        goto exit;
@@ -1500,7 +1501,7 @@ int radsrv(struct request *rq) {
        debug(DBG_WARN, "radsrv: failed to generate random auth");
        goto rmclrqexit;
     }
-    
+
 #ifdef DEBUG
     printfchars(NULL, "auth", "%02x ", auth, 16);
 #endif
@@ -1521,20 +1522,20 @@ int radsrv(struct request *rq) {
 
     if (to->conf->rewriteout && !dorewrite(msg, to->conf->rewriteout))
        goto rmclrqexit;
-    
+
     if (ttlres == -1 && (options.addttl || to->conf->addttl))
        addttlattr(msg, options.ttlattrtype, to->conf->addttl ? to->conf->addttl : options.addttl);
-    
+
     free(userascii);
     rq->to = to;
     sendrq(rq);
     pthread_mutex_unlock(&realm->mutex);
     freerealm(realm);
     return 1;
-    
- rmclrqexit:
+
+rmclrqexit:
     rmclientrq(rq, msg->id);
- exit:
+exit:
     freerq(rq);
     free(userascii);
     if (realm) {
@@ -1553,7 +1554,7 @@ void replyh(struct server *server, unsigned char *buf) {
     struct radmsg *msg = NULL;
     struct tlv *attr;
     struct list_node *node;
-    
+
     server->connectionok = 1;
     server->lostrqs = 0;
 
@@ -1565,7 +1566,7 @@ void replyh(struct server *server, unsigned char *buf) {
        debug(DBG_INFO, "replyh: no outstanding request with this id, ignoring reply");
        goto errunlock;
     }
-       
+
     msg = buf2radmsg(buf, (uint8_t *)server->conf->secret, rqout->rq->msg->auth);
     free(buf);
     buf = NULL;
@@ -1581,7 +1582,7 @@ void replyh(struct server *server, unsigned char *buf) {
     debug(DBG_DBG, "got %s message with id %d", radmsgtype2string(msg->code), msg->id);
 
     gettimeofday(&server->lastrcv, NULL);
-    
+
     if (rqout->rq->msg->code == RAD_Status_Server) {
        freerqoutdata(rqout);
        debug(DBG_DBG, "replyh: got status server response from %s", server->conf->name);
@@ -1590,18 +1591,18 @@ void replyh(struct server *server, unsigned char *buf) {
 
     gettimeofday(&server->lastreply, NULL);
     from = rqout->rq->from;
-       
+
     if (server->conf->rewritein && !dorewrite(msg, from->conf->rewritein)) {
        debug(DBG_INFO, "replyh: rewritein failed");
        goto errunlock;
     }
-    
+
     ttlres = checkttl(msg, options.ttlattrtype);
-    if (!ttlres) {    
+    if (!ttlres) {
        debug(DBG_INFO, "replyh: ignoring reply from server %s, ttl exceeded", server->conf->name);
        goto errunlock;
     }
-    
+
     /* MS MPPE */
     for (node = list_first(msg->attrs); node; node = list_next(node)) {
        attr = (struct tlv *)node->data;
@@ -1611,9 +1612,9 @@ void replyh(struct server *server, unsigned char *buf) {
            break;
        if (attr->v[0] != 0 || attr->v[1] != 0 || attr->v[2] != 1 || attr->v[3] != 55)  /* 311 == MS */
            continue;
-           
+
        sublen = attr->l - 4;
-       subattrs = attr->v + 4;  
+       subattrs = attr->v + 4;
        if (!attrvalidate(subattrs, sublen) ||
            !msmppe(subattrs, sublen, RAD_VS_ATTR_MS_MPPE_Send_Key, "MS MPPE Send Key",
                    rqout->rq, server->conf->secret, from->conf->secret) ||
@@ -1633,21 +1634,31 @@ void replyh(struct server *server, unsigned char *buf) {
            replymsg = radattr2ascii(radmsg_gettype(msg, RAD_Attr_Reply_Message));
            if (stationid) {
                if (replymsg) {
-                   debug(DBG_WARN, "%s for user %s stationid %s from %s (%s) to %s (%s)",
-                         radmsgtype2string(msg->code), username, stationid, server->conf->name, replymsg, from->conf->name, addr2string(from->addr));
+                   debug(DBG_NOTICE,
+                         "%s for user %s stationid %s from %s (%s) to %s (%s)",
+                         radmsgtype2string(msg->code), username, stationid,
+                         server->conf->name, replymsg, from->conf->name,
+                         addr2string(from->addr));
                    free(replymsg);
                } else
-                   debug(DBG_WARN, "%s for user %s stationid %s from %s to %s (%s)",
-                         radmsgtype2string(msg->code), username, stationid, server->conf->name, from->conf->name, addr2string(from->addr));
+                   debug(DBG_NOTICE,
+                         "%s for user %s stationid %s from %s to %s (%s)",
+                         radmsgtype2string(msg->code), username, stationid,
+                         server->conf->name, from->conf->name,
+                         addr2string(from->addr));
                free(stationid);
            } else {
                if (replymsg) {
-                   debug(DBG_WARN, "%s for user %s from %s (%s) to %s (%s)",
-                         radmsgtype2string(msg->code), username, server->conf->name, replymsg, from->conf->name, addr2string(from->addr));
+                   debug(DBG_NOTICE, "%s for user %s from %s (%s) to %s (%s)",
+                         radmsgtype2string(msg->code), username,
+                         server->conf->name, replymsg, from->conf->name,
+                         addr2string(from->addr));
                    free(replymsg);
                } else
-                   debug(DBG_WARN, "%s for user %s from %s to %s (%s)",
-                         radmsgtype2string(msg->code), username, server->conf->name, from->conf->name, addr2string(from->addr));
+                   debug(DBG_NOTICE, "%s for user %s from %s to %s (%s)",
+                         radmsgtype2string(msg->code), username,
+                         server->conf->name, from->conf->name,
+                         addr2string(from->addr));
            }
            free(username);
        }
@@ -1656,7 +1667,7 @@ void replyh(struct server *server, unsigned char *buf) {
     msg->id = (char)rqout->rq->rqid;
     memcpy(msg->auth, rqout->rq->rqauth, 16);
 
-#ifdef DEBUG   
+#ifdef DEBUG
     printfchars(NULL, "origauth/buf+4", "%02x ", buf + 4, 16);
 #endif
 
@@ -1672,12 +1683,12 @@ void replyh(struct server *server, unsigned char *buf) {
        debug(DBG_WARN, "replyh: rewriteout failed");
        goto errunlock;
     }
-    
+
     if (ttlres == -1 && (options.addttl || from->conf->addttl))
        addttlattr(msg, options.ttlattrtype, from->conf->addttl ? from->conf->addttl : options.addttl);
 
     debug(msg->code == RAD_Access_Accept || msg->code == RAD_Access_Reject || msg->code == RAD_Accounting_Response ? DBG_WARN : DBG_INFO,
-       "replyh: passing %s to client %s (%s)", radmsgtype2string(msg->code), from->conf->name, addr2string(from->addr));
+         "replyh: passing %s to client %s (%s)", radmsgtype2string(msg->code), from->conf->name, addr2string(from->addr));
 
     radmsg_free(rqout->rq->msg);
     rqout->rq->msg = msg;
@@ -1686,7 +1697,7 @@ void replyh(struct server *server, unsigned char *buf) {
     pthread_mutex_unlock(rqout->lock);
     return;
 
- errunlock:
+errunlock:
     radmsg_free(msg);
     pthread_mutex_unlock(rqout->lock);
     return;
@@ -1711,7 +1722,7 @@ struct request *createstatsrvrq() {
     }
     return rq;
 
- exit:
+exit:
     freerq(rq);
     return NULL;
 }
@@ -1728,16 +1739,16 @@ void *clientwr(void *arg) {
     struct timespec timeout;
     struct request *statsrvrq;
     struct clsrvconf *conf;
-    
+
     conf = server->conf;
-    
+
     if (server->dynamiclookuparg && !dynamicconfig(server)) {
        dynconffail = 1;
        server->dynstartup = 0;
        sleep(900);
        goto errexit;
     }
-    
+
     if (!resolvehostports(conf->hostports, conf->pdef->socktype)) {
        debug(DBG_WARN, "clientwr: resolve failed");
        server->dynstartup = 0;
@@ -1746,7 +1757,7 @@ void *clientwr(void *arg) {
     }
 
     memset(&timeout, 0, sizeof(struct timespec));
-    
+
     if (conf->statusserver) {
        gettimeofday(&server->lastrcv, NULL);
        gettimeofday(&laststatsrv, NULL);
@@ -1762,13 +1773,13 @@ void *clientwr(void *arg) {
        }
        server->connectionok = 1;
        if (pthread_create(&clientrdth, NULL, conf->pdef->clientconnreader, (void *)server)) {
-           debug(DBG_ERR, "clientwr: pthread_create failed");
+           debugerrno(errno, DBG_ERR, "clientwr: pthread_create failed");
            goto errexit;
        }
     } else
        server->connectionok = 1;
     server->dynstartup = 0;
-    
+
     for (;;) {
        pthread_mutex_lock(&server->newrq_mutex);
        if (!server->newrq) {
@@ -1789,7 +1800,7 @@ void *clientwr(void *arg) {
 #if 0
            if (timeout.tv_sec > now.tv_sec)
                debug(DBG_DBG, "clientwr: waiting up to %ld secs for new request", timeout.tv_sec - now.tv_sec);
-#endif     
+#endif
            pthread_cond_timedwait(&server->newrq_cond, &server->newrq_mutex, &timeout);
            timeout.tv_sec = 0;
        }
@@ -1797,10 +1808,10 @@ void *clientwr(void *arg) {
            debug(DBG_DBG, "clientwr: got new request");
            server->newrq = 0;
        }
-#if 0  
+#if 0
        else
            debug(DBG_DBG, "clientwr: request timer expired, processing request queue");
-#endif 
+#endif
        pthread_mutex_unlock(&server->newrq_mutex);
 
        for (i = 0; i < MAX_REQUESTS; i++) {
@@ -1818,7 +1829,7 @@ void *clientwr(void *arg) {
                    pthread_mutex_unlock(rqout->lock);
                }
            }
-               
+
            if (i == MAX_REQUESTS)
                break;
 
@@ -1869,7 +1880,7 @@ void *clientwr(void *arg) {
            }
        }
     }
- errexit:
+errexit:
     conf->servers = NULL;
     if (server->dynamiclookuparg) {
        removeserversubrealms(realms, conf);
@@ -1888,23 +1899,26 @@ void createlistener(uint8_t type, char *arg) {
     struct addrinfo *res;
     int s = -1, on = 1, *sp = NULL;
     struct hostportres *hp = newhostport(arg, protodefs[type]->portdefault, 0);
-    
+
     if (!hp || !resolvehostport(hp, protodefs[type]->socktype, 1))
        debugx(1, DBG_ERR, "createlistener: failed to resolve %s", arg);
-    
+
     for (res = hp->addrinfo; res; res = res->ai_next) {
         s = socket(res->ai_family, res->ai_socktype, res->ai_protocol);
         if (s < 0) {
-            debug(DBG_WARN, "createlistener: socket failed");
+            debugerrno(errno, DBG_WARN, "createlistener: socket failed");
             continue;
         }
        setsockopt(s, SOL_SOCKET, SO_REUSEADDR, &on, sizeof(on));
+
+       disable_DF_bit(s, res);
+
 #ifdef IPV6_V6ONLY
        if (res->ai_family == AF_INET6)
            setsockopt(s, IPPROTO_IPV6, IPV6_V6ONLY, &on, sizeof(on));
 #endif
        if (bind(s, res->ai_addr, res->ai_addrlen)) {
-           debug(DBG_WARN, "createlistener: bind failed");
+           debugerrno(errno, DBG_WARN, "createlistener: bind failed");
            close(s);
            s = -1;
            continue;
@@ -1915,12 +1929,12 @@ void createlistener(uint8_t type, char *arg) {
             debugx(1, DBG_ERR, "malloc failed");
        *sp = s;
        if (pthread_create(&th, NULL, protodefs[type]->listener, (void *)sp))
-            debugx(1, DBG_ERR, "pthread_create failed");
+            debugerrnox(errno, DBG_ERR, "pthread_create failed");
        pthread_detach(th);
     }
     if (!sp)
        debugx(1, DBG_ERR, "createlistener: socket/bind failed");
-    
+
     debug(DBG_WARN, "createlistener: listening for %s on %s:%s", protodefs[type]->name, hp->host ? hp->host : "*", hp->port);
     freehostport(hp);
 }
@@ -1941,7 +1955,7 @@ void sslinit() {
     int i;
     time_t t;
     pid_t pid;
-    
+
     ssl_locks = calloc(CRYPTO_num_locks(), sizeof(pthread_mutex_t));
     ssl_lock_count = OPENSSL_malloc(CRYPTO_num_locks() * sizeof(long));
     for (i = 0; i < CRYPTO_num_locks(); i++) {
@@ -1967,10 +1981,10 @@ struct list *addsrvconfs(char *value, char **names) {
     int n;
     struct list_node *entry;
     struct clsrvconf *conf = NULL;
-    
+
     if (!names || !*names)
        return NULL;
-    
+
     conflist = list_create();
     if (!conflist) {
        debug(DBG_ERR, "malloc failed");
@@ -2023,7 +2037,7 @@ struct realm *addrealm(struct list *realmlist, char *value, char **servers, char
     int n;
     struct realm *realm;
     char *s, *regex = NULL;
-    
+
     if (*value == '/') {
        /* regexp, remove optional trailing / if present */
        if (value[strlen(value) - 1] == '/')
@@ -2062,9 +2076,9 @@ struct realm *addrealm(struct list *realmlist, char *value, char **servers, char
        goto exit;
     }
     memset(realm, 0, sizeof(struct realm));
-    
+
     if (pthread_mutex_init(&realm->mutex, NULL)) {
-       debug(DBG_ERR, "mutex init failed");
+       debugerrno(errno, DBG_ERR, "mutex init failed");
        free(realm);
        realm = NULL;
        goto exit;
@@ -2081,18 +2095,18 @@ struct realm *addrealm(struct list *realmlist, char *value, char **servers, char
     }
     realm->message = message;
     realm->accresp = accresp;
-    
+
     if (regcomp(&realm->regex, regex ? regex : value + 1, REG_EXTENDED | REG_ICASE | REG_NOSUB)) {
        debug(DBG_ERR, "addrealm: failed to compile regular expression %s", regex ? regex : value + 1);
        goto errexit;
     }
-    
+
     if (servers && *servers) {
        realm->srvconfs = addsrvconfs(value, servers);
        if (!realm->srvconfs)
            goto errexit;
     }
-    
+
     if (accservers && *accservers) {
        realm->accsrvconfs = addsrvconfs(value, accservers);
        if (!realm->accsrvconfs)
@@ -2104,16 +2118,16 @@ struct realm *addrealm(struct list *realmlist, char *value, char **servers, char
        pthread_mutex_destroy(&realm->mutex);
        goto errexit;
     }
-    
+
     debug(DBG_DBG, "addrealm: added realm %s", value);
     goto exit;
 
- errexit:
+errexit:
     while (list_shift(realm->srvconfs));
     while (list_shift(realm->accsrvconfs));
     freerealm(realm);
     realm = NULL;
- exit:
+exit:
     free(regex);
     if (servers) {
        if (realm)
@@ -2141,7 +2155,7 @@ struct list *createsubrealmservers(struct realm *realm, struct list *srvconfs) {
        if (!subrealmservers)
            return NULL;
     }
-    
+
     for (entry = list_first(srvconfs); entry; entry = list_next(entry)) {
        conf = (struct clsrvconf *)entry->data;
        if (!conf->servers && conf->dynamiclookupcommand) {
@@ -2155,7 +2169,7 @@ struct list *createsubrealmservers(struct realm *realm, struct list *srvconfs) {
                srvconf->servers->dynamiclookuparg = stringcopy(realm->name, 0);
                srvconf->servers->dynstartup = 1;
                if (pthread_create(&clientth, NULL, clientwr, (void *)(srvconf->servers))) {
-                   debug(DBG_ERR, "pthread_create failed");
+                   debugerrno(errno, DBG_ERR, "pthread_create failed");
                    freeserver(srvconf->servers, 1);
                    srvconf->servers = NULL;
                } else
@@ -2172,11 +2186,11 @@ struct list *createsubrealmservers(struct realm *realm, struct list *srvconfs) {
     }
     return subrealmservers;
 }
-    
+
 struct realm *adddynamicrealmserver(struct realm *realm, char *id) {
     struct realm *newrealm = NULL;
     char *realmname, *s;
-    
+
     /* create dynamic for the realm (string after last @, exit if nothing after @ */
     realmname = strrchr(id, '@');
     if (!realmname)
@@ -2187,12 +2201,12 @@ struct realm *adddynamicrealmserver(struct realm *realm, char *id) {
     for (s = realmname; *s; s++)
        if (*s != '.' && *s != '-' && !isalnum((int)*s))
            return NULL;
-    
+
     if (!realm->subrealms)
        realm->subrealms = list_create();
     if (!realm->subrealms)
        return NULL;
-    
+
     newrealm = addrealm(realm->subrealms, realmname, NULL, NULL, stringcopy(realm->message, 0), realm->accresp);
     if (!newrealm) {
        list_destroy(realm->subrealms);
@@ -2212,17 +2226,17 @@ int dynamicconfig(struct server *server) {
     pid_t pid;
     struct clsrvconf *conf = server->conf;
     struct gconffile *cf = NULL;
-    
+
     /* for now we only learn hostname/address */
     debug(DBG_DBG, "dynamicconfig: need dynamic server config for %s", server->dynamiclookuparg);
 
     if (pipe(fd) > 0) {
-       debug(DBG_ERR, "dynamicconfig: pipe error");
+       debugerrno(errno, DBG_ERR, "dynamicconfig: pipe error");
        goto errexit;
     }
     pid = fork();
     if (pid < 0) {
-       debug(DBG_ERR, "dynamicconfig: fork error");
+       debugerrno(errno, DBG_ERR, "dynamicconfig: fork error");
        close(fd[0]);
        close(fd[1]);
        goto errexit;
@@ -2243,14 +2257,14 @@ int dynamicconfig(struct server *server) {
     ok = getgenericconfig(&cf, NULL,
                          "Server", CONF_CBK, confserver_cb, (void *)conf,
                          NULL
-                         );
+       );
     freegconf(&cf);
-       
+
     if (waitpid(pid, &status, 0) < 0) {
-       debug(DBG_ERR, "dynamicconfig: wait error");
+       debugerrno(errno, DBG_ERR, "dynamicconfig: wait error");
        goto errexit;
     }
-    
+
     if (status) {
        debug(DBG_INFO, "dynamicconfig: command exited with status %d", WEXITSTATUS(status));
        goto errexit;
@@ -2259,7 +2273,7 @@ int dynamicconfig(struct server *server) {
     if (ok)
        return 1;
 
- errexit:    
+errexit:
     debug(DBG_WARN, "dynamicconfig: failed to obtain dynamic server config");
     return 0;
 }
@@ -2267,18 +2281,23 @@ int dynamicconfig(struct server *server) {
 /* should accept both names and numeric values, only numeric right now */
 uint8_t attrname2val(char *attrname) {
     int val = 0;
-    
+
     val = atoi(attrname);
     return val > 0 && val < 256 ? val : 0;
 }
 
+/* ATTRNAME is on the form vendor[:type].
+   If only vendor is found, TYPE is set to 256 and 1 is returned.
+   If type is >= 256, 1 is returned.
+   Otherwise, 0 is returned.
+*/
 /* should accept both names and numeric values, only numeric right now */
 int vattrname2val(char *attrname, uint32_t *vendor, uint32_t *type) {
     char *s;
-    
+
     *vendor = atoi(attrname);
     s = strchr(attrname, ':');
-    if (!s) {
+    if (!s) {                  /* Only vendor was found.  */
        *type = 256;
        return 1;
     }
@@ -2286,22 +2305,42 @@ int vattrname2val(char *attrname, uint32_t *vendor, uint32_t *type) {
     return *type < 256;
 }
 
-/* should accept both names and numeric values, only numeric right now */
-struct tlv *extractattr(char *nameval) {
+/** Extract attributes from string NAMEVAL, create a struct tlv and
+ * return the tlv.  If VENDOR_FLAG, NAMEVAL is on the form
+ * "<vendor>:<name>:<val>" and otherwise it's "<name>:<val>".  Return
+ * NULL if fields are missing or if conversion fails.
+ *
+ * FIXME: Should accept both names and numeric values, only numeric
+ * right now */
+struct tlv *extractattr(char *nameval, char vendor_flag) {
     int len, name = 0;
-    char *s;
+    int vendor = 0;        /* Vendor 0 is reserved, see RFC 1700.  */
+    char *s, *s2;
     struct tlv *a;
-    
+
     s = strchr(nameval, ':');
-    name = atoi(nameval);
-    if (!s || name < 1 || name > 255)
+    if (!s)
        return NULL;
+    name = atoi(nameval);
+
+    if (vendor_flag) {
+       s2 = strchr(s + 1, ':');
+       if (!s2)
+           return NULL;
+       vendor = name;
+       name = atoi(s + 1);
+       s = s2;
+    }
     len = strlen(s + 1);
     if (len > 253)
        return NULL;
+
+    if (name < 1 || name > 255)
+       return NULL;
     a = malloc(sizeof(struct tlv));
     if (!a)
        return NULL;
+
     a->v = (uint8_t *)stringcopy(s + 1, 0);
     if (!a->v) {
        free(a);
@@ -2309,6 +2348,10 @@ struct tlv *extractattr(char *nameval) {
     }
     a->t = name;
     a->l = len;
+
+    if (vendor_flag)
+       a = makevendortlv(vendor, a);
+
     return a;
 }
 
@@ -2351,7 +2394,7 @@ struct modattr *extractmodattr(char *nameval) {
        debug(DBG_ERR, "malloc failed");
        return NULL;
     }
-       
+
     m->regex = malloc(sizeof(regex_t));
     if (!m->regex) {
        free(m->replacement);
@@ -2359,7 +2402,7 @@ struct modattr *extractmodattr(char *nameval) {
        debug(DBG_ERR, "malloc failed");
        return NULL;
     }
-    
+
     if (regcomp(m->regex, s, REG_ICASE | REG_EXTENDED)) {
        free(m->regex);
        free(m->replacement);
@@ -2381,7 +2424,8 @@ struct rewrite *getrewrite(char *alt1, char *alt2) {
     return NULL;
 }
 
-void addrewrite(char *value, char **rmattrs, char **rmvattrs, char **addattrs, char **modattrs) {
+void addrewrite(char *value, char **rmattrs, char **rmvattrs, char **addattrs, char **addvattrs, char **modattrs)
+{
     struct rewrite *rewrite = NULL;
     int i, n;
     uint8_t *rma = NULL;
@@ -2389,47 +2433,62 @@ void addrewrite(char *value, char **rmattrs, char **rmvattrs, char **addattrs, c
     struct list *adda = NULL, *moda = NULL;
     struct tlv *a;
     struct modattr *m;
-    
+
     if (rmattrs) {
        for (n = 0; rmattrs[n]; n++);
        rma = calloc(n + 1, sizeof(uint8_t));
        if (!rma)
            debugx(1, DBG_ERR, "malloc failed");
-    
+
        for (i = 0; i < n; i++)
            if (!(rma[i] = attrname2val(rmattrs[i])))
-               debugx(1, DBG_ERR, "addrewrite: invalid attribute %s", rmattrs[i]);
+               debugx(1, DBG_ERR, "addrewrite: removing invalid attribute %s", rmattrs[i]);
        freegconfmstr(rmattrs);
        rma[i] = 0;
     }
-    
+
     if (rmvattrs) {
        for (n = 0; rmvattrs[n]; n++);
        rmva = calloc(2 * n + 1, sizeof(uint32_t));
        if (!rmva)
            debugx(1, DBG_ERR, "malloc failed");
-    
+
        for (p = rmva, i = 0; i < n; i++, p += 2)
            if (!vattrname2val(rmvattrs[i], p, p + 1))
-               debugx(1, DBG_ERR, "addrewrite: invalid vendor attribute %s", rmvattrs[i]);
+               debugx(1, DBG_ERR, "addrewrite: removing invalid vendor attribute %s", rmvattrs[i]);
        freegconfmstr(rmvattrs);
        *p = 0;
     }
-    
+
     if (addattrs) {
        adda = list_create();
        if (!adda)
            debugx(1, DBG_ERR, "malloc failed");
        for (i = 0; addattrs[i]; i++) {
-           a = extractattr(addattrs[i]);
+           a = extractattr(addattrs[i], 0);
            if (!a)
-               debugx(1, DBG_ERR, "addrewrite: invalid attribute %s", addattrs[i]);
+               debugx(1, DBG_ERR, "addrewrite: adding invalid attribute %s", addattrs[i]);
            if (!list_push(adda, a))
                debugx(1, DBG_ERR, "malloc failed");
        }
        freegconfmstr(addattrs);
     }
 
+    if (addvattrs) {
+       if (!adda)
+           adda = list_create();
+       if (!adda)
+           debugx(1, DBG_ERR, "malloc failed");
+       for (i = 0; addvattrs[i]; i++) {
+           a = extractattr(addvattrs[i], 1);
+           if (!a)
+               debugx(1, DBG_ERR, "addrewrite: adding invalid vendor attribute %s", addvattrs[i]);
+           if (!list_push(adda, a))
+               debugx(1, DBG_ERR, "malloc failed");
+       }
+       freegconfmstr(addvattrs);
+    }
+
     if (modattrs) {
        moda = list_create();
        if (!moda)
@@ -2437,13 +2496,13 @@ void addrewrite(char *value, char **rmattrs, char **rmvattrs, char **addattrs, c
        for (i = 0; modattrs[i]; i++) {
            m = extractmodattr(modattrs[i]);
            if (!m)
-               debugx(1, DBG_ERR, "addrewrite: invalid attribute %s", modattrs[i]);
+               debugx(1, DBG_ERR, "addrewrite: modifying invalid attribute %s", modattrs[i]);
            if (!list_push(moda, m))
                debugx(1, DBG_ERR, "malloc failed");
        }
        freegconfmstr(modattrs);
     }
-       
+
     if (rma || rmva || adda || moda) {
        rewrite = malloc(sizeof(struct rewrite));
        if (!rewrite)
@@ -2453,7 +2512,7 @@ void addrewrite(char *value, char **rmattrs, char **rmvattrs, char **addattrs, c
        rewrite->addattrs = adda;
        rewrite->modattrs = moda;
     }
-    
+
     if (!hash_insert(rewriteconfs, value, strlen(value), rewrite))
        debugx(1, DBG_ERR, "malloc failed");
     debug(DBG_DBG, "addrewrite: added rewrite block %s", value);
@@ -2461,7 +2520,7 @@ void addrewrite(char *value, char **rmattrs, char **rmvattrs, char **addattrs, c
 
 int setttlattr(struct options *opts, char *defaultattr) {
     char *ttlattr = opts->ttlattr ? opts->ttlattr : defaultattr;
-     
+
     if (vattrname2val(ttlattr, opts->ttlattrtype, opts->ttlattrtype + 1) &&
        (opts->ttlattrtype[1] != 256 || opts->ttlattrtype[0] < 256))
        return 1;
@@ -2504,7 +2563,7 @@ void freeclsrvconf(struct clsrvconf *conf) {
 
 int mergeconfstring(char **dst, char **src) {
     char *t;
-    
+
     if (*src) {
        *dst = *src;
        *src = NULL;
@@ -2524,7 +2583,7 @@ int mergeconfstring(char **dst, char **src) {
 char **mstringcopy(char **in) {
     char **out;
     int n;
-    
+
     if (!in)
        return NULL;
 
@@ -2545,7 +2604,7 @@ char **mstringcopy(char **in) {
 
 int mergeconfmstring(char ***dst, char ***src) {
     char **t;
-    
+
     if (*src) {
        *dst = *src;
        *src = NULL;
@@ -2589,7 +2648,7 @@ int confclient_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
     struct clsrvconf *conf;
     char *conftype = NULL, *rewriteinalias = NULL;
     long int dupinterval = LONG_MIN, addttl = LONG_MIN;
-    
+
     debug(DBG_DBG, "confclient_cb called for %s", block);
 
     conf = malloc(sizeof(struct clsrvconf));
@@ -2597,16 +2656,16 @@ int confclient_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
        debugx(1, DBG_ERR, "malloc failed");
     memset(conf, 0, sizeof(struct clsrvconf));
     conf->certnamecheck = 1;
-    
+
     if (!getgenericconfig(cf, block,
                          "type", CONF_STR, &conftype,
                          "host", CONF_MSTR, &conf->hostsrc,
                          "secret", CONF_STR, &conf->secret,
-#if defined(RADPROT_TLS) || defined(RADPROT_DTLS)    
+#if defined(RADPROT_TLS) || defined(RADPROT_DTLS)
                          "tls", CONF_STR, &conf->tls,
                          "matchcertificateattribute", CONF_STR, &conf->matchcertattr,
                          "CertificateNameCheck", CONF_BLN, &conf->certnamecheck,
-#endif                   
+#endif
                          "DuplicateInterval", CONF_LINT, &dupinterval,
                          "addTTL", CONF_LINT, &addttl,
                          "rewrite", CONF_STR, &rewriteinalias,
@@ -2614,9 +2673,9 @@ int confclient_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
                          "rewriteOut", CONF_STR, &conf->confrewriteout,
                          "rewriteattribute", CONF_STR, &conf->confrewriteusername,
                          NULL
-                         ))
+           ))
        debugx(1, DBG_ERR, "configuration error");
-    
+
     conf->name = stringcopy(val, 0);
     if (conf->name && !conf->hostsrc) {
        conf->hostsrc = malloc(2 * sizeof(char *));
@@ -2627,7 +2686,7 @@ int confclient_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
     }
     if (!conf->name || !conf->hostsrc || !conf->hostsrc[0])
        debugx(1, DBG_ERR, "malloc failed");
-       
+
     if (!conftype)
        debugx(1, DBG_ERR, "error in block %s, option type missing", block);
     conf->type = protoname2int(conftype);
@@ -2636,7 +2695,7 @@ int confclient_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
     free(conftype);
     conf->pdef = protodefs[conf->type];
 
-#if defined(RADPROT_TLS) || defined(RADPROT_DTLS)    
+#if defined(RADPROT_TLS) || defined(RADPROT_DTLS)
     if (conf->type == RAD_TLS || conf->type == RAD_DTLS) {
        conf->tlsconf = conf->tls ? tlsgettls(conf->tls, NULL) : tlsgettls("defaultclient", "default");
        if (!conf->tlsconf)
@@ -2645,20 +2704,20 @@ int confclient_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
            debugx(1, DBG_ERR, "error in block %s, invalid MatchCertificateAttributeValue", block);
     }
 #endif
-    
+
     if (dupinterval != LONG_MIN) {
        if (dupinterval < 0 || dupinterval > 255)
            debugx(1, DBG_ERR, "error in block %s, value of option DuplicateInterval is %d, must be 0-255", block, dupinterval);
        conf->dupinterval = (uint8_t)dupinterval;
     } else
        conf->dupinterval = conf->pdef->duplicateintervaldefault;
-    
+
     if (addttl != LONG_MIN) {
        if (addttl < 1 || addttl > 255)
            debugx(1, DBG_ERR, "error in block %s, value of option addTTL is %d, must be 1-255", block, addttl);
        conf->addttl = (uint8_t)addttl;
     }
-    
+
     if (!conf->confrewritein)
        conf->confrewritein = rewriteinalias;
     else
@@ -2666,7 +2725,7 @@ int confclient_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
     conf->rewritein = conf->confrewritein ? getrewrite(conf->confrewritein, NULL) : getrewrite("defaultclient", "default");
     if (conf->confrewriteout)
        conf->rewriteout = getrewrite(conf->confrewriteout, NULL);
-    
+
     if (conf->confrewriteusername) {
        conf->rewriteusername = extractmodattr(conf->confrewriteusername);
        if (!conf->rewriteusername)
@@ -2676,7 +2735,7 @@ int confclient_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
     if (!addhostport(&conf->hostports, conf->hostsrc, conf->pdef->portdefault, 1) ||
        !resolvehostports(conf->hostports, conf->pdef->socktype))
        debugx(1, DBG_ERR, "resolve failed, exiting");
-    
+
     if (!conf->secret) {
        if (!conf->pdef->secretdefault)
            debugx(1, DBG_ERR, "error in block %s, secret must be specified for transport type %s", block, conf->pdef->name);
@@ -2696,7 +2755,7 @@ int confclient_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
 }
 
 int compileserverconfig(struct clsrvconf *conf, const char *block) {
-#if defined(RADPROT_TLS) || defined(RADPROT_DTLS)    
+#if defined(RADPROT_TLS) || defined(RADPROT_DTLS)
     if (conf->type == RAD_TLS || conf->type == RAD_DTLS) {
        conf->tlsconf = conf->tls ? tlsgettls(conf->tls, NULL) : tlsgettls("defaultserver", "default");
        if (!conf->tlsconf) {
@@ -2709,7 +2768,7 @@ int compileserverconfig(struct clsrvconf *conf, const char *block) {
        }
     }
 #endif
-    
+
     if (!conf->portsrc) {
        conf->portsrc = stringcopy(conf->pdef->portdefault, 0);
        if (!conf->portsrc) {
@@ -2717,16 +2776,16 @@ int compileserverconfig(struct clsrvconf *conf, const char *block) {
            return 0;
        }
     }
-    
+
     if (conf->retryinterval == 255)
        conf->retryinterval = conf->pdef->retryintervaldefault;
     if (conf->retrycount == 255)
        conf->retrycount = conf->pdef->retrycountdefault;
-    
+
     conf->rewritein = conf->confrewritein ? getrewrite(conf->confrewritein, NULL) : getrewrite("defaultserver", "default");
     if (conf->confrewriteout)
        conf->rewriteout = getrewrite(conf->confrewriteout, NULL);
-    
+
     if (!addhostport(&conf->hostports, conf->hostsrc, conf->portsrc, 0)) {
        debug(DBG_ERR, "error in block %s, failed to parse %s", block, conf->hostsrc);
        return 0;
@@ -2738,12 +2797,12 @@ int compileserverconfig(struct clsrvconf *conf, const char *block) {
     }
     return 1;
 }
-                       
+
 int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char *val) {
     struct clsrvconf *conf, *resconf;
     char *conftype = NULL, *rewriteinalias = NULL;
     long int retryinterval = LONG_MIN, retrycount = LONG_MIN, addttl = LONG_MIN;
-    
+
     debug(DBG_DBG, "confserver_cb called for %s", block);
 
     conf = malloc(sizeof(struct clsrvconf));
@@ -2752,6 +2811,7 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
        return 0;
     }
     memset(conf, 0, sizeof(struct clsrvconf));
+    conf->loopprevention = UCHAR_MAX; /* Uninitialized.  */
     resconf = (struct clsrvconf *)arg;
     if (resconf) {
        conf->statusserver = resconf->statusserver;
@@ -2764,11 +2824,11 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
                          "host", CONF_MSTR, &conf->hostsrc,
                          "port", CONF_STR, &conf->portsrc,
                          "secret", CONF_STR, &conf->secret,
-#if defined(RADPROT_TLS) || defined(RADPROT_DTLS)    
+#if defined(RADPROT_TLS) || defined(RADPROT_DTLS)
                          "tls", CONF_STR, &conf->tls,
                          "MatchCertificateAttribute", CONF_STR, &conf->matchcertattr,
                          "CertificateNameCheck", CONF_BLN, &conf->certnamecheck,
-#endif                   
+#endif
                          "addTTL", CONF_LINT, &addttl,
                          "rewrite", CONF_STR, &rewriteinalias,
                          "rewriteIn", CONF_STR, &conf->confrewritein,
@@ -2777,12 +2837,13 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
                          "RetryInterval", CONF_LINT, &retryinterval,
                          "RetryCount", CONF_LINT, &retrycount,
                          "DynamicLookupCommand", CONF_STR, &conf->dynamiclookupcommand,
+                         "LoopPrevention", CONF_BLN, &conf->loopprevention,
                          NULL
-                         )) {
+           )) {
        debug(DBG_ERR, "configuration error");
        goto errexit;
     }
-    
+
     conf->name = stringcopy(val, 0);
     if (conf->name && !conf->hostsrc) {
        conf->hostsrc = malloc(2 * sizeof(char *));
@@ -2807,7 +2868,7 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
     }
     free(conftype);
     conftype = NULL;
-       
+
     conf->pdef = protodefs[conf->type];
 
     if (!conf->confrewritein)
@@ -2824,7 +2885,7 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
        conf->retryinterval = (uint8_t)retryinterval;
     } else
        conf->retryinterval = 255;
-    
+
     if (retrycount != LONG_MIN) {
        if (retrycount < 0 || retrycount > conf->pdef->retrycountmax) {
            debug(DBG_ERR, "error in block %s, value of option RetryCount is %d, must be 0-%d", block, retrycount, conf->pdef->retrycountmax);
@@ -2833,7 +2894,7 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
        conf->retrycount = (uint8_t)retrycount;
     } else
        conf->retrycount = 255;
-    
+
     if (addttl != LONG_MIN) {
        if (addttl < 1 || addttl > 255) {
            debug(DBG_ERR, "error in block %s, value of option addTTL is %d, must be 1-255", block, addttl);
@@ -2841,7 +2902,7 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
        }
        conf->addttl = (uint8_t)addttl;
     }
-    
+
     if (resconf) {
        if (!mergesrvconf(resconf, conf))
            goto errexit;
@@ -2857,7 +2918,7 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
        if (!compileserverconfig(conf, block))
            goto errexit;
     }
-    
+
     if (!conf->secret) {
        if (!conf->pdef->secretdefault) {
            debug(DBG_ERR, "error in block %s, secret must be specified for transport type %s", block, conf->pdef->name);
@@ -2869,7 +2930,7 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
            return 0;
        }
     }
-    
+
     if (resconf)
        return 1;
 
@@ -2879,7 +2940,7 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
     }
     return 1;
 
- errexit:
+errexit:
     free(conftype);
     free(rewriteinalias);
     freeclsrvconf(conf);
@@ -2889,16 +2950,16 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
 int confrealm_cb(struct gconffile **cf, void *arg, char *block, char *opt, char *val) {
     char **servers = NULL, **accservers = NULL, *msg = NULL;
     uint8_t accresp = 0;
-    
+
     debug(DBG_DBG, "confrealm_cb called for %s", block);
-    
+
     if (!getgenericconfig(cf, block,
                          "server", CONF_MSTR, &servers,
                          "accountingServer", CONF_MSTR, &accservers,
                          "ReplyMessage", CONF_STR, &msg,
                          "AccountingResponse", CONF_BLN, &accresp,
                          NULL
-                         ))
+           ))
        debugx(1, DBG_ERR, "configuration error");
 
     addrealm(realms, val, servers, accservers, msg, accresp);
@@ -2906,19 +2967,22 @@ int confrealm_cb(struct gconffile **cf, void *arg, char *block, char *opt, char
 }
 
 int confrewrite_cb(struct gconffile **cf, void *arg, char *block, char *opt, char *val) {
-    char **rmattrs = NULL, **rmvattrs = NULL, **addattrs = NULL, **modattrs = NULL;
-    
+    char **rmattrs = NULL, **rmvattrs = NULL;
+    char **addattrs = NULL, **addvattrs = NULL;
+    char **modattrs = NULL;
+
     debug(DBG_DBG, "confrewrite_cb called for %s", block);
-    
+
     if (!getgenericconfig(cf, block,
                          "removeAttribute", CONF_MSTR, &rmattrs,
                          "removeVendorAttribute", CONF_MSTR, &rmvattrs,
                          "addAttribute", CONF_MSTR, &addattrs,
+                         "addVendorAttribute", CONF_MSTR, &addvattrs,
                          "modifyAttribute", CONF_MSTR, &modattrs,
                          NULL
-                         ))
+           ))
        debugx(1, DBG_ERR, "configuration error");
-    addrewrite(val, rmattrs, rmvattrs, addattrs, modattrs);
+    addrewrite(val, rmattrs, rmvattrs, addattrs, addvattrs, modattrs);
     return 1;
 }
 
@@ -2941,45 +3005,45 @@ void getmainconfig(const char *configfile) {
     char **listenargs[RAD_PROTOCOUNT];
     char *sourcearg[RAD_PROTOCOUNT];
     int i;
-    
+
     cfs = openconfigfile(configfile);
     memset(&options, 0, sizeof(options));
     memset(&listenargs, 0, sizeof(listenargs));
     memset(&sourcearg, 0, sizeof(sourcearg));
-    
+
     clconfs = list_create();
     if (!clconfs)
        debugx(1, DBG_ERR, "malloc failed");
-    
+
     srvconfs = list_create();
     if (!srvconfs)
        debugx(1, DBG_ERR, "malloc failed");
-    
+
     realms = list_create();
     if (!realms)
-       debugx(1, DBG_ERR, "malloc failed");    
+       debugx(1, DBG_ERR, "malloc failed");
+
     rewriteconfs = hash_create();
     if (!rewriteconfs)
-       debugx(1, DBG_ERR, "malloc failed");    
+       debugx(1, DBG_ERR, "malloc failed");
+
     if (!getgenericconfig(&cfs, NULL,
-#ifdef RADPROT_UDP                       
+#ifdef RADPROT_UDP
                          "ListenUDP", CONF_MSTR, &listenargs[RAD_UDP],
                          "SourceUDP", CONF_STR, &sourcearg[RAD_UDP],
-#endif                   
-#ifdef RADPROT_TCP                       
+#endif
+#ifdef RADPROT_TCP
                          "ListenTCP", CONF_MSTR, &listenargs[RAD_TCP],
                          "SourceTCP", CONF_STR, &sourcearg[RAD_TCP],
-#endif                   
+#endif
 #ifdef RADPROT_TLS
                          "ListenTLS", CONF_MSTR, &listenargs[RAD_TLS],
                          "SourceTLS", CONF_STR, &sourcearg[RAD_TLS],
-#endif                   
+#endif
 #ifdef RADPROT_DTLS
                          "ListenDTLS", CONF_MSTR, &listenargs[RAD_DTLS],
                          "SourceDTLS", CONF_STR, &sourcearg[RAD_DTLS],
-#endif                   
+#endif
                          "TTLAttribute", CONF_STR, &options.ttlattr,
                          "addTTL", CONF_LINT, &addttl,
                          "LogLevel", CONF_LINT, &loglevel,
@@ -2988,17 +3052,17 @@ void getmainconfig(const char *configfile) {
                          "Client", CONF_CBK, confclient_cb, NULL,
                          "Server", CONF_CBK, confserver_cb, NULL,
                          "Realm", CONF_CBK, confrealm_cb, NULL,
-#if defined(RADPROT_TLS) || defined(RADPROT_DTLS)                        
+#if defined(RADPROT_TLS) || defined(RADPROT_DTLS)
                          "TLS", CONF_CBK, conftls_cb, NULL,
-#endif                   
+#endif
                          "Rewrite", CONF_CBK, confrewrite_cb, NULL,
                          NULL
-                         ))
+           ))
        debugx(1, DBG_ERR, "configuration error");
-    
+
     if (loglevel != LONG_MIN) {
-       if (loglevel < 1 || loglevel > 4)
-           debugx(1, DBG_ERR, "error in %s, value of option LogLevel is %d, must be 1, 2, 3 or 4", configfile, loglevel);
+       if (loglevel < 1 || loglevel > 5)
+           debugx(1, DBG_ERR, "error in %s, value of option LogLevel is %d, must be 1, 2, 3, 4 or 5", configfile, loglevel);
        options.loglevel = (uint8_t)loglevel;
     }
     if (addttl != LONG_MIN) {
@@ -3023,8 +3087,8 @@ void getargs(int argc, char **argv, uint8_t *foreground, uint8_t *pretend, uint8
            *configfile = optarg;
            break;
        case 'd':
-           if (strlen(optarg) != 1 || *optarg < '1' || *optarg > '4')
-               debugx(1, DBG_ERR, "Debug level must be 1, 2, 3 or 4, not %s", optarg);
+           if (strlen(optarg) != 1 || *optarg < '1' || *optarg > '5')
+               debugx(1, DBG_ERR, "Debug level must be 1, 2, 3, 4 or 5, not %s", optarg);
            *loglevel = *optarg - '0';
            break;
        case 'f':
@@ -3037,21 +3101,21 @@ void getargs(int argc, char **argv, uint8_t *foreground, uint8_t *pretend, uint8
            *pretend = 1;
            break;
        case 'v':
-               debug(DBG_ERR, "radsecproxy revision $Rev$");
-               debug(DBG_ERR, "This binary was built with support for the following transports:");
+           debug(DBG_ERR, "radsecproxy revision %s", PACKAGE_VERSION);
+           debug(DBG_ERR, "This binary was built with support for the following transports:");
 #ifdef RADPROT_UDP
-               debug(DBG_ERR, "  UDP");
-#endif         
+           debug(DBG_ERR, "  UDP");
+#endif
 #ifdef RADPROT_TCP
-               debug(DBG_ERR, "  TCP");
-#endif         
+           debug(DBG_ERR, "  TCP");
+#endif
 #ifdef RADPROT_TLS
-               debug(DBG_ERR, "  TLS");
-#endif         
+           debug(DBG_ERR, "  TLS");
+#endif
 #ifdef RADPROT_DTLS
-               debug(DBG_ERR, "  DTLS");
+           debug(DBG_ERR, "  DTLS");
 #endif
-               exit(0);
+           exit(0);
        default:
            goto usage;
        }
@@ -3059,7 +3123,7 @@ void getargs(int argc, char **argv, uint8_t *foreground, uint8_t *pretend, uint8
     if (!(argc - optind))
        return;
 
- usage:
+usage:
     debugx(1, DBG_ERR, "Usage:\n%s [ -c configfile ] [ -d debuglevel ] [ -f ] [ -i pidfile ] [ -p ] [ -v ]", argv[0]);
 }
 
@@ -3107,10 +3171,10 @@ void *sighandler(void *arg) {
 }
 
 int createpidfile(const char *pidfile) {
-    int r;
+    int r = 0;
     FILE *f = fopen(pidfile, "w");
     if (f)
-       r = fprintf(f, "%d\n", getpid());
+       r = fprintf(f, "%ld\n", (long) getpid());
     return f && !fclose(f) && r >= 0;
 }
 
@@ -3122,7 +3186,7 @@ int main(int argc, char **argv) {
     char *configfile = NULL, *pidfile = NULL;
     struct clsrvconf *srvconf;
     int i;
-    
+
     debug_init("radsecproxy");
     debug_set_level(DEBUG_LEVEL);
 
@@ -3131,7 +3195,7 @@ int main(int argc, char **argv) {
 
     /* needed even if no TLS/DTLS transport */
     sslinit();
-    
+
     getargs(argc, argv, &foreground, &pretend, &loglevel, &configfile, &pidfile);
     if (loglevel)
        debug_set_level(loglevel);
@@ -3154,12 +3218,12 @@ int main(int argc, char **argv) {
 
     if (!foreground && (daemon(0, 0) < 0))
        debugx(1, DBG_ERR, "daemon() failed: %s", strerror(errno));
-    
+
     debug_timestamp_on();
-    debug(DBG_INFO, "radsecproxy revision $Rev$ starting");
+    debug(DBG_INFO, "radsecproxy revision %s starting", PACKAGE_VERSION);
     if (pidfile && !createpidfile(pidfile))
        debugx(1, DBG_ERR, "failed to create pidfile %s: %s", pidfile, strerror(errno));
-    
+
     sigemptyset(&sigset);
     /* exit on all but SIGHUP|SIGPIPE, ignore more? */
     sigaddset(&sigset, SIGHUP);
@@ -3186,8 +3250,12 @@ int main(int argc, char **argv) {
         if (find_clconf_type(i, NULL))
            createlisteners(i);
     }
-    
+
     /* just hang around doing nothing, anything to do here? */
     for (;;)
        sleep(1000);
 }
+
+/* Local Variables: */
+/* c-file-style: "stroustrup" */
+/* End: */