* Holy war on strlen: use IsEmptyStr where apropriate.
[citadel.git] / citadel / internet_addressing.c
index b96aec1dcbca7985735ac23ec0967da783550364..c4bfc9b51210b9ceeccec77037f3f304bc680883 100644 (file)
@@ -5,10 +5,6 @@
  * to users on the Citadel system.
  */
 
-#ifdef DLL_EXPORT
-#define IN_LIBCIT
-#endif
-
 #include "sysdep.h"
 #include <stdlib.h>
 #include <unistd.h>
@@ -36,7 +32,6 @@
 #include <limits.h>
 #include "citadel.h"
 #include "server.h"
-#include "dynloader.h"
 #include "sysdep_decls.h"
 #include "citserver.h"
 #include "support.h"
@@ -47,6 +42,7 @@
 #include "user_ops.h"
 #include "room_ops.h"
 #include "parsedate.h"
+#include "database.h"
 
 
 #ifndef HAVE_SNPRINTF
@@ -60,7 +56,7 @@ struct trynamebuf {
 };
 
 char *inetcfg = NULL;
-
+struct spamstrings_t *spamstrings = NULL;
 
 
 /*
@@ -69,18 +65,21 @@ char *inetcfg = NULL;
 int CtdlHostAlias(char *fqdn) {
        int config_lines;
        int i;
-       char buf[SIZ];
-       char host[SIZ], type[SIZ];
+       char buf[256];
+       char host[256], type[256];
 
+       if (fqdn == NULL) return(hostalias_nomatch);
+       if (IsEmptyStr(fqdn)) return(hostalias_nomatch);
+       if (!strcasecmp(fqdn, "localhost")) return(hostalias_localhost);
        if (!strcasecmp(fqdn, config.c_fqdn)) return(hostalias_localhost);
        if (!strcasecmp(fqdn, config.c_nodename)) return(hostalias_localhost);
        if (inetcfg == NULL) return(hostalias_nomatch);
 
        config_lines = num_tokens(inetcfg, '\n');
        for (i=0; i<config_lines; ++i) {
-               extract_token(buf, inetcfg, i, '\n');
-               extract_token(host, buf, 0, '|');
-               extract_token(type, buf, 1, '|');
+               extract_token(buf, inetcfg, i, '\n', sizeof buf);
+               extract_token(host, buf, 0, '|', sizeof host);
+               extract_token(type, buf, 1, '|', sizeof type);
 
                if ( (!strcasecmp(type, "localhost"))
                   && (!strcasecmp(fqdn, host)))
@@ -90,6 +89,14 @@ int CtdlHostAlias(char *fqdn) {
                   && (!strcasecmp(&fqdn[strlen(fqdn)-strlen(host)], host)))
                        return(hostalias_gatewaydomain);
 
+               if ( (!strcasecmp(type, "directory"))
+                  && (!strcasecmp(&fqdn[strlen(fqdn)-strlen(host)], host)))
+                       return(hostalias_directory);
+
+               if ( (!strcasecmp(type, "masqdomain"))
+                  && (!strcasecmp(&fqdn[strlen(fqdn)-strlen(host)], host)))
+                       return(hostalias_masq);
+
        }
 
        return(hostalias_nomatch);
@@ -106,7 +113,7 @@ int CtdlHostAlias(char *fqdn) {
  *
  * FIXME ... this needs to be updated to handle aliases.
  */
