Alloc connections pools in the NULL ctx
[freeradius.git] / src / lib / misc.c
index a422e75..d4d87f4 100644 (file)
@@ -35,7 +35,6 @@ RCSID("$Id$")
        } while (0)
 
 #ifdef HAVE_PTHREAD_H
-static pthread_mutex_t autofree_context = PTHREAD_MUTEX_INITIALIZER;
 #  define PTHREAD_MUTEX_LOCK pthread_mutex_lock
 #  define PTHREAD_MUTEX_UNLOCK pthread_mutex_unlock
 #else
@@ -53,6 +52,11 @@ static char const *months[] = {
 
 fr_thread_local_setup(char *, fr_inet_ntop_buffer);    /* macro */
 
+typedef struct fr_talloc_link {
+       bool armed;
+       TALLOC_CTX *child;
+} fr_talloc_link_t;
+
 /** Sets a signal handler using sigaction if available, else signal
  *
  * @param sig to set handler for.
@@ -81,25 +85,50 @@ int fr_set_signal(int sig, sig_t func)
        return 0;
 }
 
-/** Allocates a new talloc context from the root autofree context
+static int _fr_trigger_talloc_ctx_free(fr_talloc_link_t *trigger)
+{
+       if (trigger->armed) talloc_free(trigger->child);
+
+       return 0;
+}
+
+static int _fr_disarm_talloc_ctx_free(bool **armed)
+{
+       **armed = false;
+       return 0;
+}
+
+/** Link a parent and a child context, so the child is freed before the parent
  *
- * This function is threadsafe, whereas using the NULL context is not.
+ * @note This is not thread safe. Do not free parent before threads are joined, do not call from a child thread.
+ * @note It's OK to free the child before threads are joined, but this will leak memory until the parent is freed.
  *
- * @note The returned context must be freed by the caller.
- * @returns a new talloc context parented by the root autofree context.
+ * @param parent who's fate the child should share.
+ * @param child bound to parent's lifecycle.
+ * @return 0 on success -1 on failure.
  */
-TALLOC_CTX *fr_autofree_ctx(void)
+int fr_link_talloc_ctx_free(TALLOC_CTX *parent, TALLOC_CTX *child)
 {
-       static TALLOC_CTX *ctx = NULL, *child;
-       PTHREAD_MUTEX_LOCK(&autofree_context);
-       if (!ctx) {
-               ctx = talloc_autofree_context();
+       fr_talloc_link_t *trigger;
+       bool **disarm;
+
+       trigger = talloc(parent, fr_talloc_link_t);
+       if (!trigger) return -1;
+
+       disarm = talloc(child, bool *);
+       if (!disarm) {
+               talloc_free(trigger);
+               return -1;
        }
 
-       child = talloc_new(ctx);
-       PTHREAD_MUTEX_UNLOCK(&autofree_context);
+       trigger->child = child;
+       trigger->armed = true;
+       *disarm = &trigger->armed;
 
-       return child;
+       talloc_set_destructor(trigger, _fr_trigger_talloc_ctx_free);
+       talloc_set_destructor(disarm, _fr_disarm_talloc_ctx_free);
+
+       return 0;
 }
 
 /*
@@ -174,6 +203,281 @@ char const *ip_ntoa(char *buffer, uint32_t ipaddr)
        return buffer;
 }
 
+/** Parse an IPv4 address or IPv4 prefix in presentation format (and others)
+ *
+ * @param out Where to write the ip address value.
+ * @param value to parse, may be dotted quad [+ prefix], or integer, or octal number, or '*' (INADDR_ANY).
+ * @param inlen Length of value, if value is \0 terminated inlen may be 0.
+ * @param resolve If true and value doesn't look like an IP address, try and resolve value as a hostname.
+ * @param fallback to IPv4 resolution if no A records can be found.
+ * @return 0 if ip address was parsed successfully, else -1 on error.
+ */
+int fr_pton4(fr_ipaddr_t *out, char const *value, size_t inlen, bool resolve, bool fallback)
+{
+       char *p;
+       unsigned int prefix;
+       char *eptr;
+
+       /* Dotted quad + / + [0-9]{1,2} */
+       char buffer[INET_ADDRSTRLEN + 3];
+
+       /*
+        *      Copy to intermediary buffer if we were given a length
+        */
+       if (inlen > 0) {
+               if (inlen >= sizeof(buffer)) {
+                       fr_strerror_printf("Invalid IPv4 address string \"%s\"", value);
+                       return -1;
+               }
+               memcpy(buffer, value, inlen);
+               buffer[inlen] = '\0';
+       }
+
+       p = strchr(value, '/');
+       /*
+        *      192.0.2.2 is parsed as if it was /32
+        */
+       if (!p) {
+               /*
+                *      Allow '*' as the wildcard address usually 0.0.0.0
+                */
+               if ((value[0] == '*') && (value[1] == '\0')) {
+                       out->ipaddr.ip4addr.s_addr = htonl(INADDR_ANY);
+               /*
+                *      Convert things which are obviously integers to IP addresses
+                *
+                *      We assume the number is the bigendian representation of the
+                *      IP address.
+                */
+               } else if (is_integer(value) || ((value[0] == '0') && (value[1] == 'x'))) {
+                       out->ipaddr.ip4addr.s_addr = htonl(strtoul(value, NULL, 0));
+               } else if (!resolve) {
+                       if (inet_pton(AF_INET, value, &(out->ipaddr.ip4addr.s_addr)) <= 0) {
+                               fr_strerror_printf("Failed to parse IPv4 address string \"%s\"", value);
+                               return -1;
+                       }
+               } else if (ip_hton(out, AF_INET, value, fallback) < 0) return -1;
+
+               out->prefix = 32;
+               out->af = AF_INET;
+
+               return 0;
+       }
+
+       /*
+        *      Otherwise parse the prefix
+        */
+       if ((size_t)(p - value) >= INET_ADDRSTRLEN) {
+               fr_strerror_printf("Invalid IPv4 address string \"%s\"", value);
+               return -1;
+       }
+
+       /*
+        *      Copy the IP portion into a temporary buffer if we haven't already.
+        */
+       if (inlen == 0) memcpy(buffer, value, p - value);
+       buffer[p - value] = '\0';
+
+       if (!resolve) {
+               if (inet_pton(AF_INET, buffer, &(out->ipaddr.ip4addr.s_addr)) <= 0) {
+                       fr_strerror_printf("Failed to parse IPv4 address string \"%s\"", value);
+                       return -1;
+               }
+       } else if (ip_hton(out, AF_INET, buffer, fallback) < 0) return -1;
+
+       prefix = strtoul(p + 1, &eptr, 10);
+       if (prefix > 32) {
+               fr_strerror_printf("Invalid IPv4 mask length \"%s\".  Should be between 0-32", p);
+               return -1;
+       }
+       if (eptr[0] != '\0') {
+               fr_strerror_printf("Failed to parse IPv4 address string \"%s\", "
+                                  "got garbage after mask length \"%s\"", value, eptr);
+               return -1;
+       }
+
+       if (prefix < 32) {
+               out->ipaddr.ip4addr = fr_inaddr_mask(&(out->ipaddr.ip4addr), prefix);
+       }
+
+       out->prefix = (uint8_t) prefix;
+       out->af = AF_INET;
+
+       return 0;
+}
+
+/** Parse an IPv6 address or IPv6 prefix in presentation format (and others)
+ *
+ * @param out Where to write the ip address value.
+ * @param value to parse.
+ * @param inlen Length of value, if value is \0 terminated inlen may be 0.
+ * @param resolve If true and value doesn't look like an IP address, try and resolve value as a hostname.
+ * @param fallback to IPv4 resolution if no AAAA records can be found.
+ * @return 0 if ip address was parsed successfully, else -1 on error.
+ */
+int fr_pton6(fr_ipaddr_t *out, char const *value, size_t inlen, bool resolve, bool fallback)
+{
+       char const *p;
+       unsigned int prefix;
+       char *eptr;
+
+       /* IPv6  + / + [0-9]{1,3} */
+       char buffer[INET6_ADDRSTRLEN + 4];
+
+       /*
+        *      Copy to intermediary buffer if we were given a length
+        */
+       if (inlen > 0) {
+               if (inlen >= sizeof(buffer)) {
+                       fr_strerror_printf("Invalid IPv6 address string \"%s\"", value);
+                       return -1;
+               }
+               memcpy(buffer, value, inlen);
+               buffer[inlen] = '\0';
+       }
+
+       p = strchr(value, '/');
+       if (!p) {
+               /*
+                *      Allow '*' as the wildcard address
+                */
+               if ((value[0] == '*') && (value[1] == '\0')) {
+                       memset(&out->ipaddr.ip6addr.s6_addr, 0, sizeof(out->ipaddr.ip6addr.s6_addr));
+               } else if (!resolve) {
+                       if (inet_pton(AF_INET6, value, &(out->ipaddr.ip6addr.s6_addr)) <= 0) {
+                               fr_strerror_printf("Failed to parse IPv6 address string \"%s\"", value);
+                               return -1;
+                       }
+               } else if (ip_hton(out, AF_INET6, value, fallback) < 0) return -1;
+
+               out->prefix = 128;
+               out->af = AF_INET6;
+
+               return 0;
+       }
+
+       if ((p - value) >= INET6_ADDRSTRLEN) {
+               fr_strerror_printf("Invalid IPv6 address string \"%s\"", value);
+               return -1;
+       }
+
+       /*
+        *      Copy string to temporary buffer if we didn't do it earlier
+        */
+       if (inlen == 0) memcpy(buffer, value, p - value);
+       buffer[p - value] = '\0';
+
+       if (!resolve) {
+               if (inet_pton(AF_INET6, buffer, &(out->ipaddr.ip6addr.s6_addr)) <= 0) {
+                       fr_strerror_printf("Failed to parse IPv6 address string \"%s\"", value);
+                       return -1;
+               }
+       } else if (ip_hton(out, AF_INET6, buffer, fallback) < 0) return -1;
+
+       prefix = strtoul(p + 1, &eptr, 10);
+       if (prefix > 128) {
+               fr_strerror_printf("Invalid IPv6 mask length \"%s\".  Should be between 0-128", p);
+               return -1;
+       }
+       if (eptr[0] != '\0') {
+               fr_strerror_printf("Failed to parse IPv6 address string \"%s\", "
+                                  "got garbage after mask length \"%s\"", value, eptr);
+               return -1;
+       }
+
+       if (prefix < 128) {
+               struct in6_addr addr;
+
+               addr = fr_in6addr_mask(&(out->ipaddr.ip6addr), prefix);
+               memcpy(&(out->ipaddr.ip6addr.s6_addr), &addr, sizeof(addr));
+       }
+
+       out->prefix = (uint8_t) prefix;
+       out->af = AF_INET6;
+
+       return 0;
+}
+
+/** Simple wrapper to decide whether an IP value is v4 or v6 and call the appropriate parser.
+ *
+ * @param out Where to write the ip address value.
+ * @param value to parse.
+ * @param inlen Length of value, if value is \0 terminated inlen may be 0.
+ * @param resolve If true and value doesn't look like an IP address, try and resolve value as a hostname.
+ * @return 0 if ip address was parsed successfully, else -1 on error.
+ */
+int fr_pton(fr_ipaddr_t *out, char const *value, size_t inlen, bool resolve)
+{
+       size_t len, i;
+
+       len = (inlen == 0) ? strlen(value) : inlen;
+       for (i = 0; i < len; i++) switch (value[i]) {
+       /*
+        *      Chars illegal in domain names and IPv4 addresses.
+        *      Must be v6 and cannot be a domain.
+        */
+       case ':':
+       case '[':
+       case ']':
+               return fr_pton6(out, value, inlen, false, false);
+
+       /*
+        *      Chars which don't really tell us anything
+        */
+       case '.':
+       case '/':
+               continue;
+
+       default:
+               /*
+                *      Outside the range of IPv4 chars, must be a domain
+                *      Use A record in preference to AAAA record.
+                */
+               if ((value[i] < '0') || (value[i] > '9')) {
+                       if (!resolve) return -1;
+                       return fr_pton4(out, value, inlen, true, true);
+               }
+               break;
+       }
+
+       /*
+        *      All chars were in the IPv4 set [0-9/.], must be an IPv4
+        *      address.
+        */
+       return fr_pton4(out, value, inlen, false, false);
+}
+
+/** Check if the IP address is equivalent to INADDR_ANY
+ *
+ * @param addr to chec.
+ * @return true if IP address matches INADDR_ANY or INADDR6_ANY (assumed to be 0), else false.
+ */
+bool is_wildcard(fr_ipaddr_t *addr)
+{
+       static struct in6_addr in6_addr;
+
+       switch (addr->af) {
+       case AF_INET:
+               return (addr->ipaddr.ip4addr.s_addr == htons(INADDR_ANY));
+
+       case AF_INET6:
+               return (memcmp(addr->ipaddr.ip6addr.s6_addr, in6_addr.s6_addr, sizeof(in6_addr.s6_addr)) == 0) ? true :false;
+
+       default:
+               fr_assert(0);
+               return false;
+       }
+}
+
+int fr_ntop(char *out, size_t outlen, fr_ipaddr_t *addr)
+{
+       char buffer[INET6_ADDRSTRLEN];
+
+       if (inet_ntop(addr->af, &(addr->ipaddr), buffer, sizeof(buffer)) == NULL) return -1;
+
+       return snprintf(out, outlen, "%s/%i", buffer, addr->prefix);
+}
+
 /*
  *     Internal wrapper for locking, to minimize the number of ifdef's
  *
@@ -519,44 +823,34 @@ char const *inet_ntop(int af, void const *src, char *dst, size_t cnt)
 }
 #endif
 
-
-/*
- *     Try to convert the address to v4 then v6
- */
-int ip_ptonx(char const *src, fr_ipaddr_t *dst)
-{
-       if (inet_pton(AF_INET, src, &dst->ipaddr.ip4addr) == 1) {
-               dst->af = AF_INET;
-               return 1;
-       }
-
-#ifdef HAVE_STRUCT_SOCKADDR_IN6
-       if (inet_pton(AF_INET6, src, &dst->ipaddr.ip6addr) == 1) {
-               dst->af = AF_INET6;
-               return 1;
-       }
-#endif
-
-       return 0;
-}
-
-/*
- *     Wrappers for IPv4/IPv6 host to IP address lookup.
- *     This API returns only one IP address, of the specified
- *     address family, or the first address (of whatever family),
- *     if AF_UNSPEC is used.
+/** Wrappers for IPv4/IPv6 host to IP address lookup
+ *
+ * This function returns only one IP address, of the specified address family,
+ * or the first address (of whatever family), if AF_UNSPEC is used.
+ *
+ * If fallback is specified and af is AF_INET, but no AF_INET records were
+ * found and a record for AF_INET6 exists that record will be returned.
+ *
+ * If fallback is specified and af is AF_INET6, and a record with AF_INET4 exists
+ * that record will be returned instead.
+ *
+ * @param out Where to write result.
+ * @param af To search for in preference.
+ * @param hostname to search for.
+ * @param fallback to the other adress family, if no records matching af, found.
+ * @return 0 on success, else -1 on failure.
  */
-int ip_hton(char const *src, int af, fr_ipaddr_t *dst)
+int ip_hton(fr_ipaddr_t *out, int af, char const *hostname, bool fallback)
 {
        int rcode;
-       struct addrinfo hints, *ai = NULL, *res = NULL;
+       struct addrinfo hints, *ai = NULL, *alt = NULL, *res = NULL;
 
        if (!fr_hostname_lookups) {
 #ifdef HAVE_STRUCT_SOCKADDR_IN6
                if (af == AF_UNSPEC) {
                        char const *p;
 
-                       for (p = src; *p != '\0'; p++) {
+                       for (p = hostname; *p != '\0'; p++) {
                                if ((*p == ':') ||
                                    (*p == '[') ||
                                    (*p == ']')) {
@@ -569,11 +863,11 @@ int ip_hton(char const *src, int af, fr_ipaddr_t *dst)
 
                if (af == AF_UNSPEC) af = AF_INET;
 
-               if (!inet_pton(af, src, &(dst->ipaddr))) {
+               if (!inet_pton(af, hostname, &(out->ipaddr))) {
                        return -1;
                }
 
-               dst->af = af;
+               out->af = af;
                return 0;
        }
 
@@ -589,30 +883,31 @@ int ip_hton(char const *src, int af, fr_ipaddr_t *dst)
                /*
                 *      If it's all numeric, avoid getaddrinfo()
                 */
-               if (inet_pton(af, src, &dst->ipaddr.ip4addr) == 1) {
+               if (inet_pton(af, hostname, &out->ipaddr.ip4addr) == 1) {
                        return 0;
                }
        }
 #endif
 
-       if ((rcode = getaddrinfo(src, NULL, &hints, &res)) != 0) {
+       if ((rcode = getaddrinfo(hostname, NULL, &hints, &res)) != 0) {
                fr_strerror_printf("ip_hton: %s", gai_strerror(rcode));
                return -1;
        }
 
        for (ai = res; ai; ai = ai->ai_next) {
-               if ((af == ai->ai_family) || (af == AF_UNSPEC))
-                       break;
+               if ((af == ai->ai_family) || (af == AF_UNSPEC)) break;
+               if (!alt && fallback && ((ai->ai_family == AF_INET) || (ai->ai_family == AF_INET6))) alt = ai;
        }
 
+       if (!ai) ai = alt;
        if (!ai) {
-               fr_strerror_printf("ip_hton failed to find requested information for host %.100s", src);
+               fr_strerror_printf("ip_hton failed to find requested information for host %.100s", hostname);
                freeaddrinfo(ai);
                return -1;
        }
 
        rcode = fr_sockaddr2ipaddr((struct sockaddr_storage *)ai->ai_addr,
-                                  ai->ai_addrlen, dst, NULL);
+                                  ai->ai_addrlen, out, NULL);
        freeaddrinfo(ai);
        if (!rcode) return -1;
 
@@ -653,7 +948,7 @@ char const *ip_ntoh(fr_ipaddr_t const *src, char *dst, size_t cnt)
  * @param prefix Number of contiguous bits to mask.
  * @return an ipv6 address with the host portion zeroed out.
  */
-struct in_addr fr_ipaddr_mask(struct in_addr const *ipaddr, uint8_t prefix)
+struct in_addr fr_inaddr_mask(struct in_addr const *ipaddr, uint8_t prefix)
 {
        uint32_t ret;
 
@@ -666,8 +961,8 @@ struct in_addr fr_ipaddr_mask(struct in_addr const *ipaddr, uint8_t prefix)
                return *ipaddr;
        }
 
-       ret = htonl(~((0x00000001UL << (32 - prefix)) - 1)) & *((uint32_t const *)ipaddr);
-       return *(struct in_addr *) &ret;
+       ret = htonl(~((0x00000001UL << (32 - prefix)) - 1)) & ipaddr->s_addr;
+       return (*(struct in_addr *)&ret);
 }
 
 /** Mask off a portion of an IPv6 address
@@ -676,7 +971,7 @@ struct in_addr fr_ipaddr_mask(struct in_addr const *ipaddr, uint8_t prefix)
  * @param prefix Number of contiguous bits to mask.
  * @return an ipv6 address with the host portion zeroed out.
  */
-struct in6_addr fr_ipaddr_mask6(struct in6_addr const *ipaddr, uint8_t prefix)
+struct in6_addr fr_in6addr_mask(struct in6_addr const *ipaddr, uint8_t prefix)
 {
        uint64_t const *p = (uint64_t const *) ipaddr;
        uint64_t ret[2], *o = ret;
@@ -702,25 +997,57 @@ struct in6_addr fr_ipaddr_mask6(struct in6_addr const *ipaddr, uint8_t prefix)
        return *(struct in6_addr *) &ret;
 }
 
-static char const *hextab = "0123456789abcdef";
+/** Zeroes out the host portion of an fr_ipaddr_t
+ *
+ * @param[in,out] addr to mask
+ * @param[in] prefix Length of the network portion.
+ */
+void fr_ipaddr_mask(fr_ipaddr_t *addr, uint8_t prefix)
+{
+
+       switch (addr->af) {
+       case AF_INET:
+               addr->ipaddr.ip4addr = fr_inaddr_mask(&addr->ipaddr.ip4addr, prefix);
+               break;
+
+       case AF_INET6:
+               addr->ipaddr.ip6addr = fr_in6addr_mask(&addr->ipaddr.ip6addr, prefix);
+               break;
+
+       default:
+               return;
+       }
+       addr->prefix = prefix;
+}
+
+static char const hextab[] = "0123456789abcdef";
 
 /** Convert hex strings to binary data
  *
  * @param bin Buffer to write output to.
- * @param hex input string.
  * @param outlen length of output buffer (or length of input string / 2).
+ * @param hex input string.
+ * @param inlen length of the input string
  * @return length of data written to buffer.
  */
-size_t fr_hex2bin(uint8_t *bin, char const *hex, size_t outlen)
+size_t fr_hex2bin(uint8_t *bin, size_t outlen, char const *hex, size_t inlen)
 {
        size_t i;
+       size_t len;
        char *c1, *c2;
 
-       for (i = 0; i < outlen; i++) {
-               if(!(c1 = memchr(hextab, tolower((int) hex[i << 1]), 16)) ||
-                  !(c2 = memchr(hextab, tolower((int) hex[(i << 1) + 1]), 16)))
+       /*
+        *      Smartly truncate output, caller should check number of bytes
+        *      written.
+        */
+       len = inlen >> 1;
+       if (len > outlen) len = outlen;
+
+       for (i = 0; i < len; i++) {
+               if(!(c1 = memchr(hextab, tolower((int) hex[i << 1]), sizeof(hextab))) ||
+                  !(c2 = memchr(hextab, tolower((int) hex[(i << 1) + 1]), sizeof(hextab))))
                        break;
-                bin[i] = ((c1-hextab)<<4) + (c2-hextab);
+               bin[i] = ((c1-hextab)<<4) + (c2-hextab);
        }
 
        return i;
@@ -774,13 +1101,11 @@ uint32_t fr_strtoul(char const *value, char **end)
  *
  * @return true if the entirety of the string is whitespace, else false.
  */
-bool fr_whitespace_check(char const *value)
+bool is_whitespace(char const *value)
 {
-       while (*value) {
+       do {
                if (!isspace(*value)) return false;
-
-               value++;
-       }
+       } while (*++value);
 
        return true;
 }
@@ -789,13 +1114,24 @@ bool fr_whitespace_check(char const *value)
  *
  * @return true if the entirety of the string is are numebrs, else false.
  */
-bool fr_integer_check(char const *value)
+bool is_integer(char const *value)
 {
-       while (*value) {
+       do {
                if (!isdigit(*value)) return false;
+       } while (*++value);
 
-               value++;
-       }
+       return true;
+}
+
+/** Check whether the string is allzeros
+ *
+ * @return true if the entirety of the string is are numebrs, else false.
+ */
+bool is_zero(char const *value)
+{
+       do {
+               if (*value != '0') return false;
+       } while (*++value);
 
        return true;
 }
@@ -861,7 +1197,7 @@ int fr_ipaddr_cmp(fr_ipaddr_t const *a, fr_ipaddr_t const *b)
        return -1;
 }
 
-int fr_ipaddr2sockaddr(fr_ipaddr_t const *ipaddr, int port,
+int fr_ipaddr2sockaddr(fr_ipaddr_t const *ipaddr, uint16_t port,
                       struct sockaddr_storage *sa, socklen_t *salen)
 {
        if (ipaddr->af == AF_INET) {
@@ -899,7 +1235,7 @@ int fr_ipaddr2sockaddr(fr_ipaddr_t const *ipaddr, int port,
 
 
 int fr_sockaddr2ipaddr(struct sockaddr_storage const *sa, socklen_t salen,
-                      fr_ipaddr_t *ipaddr, int *port)
+                      fr_ipaddr_t *ipaddr, uint16_t *port)
 {
        if (sa->ss_family == AF_INET) {
                struct sockaddr_in      s4;
@@ -911,6 +1247,7 @@ int fr_sockaddr2ipaddr(struct sockaddr_storage const *sa, socklen_t salen,
 
                memcpy(&s4, sa, sizeof(s4));
                ipaddr->af = AF_INET;
+               ipaddr->prefix = 32;
                ipaddr->ipaddr.ip4addr = s4.sin_addr;
                if (port) *port = ntohs(s4.sin_port);
 
@@ -925,6 +1262,7 @@ int fr_sockaddr2ipaddr(struct sockaddr_storage const *sa, socklen_t salen,
 
                memcpy(&s6, sa, sizeof(s6));
                ipaddr->af = AF_INET6;
+               ipaddr->prefix = 128;
                ipaddr->ipaddr.ip6addr = s6.sin6_addr;
                if (port) *port = ntohs(s6.sin6_port);
                ipaddr->scope = s6.sin6_scope_id;
@@ -1168,7 +1506,7 @@ int fr_get_time(char const *date_str, time_t *date)
        char            buf[64];
        char            *p;
        char            *f[4];
-       char            *tail = '\0';
+       char            *tail = NULL;
 
        /*
         * Test for unix timestamp date
@@ -1293,6 +1631,62 @@ int fr_get_time(char const *date_str, time_t *date)
        return 0;
 }
 
+/** Compares two pointers
+ *
+ * @param a first pointer to compare.
+ * @param b second pointer to compare.
+ * @return -1 if a < b, +1 if b > a, or 0 if both equal.
+ */
+int8_t fr_pointer_cmp(void const *a, void const *b)
+{
+       if (a < b) return -1;
+       if (a == b) return 0;
+
+       return 1;
+}
+
+static int _quick_partition(void const *to_sort[], int min, int max, fr_cmp_t cmp) {
+       void const *pivot = to_sort[min];
+       int i = min;
+       int j = max + 1;
+       void const *tmp;
+
+       for (;;) {
+               do ++i; while((cmp(to_sort[i], pivot) <= 0) && i <= max);
+               do --j; while(cmp(to_sort[j], pivot) > 0);
+
+               if (i >= j) break;
+
+               tmp = to_sort[i];
+               to_sort[i] = to_sort[j];
+               to_sort[j] = tmp;
+       }
+
+       tmp = to_sort[min];
+       to_sort[min] = to_sort[j];
+       to_sort[j] = tmp;
+
+       return j;
+}
+
+/** Quick sort an array of pointers using a comparator
+ *
+ * @param to_sort array of pointers to sort.
+ * @param min_idx the lowest index (usually 0).
+ * @param max_idx the highest index (usually length of array - 1).
+ * @param cmp the comparison function to use to sort the array elements.
+ */
+void fr_quick_sort(void const *to_sort[], int min_idx, int max_idx, fr_cmp_t cmp)
+{
+       int part;
+
+       if (min_idx >= max_idx) return;
+
+       part = _quick_partition(to_sort, min_idx, max_idx, cmp);
+       fr_quick_sort(to_sort, min_idx, part - 1, cmp);
+       fr_quick_sort(to_sort, part + 1, max_idx, cmp);
+}
+
 #ifdef TALLOC_DEBUG
 void fr_talloc_verify_cb(UNUSED const void *ptr, UNUSED int depth,
                         UNUSED int max_depth, UNUSED int is_ref,