Tidying: coverity issues
authorJeremy Harris <jgh146exb@wizmail.org>
Fri, 3 Jun 2016 16:43:38 +0000 (17:43 +0100)
committerJeremy Harris <jgh146exb@wizmail.org>
Fri, 3 Jun 2016 16:43:38 +0000 (17:43 +0100)
src/exim_monitor/em_queue.c
src/src/queue.c
src/src/spool_in.c

index 893e438..103e63d 100644 (file)
@@ -656,32 +656,30 @@ if (jread != NULL)
 been delivered, and removing visible names. In the nonrecipients tree,
 domains are lower cased. */
 
-if (recipients_list != NULL)
-  {
+if (recipients_list)
   for (i = 0; i < recipients_count; i++)
     {
     uschar *pp;
     uschar *r = recipients_list[i].address;
     tree_node *node = tree_search(tree_nonrecipients, r);
 
-    if (node == NULL)
+    if (!node)
       {
       uschar temp[256];
       uschar *rr = temp;
-      Ustrcpy(temp, r);
-      while (*rr != 0 && *rr != '@') rr++;
-      while (*rr != 0) { *rr = tolower(*rr); rr++; }
+      Ustrncpy(temp, r, sizeof(temp));
+      while (*rr && *rr != '@') rr++;
+      while (*rr) { *rr = tolower(*rr); rr++; }
       node = tree_search(tree_nonrecipients, temp);
       }
 
-    if ((pp = strstric(r+1, qualify_domain, FALSE)) != NULL &&
-      *(--pp) == '@') *pp = 0;
-    if (node == NULL)
+    if ((pp = strstric(r+1, qualify_domain, FALSE)) && *(--pp) == '@')
+       *pp = 0;
+    if (!node)
       (void)find_dest(p, r, dest_add, FALSE);
     else
       (void)find_dest(p, r, dest_remove, FALSE);
     }
-  }
 
 /* We also need to scan the tree of non-recipients, which might
 contain child addresses that are not in the recipients list, but
index f53759c..1b7b9f7 100644 (file)
@@ -1378,7 +1378,6 @@ switch(action)
     }
 
   if (yield)
-    {
     if (spool_write_header(id, SW_MODIFYING, &errmsg) >= 0)
       printf("has been modified\n");
     else
@@ -1386,7 +1385,6 @@ switch(action)
       yield = FALSE;
       printf("- while %s: %s\n", doing, errmsg);
       }
-    }
 
   break;
   }
@@ -1394,8 +1392,11 @@ switch(action)
 /* Closing the datafile releases the lock and permits other processes
 to operate on the message (if it still exists). */
 
-(void)close(deliver_datafile);
-deliver_datafile = -1;
+if (deliver_datafile >= 0)
+  {
+  (void)close(deliver_datafile);
+  deliver_datafile = -1;
+  }
 return yield;
 }
 
index 7eb4290..9157987 100644 (file)
@@ -63,8 +63,8 @@ for (i = 0; i < 2; i++)
     if (i == 0) continue;
     if (!queue_running)
       log_write(0, LOG_MAIN, "Spool%s%s file %s-D not found",
-       *queue_name ? " Q=" : "",
-       *queue_name ? queue_name : "",
+       *queue_name ? US" Q=" : US"",
+       *queue_name ? queue_name : US"",
        id);
     }
   else