war on BSD style curly braces
[citadel.git] / citadel / internet_addressing.c
index 33d70451a6f36406c36056d8b91954ce0e728f4e..b517871620a14f8967aa5ad5464c6283db7e0d88 100644 (file)
@@ -2,7 +2,7 @@
  * This file contains functions which handle the mapping of Internet addresses
  * to users on the Citadel system.
  *
- * Copyright (c) 1987-2018 by the citadel.org team
+ * Copyright (c) 1987-2020 by the citadel.org team
  *
  * This program is open source software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3.
@@ -134,8 +134,7 @@ void utf8ify_rfc822_string(char *buf) {
        if (start != NULL) 
                FindNextEnd (start, end);
 
-       while ((start != NULL) && (end != NULL))
-       {
+       while ((start != NULL) && (end != NULL)) {
                next = strstr(end, "=?");
                if (next != NULL)
                        FindNextEnd(next, nextend);
@@ -143,9 +142,7 @@ void utf8ify_rfc822_string(char *buf) {
                        next = NULL;
 
                /* did we find two partitions */
-               if ((next != NULL) && 
-                   ((next - end) > 2))
-               {
+               if ((next != NULL) && ((next - end) > 2)) {
                        ptr = end + 2;
                        while ((ptr < next) && 
                               (isspace(*ptr) ||
@@ -180,8 +177,7 @@ void utf8ify_rfc822_string(char *buf) {
         */
        start = strstr(buf, "=?");
        FindNextEnd((start != NULL)? start : buf, end);
-       while (start != NULL && end != NULL && end > start)
-       {
+       while (start != NULL && end != NULL && end > start) {
                extract_token(charset, start, 1, '?', sizeof charset);
                extract_token(encoding, start, 2, '?', sizeof encoding);
                extract_token(istr, start, 3, '?', sizeof istr);
@@ -197,12 +193,10 @@ void utf8ify_rfc822_string(char *buf) {
                        
                        len = strlen(istr);
                        pos = 0;
-                       while (pos < len)
-                       {
+                       while (pos < len) {
                                if (istr[pos] == '_') istr[pos] = ' ';
                                pos++;
                        }
-
                        ibuflen = CtdlDecodeQuotedPrintable(ibuf, istr, len);
                }
                else {
@@ -353,7 +347,7 @@ int CtdlIsMe(char *addr, int addr_buf_len)
 
 
 /* If the last item in a list of recipients was truncated to a partial address,
- * remove it completely in order to avoid choking libSieve
+ * remove it completely in order to avoid choking library functions.
  */
 void sanitize_truncated_recipient(char *str)
 {
@@ -380,8 +374,7 @@ void sanitize_truncated_recipient(char *str)
  * This function is self explanatory.
  * (What can I say, I'm in a weird mood today...)
  */
-void remove_any_whitespace_to_the_left_or_right_of_at_symbol(char *name)
-{
+void remove_any_whitespace_to_the_left_or_right_of_at_symbol(char *name) {
        unsigned int i;
 
        for (i = 0; i < strlen(name); ++i) {
@@ -465,8 +458,7 @@ int alias(char *name)
  *
  * Caller needs to free the result using free_recipients()
  */
-recptypes *validate_recipients(const char *supplied_recipients, const char *RemoteIdentifier, int Flags)
-{
+recptypes *validate_recipients(const char *supplied_recipients, const char *RemoteIdentifier, int Flags) {
        struct CitContext *CCC = CC;
        recptypes *ret;
        char *recipients = NULL;
@@ -594,8 +586,7 @@ recptypes *validate_recipients(const char *supplied_recipients, const char *Remo
                                        Flags,
                                        0                       /* 0 = not a reply */
                                        );
-                               if (err)
-                               {
+                               if (err) {
                                        ++ret->num_error;
                                        invalid = 1;
                                } 
@@ -690,8 +681,7 @@ recptypes *validate_recipients(const char *supplied_recipients, const char *Remo
        }
        free(org_recp);
 
-       if ( (ret->num_local + ret->num_internet + ret->num_room + ret->num_error) == 0)
-       {
+       if ( (ret->num_local + ret->num_internet + ret->num_room + ret->num_error) == 0) {
                ret->num_error = (-1);
                strcpy(ret->errormsg, "No recipients specified.");
        }
@@ -884,13 +874,11 @@ void unfold_rfc822_field(char **field, char **FieldEnd)
        {
                if ((*sField=='\r') || (*sField=='\n'))
                {
-                       int Offset = 1;
-                       while (((*(sField + Offset) == '\r') ||
-                               (*(sField + Offset) == '\n') ||
-                               (isspace(*(sField + Offset)))) && 
-                              (sField + Offset < pFieldEnd))
-                               Offset ++;
-                       sField += Offset;
+                       int offset = 1;
+                       while ( ( (*(sField + offset) == '\r') || (*(sField + offset) == '\n' )) && (sField + offset < pFieldEnd) ) {
+                               offset ++;
+                       }
+                       sField += offset;
                        *pField = *sField;
                }
                else {
@@ -1588,58 +1576,23 @@ char *harvest_collected_addresses(struct CtdlMessage *msg) {
 
 /*
  * Helper function for CtdlRebuildDirectoryIndex()
- *
- * Call this function as a ForEachUser backend in order to queue up
- * user names, or call it with a null user to make it do the processing.
- * This allows us to maintain the list as a static instead of passing
- * pointers around.
  */
-void CtdlRebuildDirectoryIndex_backend(struct ctdluser *usbuf, void *data) {
+void CtdlRebuildDirectoryIndex_backend(char *username, void *data) {
 
-       struct crdib {
-               char name[64];
-               char emails[512];
-       };
-
-       static struct crdib *e = NULL;
-       static int num_e = 0;
-       static int alloc_e = 0;
-
-       /* this is the calling mode where we add a user */
+       int j = 0;
+       struct ctdluser usbuf;
 
-       if (usbuf != NULL) {
-               if (num_e >= alloc_e) {
-                       if (alloc_e == 0) {
-                               alloc_e = 100;
-                               e = malloc(sizeof(struct crdib) * alloc_e);
-                       }
-                       else {
-                               alloc_e *= 2;
-                               e = realloc(e, (sizeof(struct crdib) * alloc_e));
-                       }
-               }
-               strcpy(e[num_e].name, usbuf->fullname);
-               strcpy(e[num_e].emails, usbuf->emailaddrs);
-               ++num_e;
+       if (CtdlGetUser(&usbuf, username) != 0) {
                return;
        }
 
-       /* this is the calling mode where we do the processing */
-
-       int i, j;
-       for (i=0; i<num_e; ++i) {
-               if ( (!IsEmptyStr(e[i].name)) && (!IsEmptyStr(e[i].emails)) ) {
-                       for (j=0; j<num_tokens(e[i].emails, '|'); ++j) {
-                               char one_email[512];
-                               extract_token(one_email, e[i].emails, j, '|', sizeof one_email);
-                               CtdlDirectoryAddUser(one_email, e[i].name);
-                       }
+       if ( (!IsEmptyStr(usbuf.fullname)) && (!IsEmptyStr(usbuf.emailaddrs)) ) {
+               for (j=0; j<num_tokens(usbuf.emailaddrs, '|'); ++j) {
+                       char one_email[512];
+                       extract_token(one_email, usbuf.emailaddrs, j, '|', sizeof one_email);
+                       CtdlDirectoryAddUser(one_email, usbuf.fullname);
                }
        }
-       free(e);
-       num_e = 0;
-       alloc_e = 0;
-       return;
 }
 
 
@@ -1650,7 +1603,6 @@ void CtdlRebuildDirectoryIndex(void) {
        syslog(LOG_INFO, "internet_addressing: rebuilding email address directory index");
        cdb_trunc(CDB_DIRECTORY);
        ForEachUser(CtdlRebuildDirectoryIndex_backend, NULL);
-       CtdlRebuildDirectoryIndex_backend(NULL, NULL);
 }