Various pointer and syntax errors.

Uncovered by Pawel Worach using clang static analysis tool.

Fixes:
 * several NULL pointer dereferences
 * several unused variable return saves
 * a parentheses typo

diff --git a/lib/charset.c b/lib/charset.c
index c092f3fbdc001d27384af074093efe2b56b02fd1..b885941efe7f170e95032c2e482777f7dad80140 100644 (file)
--- a/lib/charset.c
+++ b/lib/charset.c
@@ -28,13 +28,12 @@
 #include "config.h"
 #include "util.h"
 
-/* Convert ISO-LATIN-1 to UTF-8
- */
+/** Convert ISO-LATIN-1 to UTF-8 */
 char *
 latin1_to_utf8(char *out, size_t size, const char *in)
 {
-    unsigned char *p;
-    for (p = (unsigned char *)out; *in && size > 2; in++) {
+    unsigned char *p = (unsigned char *)out;
+    for (; *in && size > 2; in++) {
         unsigned char ch = (unsigned char)*in;
         if (ch < 0x80) {
             *p++ = ch;
@@ -46,10 +45,8 @@ latin1_to_utf8(char *out, size_t size, const char *in)
             size--;
         }
     }
-    *p++ = '\0';
+    *p = '\0';
     if (*in)
         return NULL;
     return out;
 }
-
-

diff --git a/lib/radix.c b/lib/radix.c
index 94f81f832cfd1f2605f242d443a26adc424318ec..7210aad699eac98cf8046075128b5e861822bf77 100644 (file)
--- a/lib/radix.c
+++ b/lib/radix.c
@@ -620,7 +620,8 @@ squid_rn_addroute(void *v_arg, void *n_arg, struct squid_radix_node_head *head,
         if (tt == saved_tt) {
             struct squid_radix_node *xx = x;
             /* link in at head of list */
-            (tt = treenodes)->rn_dupedkey = t;
+            tt = treenodes;
+            tt->rn_dupedkey = t;
             tt->rn_flags = t->rn_flags;
             tt->rn_p = x = t->rn_p;
             if (x->rn_l == t)
@@ -630,7 +631,8 @@ squid_rn_addroute(void *v_arg, void *n_arg, struct squid_radix_node_head *head,
             saved_tt = tt;
             x = xx;
         } else {
-            (tt = treenodes)->rn_dupedkey = t->rn_dupedkey;
+            tt = treenodes;
+            tt->rn_dupedkey = t->rn_dupedkey;
             t->rn_dupedkey = tt;
         }
 #ifdef RN_DEBUG

diff --git a/src/cf_gen.cc b/src/cf_gen.cc
index d32a3e203f34e133b0545729a0e24b8f73ab0383..a3282f0ea99df52418f34b47b8576c23cb38f6f6 100644 (file)
--- a/src/cf_gen.cc
+++ b/src/cf_gen.cc
@@ -865,6 +865,8 @@ gen_conf(Entry * head, FILE * fp, bool verbose_output)
 
         if (enabled || verbose_output) {
             for (line = entry->nocomment; line != NULL; line = line->next) {
+                if (!line->data)
+                    continue;
                 if (!enabled && line->data[0] != '#')
                     fprintf(fp, "#%s\n", line->data);
                 else

diff --git a/tools/cachemgr.cc b/tools/cachemgr.cc
index 25800b8cf719c1e73c94968a623aa5ef962f7035..0a5df021e3076749de46b7f528f66f44b96bf1dc 100644 (file)
--- a/tools/cachemgr.cc
+++ b/tools/cachemgr.cc
@@ -1,4 +1,4 @@
-/*
+1158/*
  * DEBUG: section 0     CGI Cache Manager
  * AUTHOR: Duane Wessels
  *
@@ -558,7 +558,7 @@ munge_other_line(const char *buf, cachemgr_request * req)
 
     xfree(buf_copy);
     /* record ends */
-    l += snprintf(html + l, sizeof(html) - l, "</tr>\n");
+    snprintf(html + l, sizeof(html) - l, "</tr>\n");
     next_is_header = is_header && strstr(buf, "\t\t");
     table_line_num++;
     return html;
@@ -819,7 +819,7 @@ process_request(cachemgr_request * req)
 
     if ( !S.IsAnyAddr() ) {
         (void) 0;
-    } else if ( S = req->hostname)
+    } else if ((S = req->hostname))
         (void) 0;
     else {
         snprintf(buf, 1024, "Unknown host: %s\n", req->hostname);
@@ -1155,8 +1155,7 @@ make_auth_header(const cachemgr_request * req)
 
     assert(stringLength < sizeof(buf));
 
-    stringLength += snprintf(&buf[stringLength], sizeof(buf) - stringLength,
-                             "Proxy-Authorization: Basic %s\r\n", str64);
+    snprintf(&buf[stringLength], sizeof(buf) - stringLength, "Proxy-Authorization: Basic %s\r\n", str64);
 
     return buf;
 }