quiet clang analyzer scan-build warnings

(expansion of buffer_string_lenth() inline function and CONST_BUF_LEN()
 macro, which always check for NULL, appears to cause the analyzer to
 believe that a pointer might be NULL in cases where it otherwise can
 not be NULL)

x-ref:
  http://clang-analyzer.llvm.org/faq.html

1 files changed, 2 insertions, 1 deletions

diff --git a/src/mod_extforward.c b/src/mod_extforward.c
index cca5c4b9..9c8594d4 100644
--- a/src/mod_extforward.c
+++ b/src/mod_extforward.c
@@ -501,7 +501,7 @@ static int is_proxy_trusted(plugin_data *p, const char * const ip, size_t iplen)
         sock_addr addr;
         /* C funcs inet_aton(), inet_pton() require '\0'-terminated IP str */
         char addrstr[64]; /*(larger than INET_ADDRSTRLEN and INET6_ADDRSTRLEN)*/
-        if (iplen >= sizeof(addrstr)) return 0;
+        if (0 == iplen || iplen >= sizeof(addrstr)) return 0;
         memcpy(addrstr, ip, iplen);
         addrstr[iplen] = '\0';
 
@@ -843,6 +843,7 @@ static handler_t mod_extforward_Forwarded (request_st * const r, plugin_data * c
 
     /* parse out params associated with for=<ip> addr set above */
     oproto = ohost = oby = oremote_user = -1;
+    UNUSED(oby);
     j = ofor;
     if (j > 0) { do { --j; } while (j > 0 && -1 != offsets[j]); }
     if (-1 == offsets[j]) ++j;