-int fuzzy_match(struct usersupp *us, char *matchstring) {
+int fuzzy_match(struct ctdluser *us, char *matchstring) {
        int a;
 
        if ( (!strncasecmp(matchstring, "cit", 3)) 
@@ -152,7 +159,7 @@ void unfold_rfc822_field(char *field) {
  * Split an RFC822-style address into userid, host, and full name
  *
  */
-void process_rfc822_addr(char *rfc822, char *user, char *node, char *name)
+void process_rfc822_addr(const char *rfc822, char *user, char *node, char *name)
 {
        int a;
 
@@ -160,12 +167,14 @@ void process_rfc822_addr(char *rfc822, char *user, char *node, char *name)
        strcpy(node, config.c_fqdn);
        strcpy(name, "");
 
+       if (rfc822 == NULL) return;
+
        /* extract full name - first, it's From minus <userid> */
        strcpy(name, rfc822);
        stripout(name, '<', '>');
 
        /* strip anything to the left of a bang */
-       while ((strlen(name) > 0) && (haschar(name, '!') > 0))
+       while ((!IsEmptyStr(name)) && (haschar(name, '!') > 0))
                strcpy(name, &name[1]);
 
        /* and anything to the right of a @ or % */
@@ -185,7 +194,7 @@ void process_rfc822_addr(char *rfc822, char *user, char *node, char *name)
        /* but if there are a set of quotes, that supersedes everything */
        if (haschar(rfc822, 34) == 2) {
                strcpy(name, rfc822);
-               while ((strlen(name) > 0) && (name[0] != 34)) {
+               while ((!IsEmptyStr(name)) && (name[0] != 34)) {
                        strcpy(&name[0], &name[1]);
                }
                strcpy(&name[0], &name[1]);
@@ -205,7 +214,7 @@ void process_rfc822_addr(char *rfc822, char *user, char *node, char *name)
        }
 
        /* strip anything to the left of a bang */
-       while ((strlen(user) > 0) && (haschar(user, '!') > 0))
+       while ((!IsEmptyStr(user)) && (haschar(user, '!') > 0))
                strcpy(user, &user[1]);
 
        /* and anything to the right of a @ or % */
@@ -240,15 +249,15 @@ void process_rfc822_addr(char *rfc822, char *user, char *node, char *name)
        else {
 
                /* strip anything to the left of a @ */
-               while ((strlen(node) > 0) && (haschar(node, '@') > 0))
+               while ((!IsEmptyStr(node)) && (haschar(node, '@') > 0))
                        strcpy(node, &node[1]);
        
                /* strip anything to the left of a % */
-               while ((strlen(node) > 0) && (haschar(node, '%') > 0))
+               while ((!IsEmptyStr(node)) && (haschar(node, '%') > 0))
                        strcpy(node, &node[1]);
        
                /* reduce multiple system bang paths to node!user */
-               while ((strlen(node) > 0) && (haschar(node, '!') > 1))
+               while ((!IsEmptyStr(node)) && (haschar(node, '!') > 1))
                        strcpy(node, &node[1]);
        
                /* now get rid of the user portion of a node!user string */
@@ -261,122 +270,18 @@ void process_rfc822_addr(char *rfc822, char *user, char *node, char *name)
        striplt(user);
        striplt(node);
        striplt(name);
-}
-
-
 
-/*
- * Back end for convert_internet_address()
- * (Compares an internet name [buffer1] and stores in [buffer2] if found)
- */
-void try_name(struct usersupp *us, void *data) {
-       struct passwd *pw;
-       struct trynamebuf *tnb;
-       tnb = (struct trynamebuf *)data;
-
-       if (!strncasecmp(tnb->buffer1, "cit", 3))
-               if (atol(&tnb->buffer1[3]) == us->usernum)
-                       strcpy(tnb->buffer2, us->fullname);
-
-       if (!collapsed_strcmp(tnb->buffer1, us->fullname)) 
-                       strcpy(tnb->buffer2, us->fullname);
-
-       if (us->uid != BBSUID) {
-               pw = getpwuid(us->uid);
-               if (pw != NULL) {
-                       if (!strcasecmp(tnb->buffer1, pw->pw_name)) {
-                               strcpy(tnb->buffer2, us->fullname);
-                       }
-               }
-       }
-}
-
-
-/*
- * Convert an Internet email address to a Citadel user/host combination
- */
-int convert_internet_address(char *destuser, char *desthost, char *source)
-{
-       char user[SIZ];
-       char node[SIZ];
-       char name[SIZ];
-       struct quickroom qrbuf;
-       int i;
-       int hostalias;
-       struct trynamebuf tnb;
-       char buf[SIZ];
-       int passes = 0;
-       char sourcealias[1024];
-
-       safestrncpy(sourcealias, source, sizeof(sourcealias) );
-       alias(sourcealias);
-
-REALIAS:
-       /* Split it up */
-       process_rfc822_addr(sourcealias, user, node, name);
-       lprintf(9, "process_rfc822_addr() converted to <%s@%s> (%s)\n",
-               user, node, name);
-
-       /* Map the FQDN to a Citadel node name
+       /* If we processed a string that had the address in angle brackets
+        * but no name outside the brackets, we now have an empty name.  In
+        * this case, use the user portion of the address as the name.
         */
-       hostalias =  CtdlHostAlias(node);
-       switch(hostalias) {
-               case hostalias_localhost:
-                       strcpy(node, config.c_nodename);
-                       break;
-
-               case hostalias_gatewaydomain:
-                       extract_token(buf, node, 0, '.');
-                       safestrncpy(node, buf, sizeof buf);
+       if ((IsEmptyStr(name)) && (!IsEmptyStr(user))) {
+               strcpy(name, user);
        }
-
-       /* Now try to resolve the name
-        * FIXME ... do the multiple-addresses thing
-        */
-       if (!strcasecmp(node, config.c_nodename)) {
-
-
-               /* First, see if we hit an alias.  Don't do this more than
-                * a few times, in case we accidentally hit an alias loop
-                */
-               strcpy(sourcealias, user);
-               alias(user);
-               if ( (strcasecmp(user, sourcealias)) && (++passes < 3) )
-                       goto REALIAS;
-
-               /* Try all local rooms */
-               if (!strncasecmp(user, "room_", 5)) {
-                       strcpy(name, &user[5]);
-                       for (i=0; i<strlen(name); ++i) 
-                               if (name[i]=='_') name[i]=' ';
-                       if (getroom(&qrbuf, name) == 0) {
-                               strcpy(destuser, qrbuf.QRname);
-                               strcpy(desthost, config.c_nodename);
-                               return rfc822_room_delivery;
-                       }
-               }
-
-               /* Try all local users */
-               strcpy(destuser, user);
-               strcpy(desthost, config.c_nodename);
-               strcpy(tnb.buffer1, user);
-               strcpy(tnb.buffer2, "");
-               ForEachUser(try_name, &tnb);
-               if (strlen(tnb.buffer2) == 0) return(rfc822_no_such_user);
-               strcpy(destuser, tnb.buffer2);
-               return(rfc822_address_locally_validated);
-       }
-
-       strcpy(destuser, user);
-       strcpy(desthost, node);
-       if (hostalias == hostalias_gatewaydomain)
-               return(rfc822_address_on_citadel_network);
-       return(rfc822_address_nonlocal);
 }
 
 
 
-
 /*
  * convert_field() is a helper function for convert_internet_message().
  * Given start/end positions for an rfc822 field, it converts it to a Citadel
@@ -406,7 +311,7 @@ int convert_field(struct CtdlMessage *msg, int beg, int end) {
 
        if (colonpos < 0) return(0);    /* no colon? not a valid header line */
 
-       key = mallok((end - beg) + 2);
+       key = malloc((end - beg) + 2);
        safestrncpy(key, &rfc822[beg], (end-beg)+1);
        key[colonpos - beg] = 0;
        value = &key[(colonpos - beg) + 1];
@@ -424,35 +329,47 @@ int convert_field(struct CtdlMessage *msg, int beg, int end) {
                if (parsed_date < 0L) parsed_date = time(NULL);
                snprintf(buf, sizeof buf, "%ld", (long)parsed_date );
                if (msg->cm_fields['T'] == NULL)
-                       msg->cm_fields['T'] = strdoop(buf);
+                       msg->cm_fields['T'] = strdup(buf);
                processed = 1;
        }
 
        else if (!strcasecmp(key, "From")) {
                process_rfc822_addr(value, user, node, name);
-               lprintf(9, "Converted to <%s@%s> (%s)\n", user, node, name);
+               lprintf(CTDL_DEBUG, "Converted to <%s@%s> (%s)\n", user, node, name);
                snprintf(addr, sizeof addr, "%s@%s", user, node);
                if (msg->cm_fields['A'] == NULL)
-                       msg->cm_fields['A'] = strdoop(name);
+                       msg->cm_fields['A'] = strdup(name);
                processed = 1;
                if (msg->cm_fields['F'] == NULL)
-                       msg->cm_fields['F'] = strdoop(addr);
+                       msg->cm_fields['F'] = strdup(addr);
                processed = 1;
        }
 
        else if (!strcasecmp(key, "Subject")) {
                if (msg->cm_fields['U'] == NULL)
-                       msg->cm_fields['U'] = strdoop(value);
+                       msg->cm_fields['U'] = strdup(value);
+               processed = 1;
+       }
+
+       else if (!strcasecmp(key, "To")) {
+               if (msg->cm_fields['R'] == NULL)
+                       msg->cm_fields['R'] = strdup(value);
+               processed = 1;
+       }
+
+       else if (!strcasecmp(key, "CC")) {
+               if (msg->cm_fields['Y'] == NULL)
+                       msg->cm_fields['Y'] = strdup(value);
                processed = 1;
        }
 
        else if (!strcasecmp(key, "Message-ID")) {
                if (msg->cm_fields['I'] != NULL) {
-                       lprintf(5, "duplicate message id\n");
+                       lprintf(CTDL_WARNING, "duplicate message id\n");
                }
 
                if (msg->cm_fields['I'] == NULL) {
-                       msg->cm_fields['I'] = strdoop(value);
+                       msg->cm_fields['I'] = strdup(value);
 
                        /* Strip angle brackets */
                        while (haschar(msg->cm_fields['I'], '<') > 0) {
@@ -467,8 +384,20 @@ int convert_field(struct CtdlMessage *msg, int beg, int end) {
                processed = 1;
        }
 
+       else if (!strcasecmp(key, "Return-Path")) {
+               if (msg->cm_fields['P'] == NULL)
+                       msg->cm_fields['P'] = strdup(value);
+               processed = 1;
+       }
+
+       else if (!strcasecmp(key, "Envelope-To")) {
+               if (msg->cm_fields['V'] == NULL)
+                       msg->cm_fields['V'] = strdup(value);
+               processed = 1;
+       }
+
        /* Clean up and move on. */
-       phree(key);     /* Don't free 'value', it's actually the same buffer */
+       free(key);      /* Don't free 'value', it's actually the same buffer */
        return(processed);
 }
 
@@ -488,7 +417,7 @@ struct CtdlMessage *convert_internet_message(char *rfc822) {
        char buf[SIZ];
        int converted;
 
-       msg = mallok(sizeof(struct CtdlMessage));
+       msg = malloc(sizeof(struct CtdlMessage));
        if (msg == NULL) return msg;
 
        memset(msg, 0, sizeof(struct CtdlMessage));
@@ -497,7 +426,6 @@ struct CtdlMessage *convert_internet_message(char *rfc822) {
        msg->cm_format_type = FMT_RFC822;       /* internet message */
        msg->cm_fields['M'] = rfc822;
 
-       lprintf(9, "Unconverted RFC822 message length = %ld\n", (long)strlen(rfc822));
        pos = 0;
        done = 0;
 
@@ -518,8 +446,7 @@ struct CtdlMessage *convert_internet_message(char *rfc822) {
                        }
 
                        /* done with headers? */
-                       if (   ((rfc822[pos]=='\n')
-                             ||(rfc822[pos]=='\r') )
+                       if (   (rfc822[pos]=='\n')
                           && ( (rfc822[pos+1]=='\n')
                              ||(rfc822[pos+1]=='\r')) ) {
                                end = pos;
@@ -553,11 +480,9 @@ struct CtdlMessage *convert_internet_message(char *rfc822) {
        /* If there's no timestamp on this message, set it to now. */
        if (msg->cm_fields['T'] == NULL) {
                snprintf(buf, sizeof buf, "%ld", (long)time(NULL));
-               msg->cm_fields['T'] = strdoop(buf);
+               msg->cm_fields['T'] = strdup(buf);
        }
 
-       lprintf(9, "RFC822 length remaining after conversion = %ld\n",
-               (long)strlen(rfc822));
        return msg;
 }
 
@@ -571,60 +496,230 @@ struct CtdlMessage *convert_internet_message(char *rfc822) {
  * field is not present, or anything else goes wrong, it returns NULL.
  */
 char *rfc822_fetch_field(char *rfc822, char *fieldname) {
-       int pos = 0;
-       int beg, end;
-       int done = 0;
-       int colonpos, i;
        char *fieldbuf = NULL;
+       char *end_of_headers;
+       char *field_start;
+       char *ptr;
+       char *cont;
+       char fieldhdr[SIZ];
 
        /* Should never happen, but sometimes we get stupid */
        if (rfc822 == NULL) return(NULL);
        if (fieldname == NULL) return(NULL);
 
-       while (!done) {
+       snprintf(fieldhdr, sizeof fieldhdr, "%s:", fieldname);
 
-               /* Locate beginning and end of field, keeping in mind that
-                * some fields might be multiline
-                */
-               beg = pos;
-               end = (-1);
-               for (pos=beg; ((pos<=strlen(rfc822))&&(end<0)); ++pos) {
-                       if ((rfc822[pos]=='\n')
-                          && (!isspace(rfc822[pos+1]))) {
-                               end = pos;
-                       }
-                       if ( (rfc822[pos]=='\n')        /* done w. headers? */
-                          && ( (rfc822[pos+1]=='\n')
-                             ||(rfc822[pos+1]=='\r'))) {
-                               end = pos;
-                               done = 1;
-                       }
+       /* Locate the end of the headers, so we don't run past that point */
+       end_of_headers = bmstrcasestr(rfc822, "\n\r\n");
+       if (end_of_headers == NULL) {
+               end_of_headers = bmstrcasestr(rfc822, "\n\n");
+       }
+       if (end_of_headers == NULL) return (NULL);
+
+       field_start = bmstrcasestr(rfc822, fieldhdr);
+       if (field_start == NULL) return(NULL);
+       if (field_start > end_of_headers) return(NULL);
+
+       fieldbuf = malloc(SIZ);
+       strcpy(fieldbuf, "");
+
+       ptr = field_start;
+       ptr = memreadline(ptr, fieldbuf, SIZ-strlen(fieldbuf) );
+       while ( (isspace(ptr[0])) && (ptr < end_of_headers) ) {
+               strcat(fieldbuf, " ");
+               cont = &fieldbuf[strlen(fieldbuf)];
+               ptr = memreadline(ptr, cont, SIZ-strlen(fieldbuf) );
+               striplt(cont);
+       }
+
+       strcpy(fieldbuf, &fieldbuf[strlen(fieldhdr)]);
+       striplt(fieldbuf);
+
+       return(fieldbuf);
+}
 
+
+
+/*****************************************************************************
+ *                      DIRECTORY MANAGEMENT FUNCTIONS                       *
+ *****************************************************************************/
+
+/*
+ * Generate the index key for an Internet e-mail address to be looked up
+ * in the database.
+ */
+void directory_key(char *key, char *addr) {
+       int i;
+       int keylen = 0;
+
+       for (i=0; i<strlen(addr); ++i) {
+               if (!isspace(addr[i])) {
+                       key[keylen++] = tolower(addr[i]);
                }
+       }
+       key[keylen++] = 0;
 
-               /* At this point we have a field.  Is it The One? */
-               if (end > beg) {
-                       fieldbuf = mallok((end-beg)+3);
-                       if (fieldbuf == NULL) return(NULL);
-                       safestrncpy(fieldbuf, &rfc822[beg], (end-beg)+1);
-                       unfold_rfc822_field(fieldbuf);
-                       colonpos = (-1);
-                       for (i = strlen(fieldbuf); i >= 0; --i) {
-                               if (fieldbuf[i] == ':') colonpos = i;
-                       }
-                       if (colonpos > 0) {
-                               fieldbuf[colonpos] = 0;
-                               if (!strcasecmp(fieldbuf, fieldname)) {
-                                       strcpy(fieldbuf, &fieldbuf[colonpos+1]);
-                                       striplt(fieldbuf);
-                                       return(fieldbuf);
+       lprintf(CTDL_DEBUG, "Directory key is <%s>\n", key);
+}
+
+
+
+/* Return nonzero if the supplied address is in a domain we keep in
+ * the directory
+ */
+int IsDirectory(char *addr, int allow_masq_domains) {
+       char domain[256];
+       int h;
+
+       extract_token(domain, addr, 1, '@', sizeof domain);
+       striplt(domain);
+
+       h = CtdlHostAlias(domain);
+
+       if ( (h == hostalias_masq) && allow_masq_domains)
+               return(1);
+       
+       if ( (h == hostalias_localhost) || (h == hostalias_directory) ) {
+               return(1);
+       }
+       else {
+               return(0);
+       }
+}
+
+
+/*
+ * Initialize the directory database (erasing anything already there)
+ */
+void CtdlDirectoryInit(void) {
+       cdb_trunc(CDB_DIRECTORY);
+}
+
+
+/*
+ * Add an Internet e-mail address to the directory for a user
+ */
+void CtdlDirectoryAddUser(char *internet_addr, char *citadel_addr) {
+       char key[SIZ];
+
+       lprintf(CTDL_DEBUG, "Dir: %s --> %s\n",
+               internet_addr, citadel_addr);
+       if (IsDirectory(internet_addr, 0) == 0) return;
+
+       directory_key(key, internet_addr);
+
+       cdb_store(CDB_DIRECTORY, key, strlen(key),
+               citadel_addr, strlen(citadel_addr)+1 );
+}
+
+
+/*
+ * Delete an Internet e-mail address from the directory.
+ *
+ * (NOTE: we don't actually use or need the citadel_addr variable; it's merely
+ * here because the callback API expects to be able to send it.)
+ */
+void CtdlDirectoryDelUser(char *internet_addr, char *citadel_addr) {
+       char key[SIZ];
+
+       directory_key(key, internet_addr);
+       cdb_delete(CDB_DIRECTORY, key, strlen(key) );
+}
+
+
+/*
+ * Look up an Internet e-mail address in the directory.
+ * On success: returns 0, and Citadel address stored in 'target'
+ * On failure: returns nonzero
+ */
+int CtdlDirectoryLookup(char *target, char *internet_addr, size_t targbuflen) {
+       struct cdbdata *cdbrec;
+       char key[SIZ];
+
+       /* Dump it in there unchanged, just for kicks */
+       safestrncpy(target, internet_addr, targbuflen);
+
+       /* Only do lookups for addresses with hostnames in them */
+       if (num_tokens(internet_addr, '@') != 2) return(-1);
+
+       /* Only do lookups for domains in the directory */
+       if (IsDirectory(internet_addr, 0) == 0) return(-1);
+
+       directory_key(key, internet_addr);
+       cdbrec = cdb_fetch(CDB_DIRECTORY, key, strlen(key) );
+       if (cdbrec != NULL) {
+               safestrncpy(target, cdbrec->ptr, targbuflen);
+               cdb_free(cdbrec);
+               return(0);
+       }
+
+       return(-1);
+}
+
+
+/*
+ * Harvest any email addresses that someone might want to have in their
+ * "collected addresses" book.
+ */
+char *harvest_collected_addresses(struct CtdlMessage *msg) {
+       char *coll = NULL;
+       char addr[256];
+       char user[256], node[256], name[256];
+       int is_harvestable;
+       int i, j, h;
+       int field = 0;
+
+       if (msg == NULL) return(NULL);
+
+       is_harvestable = 1;
+       strcpy(addr, "");       
+       if (msg->cm_fields['A'] != NULL) {
+               strcat(addr, msg->cm_fields['A']);
+       }
+       if (msg->cm_fields['F'] != NULL) {
+               strcat(addr, " <");
+               strcat(addr, msg->cm_fields['F']);
+               strcat(addr, ">");
+               if (IsDirectory(msg->cm_fields['F'], 0)) {
+                       is_harvestable = 0;
+               }
+       }
+
+       if (is_harvestable) {
+               coll = strdup(addr);
+       }
+       else {
+               coll = strdup("");
+       }
+
+       if (coll == NULL) return(NULL);
+
+       /* Scan both the R (To) and Y (CC) fields */
+       for (i = 0; i < 2; ++i) {
+               if (i == 0) field = 'R' ;
+               if (i == 1) field = 'Y' ;
+
+               if (msg->cm_fields[field] != NULL) {
+                       for (j=0; j<num_tokens(msg->cm_fields[field], ','); ++j) {
+                               extract_token(addr, msg->cm_fields[field], j, ',', sizeof addr);
+                               process_rfc822_addr(addr, user, node, name);
+                               h = CtdlHostAlias(node);
+                               if ( (h != hostalias_localhost) && (h != hostalias_directory) ) {
+                                       coll = realloc(coll, strlen(coll) + strlen(addr) + 4);
+                                       if (coll == NULL) return(NULL);
+                                       if (!IsEmptyStr(coll)) {
+                                               strcat(coll, ",");
+                                       }
+                                       striplt(addr);
+                                       strcat(coll, addr);
                                }
                        }
-                       phree(fieldbuf);
                }
+       }
 
-               /* If we've hit the end of the message, bail out */
-               if (pos > strlen(rfc822)) done = 1;
+       if (IsEmptyStr(coll)) {
+               free(coll);
+               return(NULL);
        }
-       return(NULL);
+       return(coll);
 }