commit 40bf71ac3e8eeb6e596a738d175c44478a7b3ce6
parent 532a5c1900e15dd6e9f877f40b7000f741c934b2
Author: Evgeny Grin (Karlson2k) <k2k@drgrin.dev>
Date: Thu, 24 Apr 2025 19:07:25 +0200
post_parser_funcs.c: simplified code, fixed compiler warning
Diffstat:
1 file changed, 5 insertions(+), 13 deletions(-)
diff --git a/src/mhd2/post_parser_funcs.c b/src/mhd2/post_parser_funcs.c
@@ -2461,7 +2461,6 @@ check_post_leftovers_urlenc (struct MHD_Connection *restrict c,
size_t name_len;
size_t value_start;
size_t value_len;
- bool need_more_space;
bool have_extra_space;
bool need_value_stream;
@@ -2469,7 +2468,6 @@ check_post_leftovers_urlenc (struct MHD_Connection *restrict c,
mhd_assert (pos <= c->rq.cntn.lbuf.size); // TODO: support processing in connection buffer
mhd_assert (*pdata_size >= pos);
have_extra_space = (pos < c->rq.cntn.lbuf.size);
- need_more_space = false; /**< No space for zero termination */
need_value_stream = false; /**< Value cannot be zero-terminated and must be streamed */
switch (uf->st)
{
@@ -2491,13 +2489,10 @@ check_post_leftovers_urlenc (struct MHD_Connection *restrict c,
mhd_assert (name_start + name_len <= pos);
if (! have_extra_space &&
(name_start + name_len == pos))
- need_more_space = true;
- else
- {
- buf[name_start + name_len] = 0; /* Zero-terminate the result, an extra byte is available */
- value_start = 0;
- value_len = 0;
- }
+ return report_low_lbuf_mem (c);
+ buf[name_start + name_len] = 0; /* Zero-terminate the result, an extra byte is available */
+ value_start = 0;
+ value_len = 0;
break;
case mhd_POST_UENC_ST_EQ_FOUND:
mhd_assert (pos == *pdata_size);
@@ -2580,12 +2575,9 @@ check_post_leftovers_urlenc (struct MHD_Connection *restrict c,
if (NULL != c->rq.app_act.head_act.data.post_parse.stream_reader)
p_data->force_streamed = true;
else
- need_more_space = true;
+ return report_low_lbuf_mem (c);
}
- if (need_more_space)
- return report_low_lbuf_mem (c);
-
if (process_complete_field (c,
buf,
&(p_data->next_parse_pos),