strncat() warnings on alpine (#497)

nbs-system · Jun 15, 2020 · 40d91e2 · 40d91e2
1 parent da216ca
commit 40d91e2
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/naxsi_src/naxsi_runtime.c b/naxsi_src/naxsi_runtime.c
@@ -709,10 +709,10 @@ ngx_http_dummy_is_rule_whitelisted_n(ngx_http_request_t *req,
   tmp_hashname.data = ngx_pcalloc(req->pool, tmp_hashname.len+2);
   if (target_name) {
     tmp_hashname.len++;
-    strncat((char*)tmp_hashname.data, "#", 1);
+    strcat((char*)tmp_hashname.data, "#");
   }
   strncat((char*) tmp_hashname.data, (char*)req->uri.data, req->uri.len);
-  strncat((char*)tmp_hashname.data, "#", 1);
+  strcat((char*)tmp_hashname.data, "#");
   strncat((char*)tmp_hashname.data, (char*)name->data, name->len);
 
   NX_DEBUG(_debug_whitelist_compat, NGX_LOG_DEBUG_HTTP, req->connection->log, 0, "hashing MIX [%V] ($URL:x|$X_VAR:y) or ($URL:x|$X_VAR:y|NAME)", &tmp_hashname);

diff --git a/naxsi_src/naxsi_utils.c b/naxsi_src/naxsi_utils.c
@@ -430,11 +430,11 @@ ngx_http_wlr_find(ngx_conf_t *cf, ngx_http_dummy_loc_conf_t *dlc,
 		   "whitelist targets |NAME");
 
 
-      strncat(*fullname, (const char *) "#", 1);
+      strcat(*fullname, (const char *) "#");
     }
     strncat(*fullname, (const char *) custloc_array(curr->br->custom_locations->elts)[uri_idx].target.data, 
 	    custloc_array(curr->br->custom_locations->elts)[uri_idx].target.len);
-    strncat(*fullname, (const char *) "#", 1);
+    strcat(*fullname, (const char *) "#");
     strncat(*fullname, (const char *) custloc_array(curr->br->custom_locations->elts)[name_idx].target.data, 
 	    custloc_array(curr->br->custom_locations->elts)[name_idx].target.len);
   }
@@ -452,7 +452,7 @@ ngx_http_wlr_find(ngx_conf_t *cf, ngx_http_dummy_loc_conf_t *dlc,
 		   "whitelist targets |NAME");
 
 
-      strncat(*fullname, (const char *) "#", 1);
+      strcat(*fullname, (const char *) "#");
     }
 
 
@@ -467,7 +467,7 @@ ngx_http_wlr_find(ngx_conf_t *cf, ngx_http_dummy_loc_conf_t *dlc,
 
     *fullname = ngx_pcalloc(cf->pool, custloc_array(curr->br->custom_locations->elts)[name_idx].target.len + 2);
     if (curr->br->target_name)
-      strncat(*fullname, (const char *) "#", 1);
+      strcat(*fullname, (const char *) "#");
     strncat(*fullname, (const char *) custloc_array(curr->br->custom_locations->elts)[name_idx].target.data, 
 	    custloc_array(curr->br->custom_locations->elts)[name_idx].target.len);
   }