Added right return values to non-void functions with "forever" loop

to fix Eclipse static code analysis

libhtp/htp/htp_request.c

@@ -99,6 +99,7 @@ int htp_connp_REQ_BODY_CHUNKED_DATA_END(htp_connp_t *connp) {
             return HTP_OK;
         }
     }
+    return HTP_ERROR;
 }
 
 /**
@@ -151,6 +152,7 @@ int htp_connp_REQ_BODY_CHUNKED_DATA(htp_connp_t *connp) {
             }
         }
     }
+    return HTP_ERROR;
 }
 
 /**
@@ -194,6 +196,7 @@ int htp_connp_REQ_BODY_CHUNKED_LENGTH(htp_connp_t *connp) {
             return HTP_OK;
         }
     }
+    return HTP_ERROR;
 }
 
 /**
@@ -252,6 +255,7 @@ int htp_connp_REQ_BODY_IDENTITY(htp_connp_t *connp) {
             }
         }
     }
+    return HTP_ERROR;
 }
 
 /**
@@ -525,6 +529,7 @@ int htp_connp_REQ_HEADERS(htp_connp_t *connp) {
             connp->in_header_line_counter++;
         }
     }
+    return HTP_ERROR;
 }
 
 /**
@@ -710,6 +715,7 @@ int htp_connp_REQ_LINE(htp_connp_t *connp) {
             return HTP_OK;
         }
     }
+    return HTP_ERROR;
 }
 
 /**
@@ -904,5 +910,6 @@ int htp_connp_req_data(htp_connp_t *connp, htp_time_t timestamp, unsigned char *
             return STREAM_STATE_ERROR;
         }
     }
+    return HTP_ERROR;
 }
 

libhtp/htp/htp_response.c

@@ -54,6 +54,8 @@ int htp_connp_RES_BODY_CHUNKED_DATA_END(htp_connp_t *connp) {
             return HTP_OK;
         }
     }
+
+    return HTP_ERROR;
 }
 
 /**
@@ -114,6 +116,7 @@ int htp_connp_RES_BODY_CHUNKED_DATA(htp_connp_t *connp) {
             }
         }
     }
+    return HTP_ERROR;
 }
 
 /**
@@ -157,6 +160,7 @@ int htp_connp_RES_BODY_CHUNKED_LENGTH(htp_connp_t *connp) {
             return HTP_OK;
         }
     }
+    return HTP_ERROR;
 }
 
 /**
@@ -247,6 +251,7 @@ int htp_connp_RES_BODY_IDENTITY(htp_connp_t *connp) {
             }
         }
     }
+    return HTP_ERROR;
 }
 
 /**
@@ -552,6 +557,7 @@ int htp_connp_RES_HEADERS(htp_connp_t *connp) {
             connp->out_header_line_counter++;
         }
     }
+    return HTP_ERROR;
 }
 
 /**
@@ -635,6 +641,7 @@ int htp_connp_RES_LINE(htp_connp_t *connp) {
             return HTP_OK;
         }
     }
+    return HTP_ERROR;
 }
 
 size_t htp_connp_res_data_consumed(htp_connp_t *connp) {
@@ -836,5 +843,6 @@ int htp_connp_res_data(htp_connp_t *connp, htp_time_t timestamp, unsigned char *
             return STREAM_STATE_ERROR;
         }
     }
+    return HTP_ERROR;
 }
 

libhtp/htp/htp_util.c

@@ -647,6 +647,7 @@ uint8_t bestfit_codepoint(htp_cfg_t *cfg, uint32_t codepoint) {
         // Move to the next triplet
         p += 3;
     }
+    return cfg->path_replacement_char;
 }
 
 /**