From: Florin Coras <fcoras@cisco.com>
Date: Mon, 7 Feb 2022 00:52:57 +0000 (-0800)
Subject: http_static: fix coverity warning
X-Git-Tag: v22.10-rc0~412
X-Git-Url: https://gerrit.fd.io/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F23%2F35223%2F2;p=vpp.git

http_static: fix coverity warning

Type: fix

Signed-off-by: Florin Coras <fcoras@cisco.com>
Change-Id: I6d5527c49e9085f35be988549c2f1460be7fd4f0
---

diff --git a/src/plugins/http_static/static_server.c b/src/plugins/http_static/static_server.c
index 48a33107926..0a1eb356fe8 100644
--- a/src/plugins/http_static/static_server.c
+++ b/src/plugins/http_static/static_server.c
@@ -319,7 +319,7 @@ try_url_handler (hss_main_t *hsm, hss_session_t *hs, http_req_method_t rt,
   uword *p, *url_table;
   int rv;
 
-  if (!hsm->enable_url_handlers)
+  if (!hsm->enable_url_handlers || !request)
     return -1;
 
   /* Look for built-in GET / POST handlers */
@@ -389,7 +389,9 @@ handle_request (hss_session_t *hs, http_req_method_t rt, u8 *request)
    * Construct the file to open
    * Browsers are capable of sporadically including a leading '/'
    */
-  if (request[0] == '/')
+  if (!request)
+    path = format (0, "%s%c", hsm->www_root, 0);
+  else if (request[0] == '/')
     path = format (0, "%s%s%c", hsm->www_root, request, 0);
   else
     path = format (0, "%s/%s%c", hsm->www_root, request, 0);