diff options
| author | Rafael G. Martins <rafael@rafaelmartins.eng.br> | 2023-02-11 23:18:51 +0100 |
|---|---|---|
| committer | Rafael G. Martins <rafael@rafaelmartins.eng.br> | 2023-02-11 23:18:51 +0100 |
| commit | 3f6ae823cd8ff8049abeca2be495c727563c919e (patch) | |
| tree | 33cdbeb162263c0c360dd1b915b7060634d0986e /src | |
| parent | e52e1f0b9d0796412ee36de260c6bbbcde348b0a (diff) | |
| download | blogc-3f6ae823cd8ff8049abeca2be495c727563c919e.tar.gz blogc-3f6ae823cd8ff8049abeca2be495c727563c919e.tar.bz2 blogc-3f6ae823cd8ff8049abeca2be495c727563c919e.zip | |
make: fix global variable name validation
Diffstat (limited to 'src')
| -rw-r--r-- | src/blogc-make/settings.c | 19 |
1 files changed, 17 insertions, 2 deletions
diff --git a/src/blogc-make/settings.c b/src/blogc-make/settings.c index be976e3..1079fc0 100644 --- a/src/blogc-make/settings.c +++ b/src/blogc-make/settings.c @@ -119,9 +119,24 @@ bm_settings_parse(const char *content, size_t content_len, bc_error_t **err) if (global != NULL) { for (size_t i = 0; global[i] != NULL; i++) { for (size_t j = 0; global[i][j] != '\0'; j++) { - if (!((global[i][j] >= 'A' && global[i][j] <= 'Z') || global[i][j] == '_')) { + if (j == 0) { + if (!(global[i][j] >= 'A' && global[i][j] <= 'Z')) { + *err = bc_error_new_printf(BLOGC_MAKE_ERROR_SETTINGS, + "Invalid [%s] key (first character must be uppercase): %s", + section, global[i]); + bc_strv_free(global); + bm_settings_free(rv); + rv = NULL; + goto cleanup; + } + continue; + } + if (!((global[i][j] >= 'A' && global[i][j] <= 'Z') || + (global[i][j] >= '0' && global[i][j] <= '9') || + global[i][j] == '_')) { *err = bc_error_new_printf(BLOGC_MAKE_ERROR_SETTINGS, - "Invalid [%s] key: %s", section, global[i]); + "Invalid [%s] key (must be uppercase with '_' and digits after first character): %s", + section, global[i]); bc_strv_free(global); bm_settings_free(rv); rv = NULL; |