summaryrefslogtreecommitdiff
path: root/src/sp_config.c
diff options
context:
space:
mode:
authorjvoisin2024-06-18 21:18:42 +0200
committerjvoisin2024-06-18 21:18:42 +0200
commit358bd3d82884f24353f6edddf097512b446938b9 (patch)
tree9708469242018923710260bb96ece957db221596 /src/sp_config.c
parentb005df282da43a2ba17b38e7da06a69353ea2845 (diff)
Fix an UBSAN warning
No need to use an ulong, an uint is more than enough for our usecases. This fixes the following warning: ``` src/sp_config.c:207:20: runtime error: store to misaligned address 0x796d1f9a78d4 for type 'u_long', which requires 8 byte alignment 0x796d1f9a78d4: note: pointer points here 00 00 00 00 ff 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 ^ #0 0x796d1f92251a in parse_ulong src/sp_config.c:207 #1 0x796d1f9239a9 in sp_process_rule src/sp_config.c:76 #2 0x796d1f92404f in sp_process_config_root src/sp_config.c:31 #3 0x796d1f96770d in sp_config_scan src/sp_config_scanner.c:1482 #4 0x796d1f922cd3 in sp_parse_config src/sp_config.c:62 #5 0x796d1f91e6cf in OnUpdateConfiguration src/snuffleupagus.c:522 #6 0x598aae727646 in zend_register_ini_entries_ex (/usr/bin/php8.2+0x36d646) (BuildId: 5228f916ded87172ddf0f3eca448f1e43874d60f) #7 0x796d1f91cae1 in zm_startup_snuffleupagus src/snuffleupagus.c:121 #8 0x598aae6b70df in zend_startup_module_ex (/usr/bin/php8.2+0x2fd0df) (BuildId: 5228f916ded87172ddf0f3eca448f1e43874d60f) #9 0x598aae6b717f (/usr/bin/php8.2+0x2fd17f) (BuildId: 5228f916ded87172ddf0f3eca448f1e43874d60f) #10 0x598aae6c566a in zend_hash_apply (/usr/bin/php8.2+0x30b66a) (BuildId: 5228f916ded87172ddf0f3eca448f1e43874d60f) #11 0x598aae64c0b5 in php_module_startup (/usr/bin/php8.2+0x2920b5) (BuildId: 5228f916ded87172ddf0f3eca448f1e43874d60f) #12 0x598aae4e2265 (/usr/bin/php8.2+0x128265) (BuildId: 5228f916ded87172ddf0f3eca448f1e43874d60f) #13 0x796d21b4dc89 in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58 #14 0x796d21b4dd44 in __libc_start_main_impl ../csu/libc-start.c:360 #15 0x598aae4e3550 in _start (/usr/bin/php8.2+0x129550) (BuildId: 5228f916ded87172ddf0f3eca448f1e43874d60f) SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior src/sp_config.c:207:20 in ```
Diffstat (limited to 'src/sp_config.c')
-rw-r--r--src/sp_config.c6
1 files changed, 3 insertions, 3 deletions
diff --git a/src/sp_config.c b/src/sp_config.c
index de58c2a..ed41a6a 100644
--- a/src/sp_config.c
+++ b/src/sp_config.c
@@ -11,7 +11,7 @@ static zend_result sp_process_config_root(sp_parsed_keyword *parsed_rule) {
11 {parse_unserialize_noclass, SP_TOKEN_UNSERIALIZE_NOCLASS, &(SPCFG(unserialize_noclass))}, 11 {parse_unserialize_noclass, SP_TOKEN_UNSERIALIZE_NOCLASS, &(SPCFG(unserialize_noclass))},
12 {parse_enable, SP_TOKEN_HARDEN_RANDOM, &(SPCFG(random).enable)}, 12 {parse_enable, SP_TOKEN_HARDEN_RANDOM, &(SPCFG(random).enable)},
13 {parse_log_media, SP_TOKEN_LOG_MEDIA, &(SPCFG(log_media))}, 13 {parse_log_media, SP_TOKEN_LOG_MEDIA, &(SPCFG(log_media))},
14 {parse_ulong, SP_TOKEN_LOG_MAX_LEN, &(SPCFG(log_max_len))}, 14 {parse_uint, SP_TOKEN_LOG_MAX_LEN, &(SPCFG(log_max_len))},
15 {parse_disabled_functions, SP_TOKEN_DISABLE_FUNC, NULL}, 15 {parse_disabled_functions, SP_TOKEN_DISABLE_FUNC, NULL},
16 {parse_readonly_exec, SP_TOKEN_READONLY_EXEC, &(SPCFG(readonly_exec))}, 16 {parse_readonly_exec, SP_TOKEN_READONLY_EXEC, &(SPCFG(readonly_exec))},
17 {parse_enable, SP_TOKEN_GLOBAL_STRICT, &(SPCFG(global_strict).enable)}, 17 {parse_enable, SP_TOKEN_GLOBAL_STRICT, &(SPCFG(global_strict).enable)},
@@ -198,13 +198,13 @@ SP_PARSEKW_FN(parse_int) {
198 return ret; 198 return ret;
199} 199}
200 200
201SP_PARSEKW_FN(parse_ulong) { 201SP_PARSEKW_FN(parse_uint) {
202 int ret = SP_PARSER_SUCCESS; 202 int ret = SP_PARSER_SUCCESS;
203 SP_PARSE_ARG(value); 203 SP_PARSE_ARG(value);
204 204
205 char *endptr; 205 char *endptr;
206 errno = 0; 206 errno = 0;
207 *(u_long*)retval = (u_long)strtoul(ZSTR_VAL(value), &endptr, 10); 207 *(u_int*)retval = (u_int)strtoul(ZSTR_VAL(value), &endptr, 10);
208 if (errno != 0 || !endptr || endptr == ZSTR_VAL(value)) { 208 if (errno != 0 || !endptr || endptr == ZSTR_VAL(value)) {
209 sp_log_err("config", "Failed to parse arg '%s' of `%s` on line %zu", ZSTR_VAL(value), token, kw->lineno); 209 sp_log_err("config", "Failed to parse arg '%s' of `%s` on line %zu", ZSTR_VAL(value), token, kw->lineno);
210 ret = SP_PARSER_ERROR; 210 ret = SP_PARSER_ERROR;