stephenseo/c_simple_http
1
0
Fork 0
Code Issues Pull requests Projects Releases 8 Packages Wiki Activity Actions

Fix edge case of IF with FOREACH variable

Browse source
This commit is contained in:
Stephen Seo 2024-11-29 11:38:10 +09:00
parent cfa7a065b2
commit 16154393d0
2 changed files with 51 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
example_config/example.config
View file

@ -25,6 +25,18 @@ HTML='''
</style> </style>
</head> </head>
<body> <body>
<h2><a href="/inner">inner</a></h2><br>
<h2>Test EachIf Expr</h2>
{{{!FOREACH EachIfTest_Array}}}
{{{!IF EachIfTest_Array==true}}}
EachIfTest_Array entry is <b>true</b>. <br>
{{{!ELSE}}}
EachIfTest_Array entry is <b>false</b>. <br>
{{{!ENDIF}}}
{{{!ENDFOREACH}}}
<br>
<h2>Test IF/FOREACH Expr</h2> <h2>Test IF/FOREACH Expr</h2>
Outer IF<br> Outer IF<br>
@ -150,6 +162,11 @@ Each_FILE='''example_config/each_file_zero.html'''
Each_FILE='''example_config/each_file_one.html''' Each_FILE='''example_config/each_file_one.html'''
Each_FILE='''example_config/each_file_two.html''' Each_FILE='''example_config/each_file_two.html'''
EachIfTest_Array=false
EachIfTest_Array=true
EachIfTest_Array=false
EachIfTest_Array=true
PATH=/inner PATH=/inner
HTML_FILE='''example_config/inner.html''' HTML_FILE='''example_config/inner.html'''
VAR_FILE='''example_config/var.html''' VAR_FILE='''example_config/var.html'''

41
src/http_template.c
View file

@ -180,6 +180,8 @@ int c_simple_http_internal_parse_if_expression(
const char *var, const char *var,
const size_t var_offset, const size_t var_offset,
const size_t var_size, const size_t var_size,
const uint64_t *for_state_idx,
SDArchiverHashMap *array_vars,
char **left_side_out, char **left_side_out,
char **right_side_out, char **right_side_out,
uint_fast8_t *is_equality_out, uint_fast8_t *is_equality_out,
@ -269,11 +271,31 @@ int c_simple_http_internal_parse_if_expression(
simple_archiver_list_free(&var_parts); simple_archiver_list_free(&var_parts);
var_parts = simple_archiver_list_init(); var_parts = simple_archiver_list_init();
C_SIMPLE_HTTP_ConfigValue *config_value = // At this point, var_buf contains the left-hand-side string that hasn't been
simple_archiver_hash_map_get(wrapped_hash_map->hash_map, // expanded yet.
var_buf,
strlen(var_buf) + 1); // Check if ForEach variable.
if (!config_value || !config_value->value) { char *value = NULL;
C_SIMPLE_HTTP_ConfigValue *config_value = NULL;
if (for_state_idx != NULL) {
if (var_index != -1) {
fprintf(stderr,
"ERROR Invalid indexing on expanded FOREACH variable! %s\n",
var);
return 1;
}
value = c_simple_http_internal_get_for_var(var_buf,
*for_state_idx,
array_vars);
}
if (!value) {
// No ForEach variable, get as regular variable.
config_value =
simple_archiver_hash_map_get(wrapped_hash_map->hash_map,
var_buf,
strlen(var_buf) + 1);
}
if (!value && (!config_value || !config_value->value)) {
fprintf(stderr, "ERROR Invalid VAR after \"IF/ELSEIF\"! %s\n", var); fprintf(stderr, "ERROR Invalid VAR after \"IF/ELSEIF\"! %s\n", var);
return 1; return 1;
} else if (var_index >= 0) { } else if (var_index >= 0) {
@ -291,7 +313,8 @@ int c_simple_http_internal_parse_if_expression(
uint64_t left_side_size = 0; uint64_t left_side_size = 0;
if (c_simple_http_internal_ends_with_FILE(var_buf) == 0) { if (c_simple_http_internal_ends_with_FILE(var_buf) == 0) {
*left_side_out = *left_side_out =
c_simple_http_FILE_to_c_str(config_value->value, &left_side_size); c_simple_http_FILE_to_c_str(value ? value : config_value->value,
&left_side_size);
if (!*left_side_out || left_side_size == 0) { if (!*left_side_out || left_side_size == 0) {
fprintf(stderr, "ERROR _FILE variable could not be read! %s\n", var); fprintf(stderr, "ERROR _FILE variable could not be read! %s\n", var);
if (*left_side_out) { if (*left_side_out) {
@ -304,7 +327,7 @@ int c_simple_http_internal_parse_if_expression(
left_side_size = strlen(*left_side_out); left_side_size = strlen(*left_side_out);
c_simple_http_internal_set_out_insert(files_set_out, config_value->value); c_simple_http_internal_set_out_insert(files_set_out, config_value->value);
} else { } else {
*left_side_out = strdup(config_value->value); *left_side_out = strdup(value ? value : config_value->value);
c_simple_http_trim_end_whitespace(*left_side_out); c_simple_http_trim_end_whitespace(*left_side_out);
left_side_size = strlen(*left_side_out); left_side_size = strlen(*left_side_out);
} }
@ -599,6 +622,8 @@ int c_simple_http_internal_handle_inside_delimeters(
var, var,
1 + 3, 1 + 3,
var_size, var_size,
(*state) & 0x4 ? &for_state_idx : NULL,
array_vars,
&left_side, &left_side,
&right_side, &right_side,
&is_equality, &is_equality,
@ -673,6 +698,8 @@ int c_simple_http_internal_handle_inside_delimeters(
var, var,
1 + 7, 1 + 7,
var_size, var_size,
(*state) & 0x4 ? &for_state_idx : NULL,
array_vars,
&left_side, &left_side,
&right_side, &right_side,
&is_equality, &is_equality,