From: Kevin Day Date: Sun, 25 Dec 2022 02:00:05 +0000 (-0600) Subject: Security: Check if either 'main' or 'setting' is NULL. X-Git-Url: https://git.kevux.org/?a=commitdiff_plain;h=b2a0469394af080573e168b4f750263e3959e7e5;p=fll Security: Check if either 'main' or 'setting' is NULL. Name the void pointer 'setting' as 'void_setting' for consistency purposes. Note that this should not be named 'void_setting' in the header files. --- diff --git a/level_3/fss_write/c/basic/fss_write.c b/level_3/fss_write/c/basic/fss_write.c index 12c8975..78e7332 100644 --- a/level_3/fss_write/c/basic/fss_write.c +++ b/level_3/fss_write/c/basic/fss_write.c @@ -8,6 +8,8 @@ extern "C" { #ifndef _di_fss_write_basic_process_content_ void fss_write_basic_process_content(fll_program_data_t * const main, void * const void_setting, const bool last) { + if (!main || !void_setting) return; + fss_write_setting_t * const setting = macro_fss_write_setting(void_setting); if (setting->content) { @@ -51,15 +53,19 @@ extern "C" { #endif // _di_fss_write_basic_process_content_ #ifndef _di_fss_write_basic_process_help_ - void fss_write_basic_process_help(fll_program_data_t * const main, void * const setting) { + void fss_write_basic_process_help(fll_program_data_t * const main, void * const void_setting) { + + if (!main || !void_setting) return; - fss_write_basic_print_help(setting, main->message); + fss_write_basic_print_help(void_setting, main->message); } #endif // _di_fss_write_basic_process_help_ #ifndef _di_fss_write_basic_process_object_ void fss_write_basic_process_object(fll_program_data_t * const main, void * const void_setting) { + if (!main || !void_setting) return; + fss_write_setting_t * const setting = macro_fss_write_setting(void_setting); if (setting->object) { diff --git a/level_3/fss_write/c/basic_list/fss_write.c b/level_3/fss_write/c/basic_list/fss_write.c index f80dac2..38d636b 100644 --- a/level_3/fss_write/c/basic_list/fss_write.c +++ b/level_3/fss_write/c/basic_list/fss_write.c @@ -8,6 +8,8 @@ extern "C" { #ifndef _di_fss_write_basic_list_process_content_ void fss_write_basic_list_process_content(fll_program_data_t * const main, void * const void_setting, const bool last) { + if (!main || !void_setting) return; + fss_write_setting_t * const setting = macro_fss_write_setting(void_setting); if (setting->content) { @@ -54,15 +56,19 @@ extern "C" { #endif // _di_fss_write_basic_list_process_content_ #ifndef _di_fss_write_basic_list_process_help_ - void fss_write_basic_list_process_help(fll_program_data_t * const main, void * const setting) { + void fss_write_basic_list_process_help(fll_program_data_t * const main, void * const void_setting) { + + if (!main || !void_setting) return; - fss_write_basic_list_print_help(setting, main->message); + fss_write_basic_list_print_help(void_setting, main->message); } #endif // _di_fss_write_basic_list_process_help_ #ifndef _di_fss_write_basic_list_process_object_ void fss_write_basic_list_process_object(fll_program_data_t * const main, void * const void_setting) { + if (!main || !void_setting) return; + fss_write_setting_t * const setting = macro_fss_write_setting(void_setting); if (setting->object) { diff --git a/level_3/fss_write/c/embedded_list/fss_write.c b/level_3/fss_write/c/embedded_list/fss_write.c index 543661d..9978353 100644 --- a/level_3/fss_write/c/embedded_list/fss_write.c +++ b/level_3/fss_write/c/embedded_list/fss_write.c @@ -8,6 +8,8 @@ extern "C" { #ifndef _di_fss_write_embedded_list_process_content_ void fss_write_embedded_list_process_content(fll_program_data_t * const main, void * const void_setting, const bool last) { + if (!main || !void_setting) return; + fss_write_setting_t * const setting = macro_fss_write_setting(void_setting); if (setting->content) { @@ -59,15 +61,19 @@ extern "C" { #endif // _di_fss_write_embedded_list_process_content_ #ifndef _di_fss_write_embedded_list_process_help_ - void fss_write_embedded_list_process_help(fll_program_data_t * const main, void * const setting) { + void fss_write_embedded_list_process_help(fll_program_data_t * const main, void * const void_setting) { + + if (!main || !void_setting) return; - fss_write_embedded_list_print_help(setting, main->message); + fss_write_embedded_list_print_help(void_setting, main->message); } #endif // _di_fss_write_embedded_list_process_help_ #ifndef _di_fss_write_embedded_list_process_object_ void fss_write_embedded_list_process_object(fll_program_data_t * const main, void * const void_setting) { + if (!main || !void_setting) return; + fss_write_setting_t * const setting = macro_fss_write_setting(void_setting); if (setting->object) { diff --git a/level_3/fss_write/c/extended/fss_write.c b/level_3/fss_write/c/extended/fss_write.c index 86ec242..15b2cfa 100644 --- a/level_3/fss_write/c/extended/fss_write.c +++ b/level_3/fss_write/c/extended/fss_write.c @@ -8,6 +8,8 @@ extern "C" { #ifndef _di_fss_write_extended_process_content_ void fss_write_extended_process_content(fll_program_data_t * const main, void * const void_setting, const bool last) { + if (!main || !void_setting) return; + fss_write_setting_t * const setting = macro_fss_write_setting(void_setting); if (setting->content) { @@ -63,15 +65,19 @@ extern "C" { #endif // _di_fss_write_extended_process_content_ #ifndef _di_fss_write_extended_process_help_ - void fss_write_extended_process_help(fll_program_data_t * const main, void * const setting) { + void fss_write_extended_process_help(fll_program_data_t * const main, void * const void_setting) { + + if (!main || !void_setting) return; - fss_write_extended_print_help(setting, main->message); + fss_write_extended_print_help(void_setting, main->message); } #endif // _di_fss_write_extended_process_help_ #ifndef _di_fss_write_extended_process_object_ void fss_write_extended_process_object(fll_program_data_t * const main, void * const void_setting) { + if (!main || !void_setting) return; + fss_write_setting_t * const setting = macro_fss_write_setting(void_setting); if (setting->object) { diff --git a/level_3/fss_write/c/extended_list/fss_write.c b/level_3/fss_write/c/extended_list/fss_write.c index 1496c2f..e905ce3 100644 --- a/level_3/fss_write/c/extended_list/fss_write.c +++ b/level_3/fss_write/c/extended_list/fss_write.c @@ -8,6 +8,8 @@ extern "C" { #ifndef _di_fss_write_extended_list_process_content_ void fss_write_extended_list_process_content(fll_program_data_t * const main, void * const void_setting, const bool last) { + if (!main || !void_setting) return; + fss_write_setting_t * const setting = macro_fss_write_setting(void_setting); if (setting->content) { @@ -59,15 +61,19 @@ extern "C" { #endif // _di_fss_write_extended_list_process_content_ #ifndef _di_fss_write_extended_list_process_help_ - void fss_write_extended_list_process_help(fll_program_data_t * const main, void * const setting) { + void fss_write_extended_list_process_help(fll_program_data_t * const main, void * const void_setting) { + + if (!main || !void_setting) return; - fss_write_extended_list_print_help(macro_fss_write_setting(setting), main->message); + fss_write_extended_list_print_help(macro_fss_write_setting(void_setting), main->message); } #endif // _di_fss_write_extended_list_process_help_ #ifndef _di_fss_write_extended_list_process_object_ void fss_write_extended_list_process_object(fll_program_data_t * const main, void * const void_setting) { + if (!main || !void_setting) return; + fss_write_setting_t * const setting = macro_fss_write_setting(void_setting); if (setting->object) { diff --git a/level_3/fss_write/c/main/fss_write.c b/level_3/fss_write/c/main/fss_write.c index a07d92f..4ba9537 100644 --- a/level_3/fss_write/c/main/fss_write.c +++ b/level_3/fss_write/c/main/fss_write.c @@ -47,14 +47,14 @@ extern "C" { #endif // _di_fss_write_main_ #ifndef _di_fss_write_process_normal_ - void fss_write_process_normal(fll_program_data_t * const main, void * const setting) { + void fss_write_process_normal(fll_program_data_t * const main, void * const void_setting) { fss_write_process_normal_data( main, - macro_fss_write_setting(setting), - (macro_fss_write_setting(setting)->flag & fss_write_flag_object_e) - ? macro_fss_write_setting(setting)->objects.used - : macro_fss_write_setting(setting)->contentss.used + macro_fss_write_setting(void_setting), + (macro_fss_write_setting(void_setting)->flag & fss_write_flag_object_e) + ? macro_fss_write_setting(void_setting)->objects.used + : macro_fss_write_setting(void_setting)->contentss.used ); } #endif // _di_fss_write_process_normal_ diff --git a/level_3/fss_write/c/main/main.c b/level_3/fss_write/c/main/main.c index 367f2cd..832854e 100644 --- a/level_3/fss_write/c/main/main.c +++ b/level_3/fss_write/c/main/main.c @@ -43,9 +43,11 @@ int main(const int argc, const f_string_t *argv, const f_string_t *envp) { } #ifndef _di_fss_write_main_process_help_ - void fss_write_main_process_help(fll_program_data_t * const main, void * const setting) { + void fss_write_main_process_help(fll_program_data_t * const main, void * const void_setting) { - fss_write_main_print_help(macro_fss_write_setting(setting), main->message); + if (!main || !void_setting) return; + + fss_write_main_print_help(macro_fss_write_setting(void_setting), main->message); } #endif // _di_fss_write_main_process_help_ diff --git a/level_3/fss_write/c/payload/fss_write.c b/level_3/fss_write/c/payload/fss_write.c index 024b3f6..cc05056 100644 --- a/level_3/fss_write/c/payload/fss_write.c +++ b/level_3/fss_write/c/payload/fss_write.c @@ -6,15 +6,19 @@ extern "C" { #endif #ifndef _di_fss_write_payload_process_help_ - void fss_write_payload_process_help(fll_program_data_t * const main, void * const setting) { + void fss_write_payload_process_help(fll_program_data_t * const main, void * const void_setting) { - fss_write_payload_print_help(macro_fss_write_setting(setting), main->message); + if (!main || !void_setting) return; + + fss_write_payload_print_help(macro_fss_write_setting(void_setting), main->message); } #endif // _di_fss_write_payload_process_help_ #ifndef _di_fss_write_payload_process_pipe_ void fss_write_payload_process_pipe(fll_program_data_t * const main, void * const void_setting) { + if (!main || !void_setting) return; + f_status_t status_pipe = F_none; f_file_t input = f_file_t_initialize; input.id = F_type_descriptor_input_d; @@ -376,6 +380,8 @@ extern "C" { #ifndef _di_fss_write_payload_process_set_ void fss_write_payload_process_set(fll_program_data_t * const main, void * const void_setting) { + if (!main || !void_setting) return; + fss_write_setting_t * const setting = macro_fss_write_setting(void_setting); if ((!(setting->flag & fss_write_flag_partial_e) || (setting->flag & fss_write_flag_partial_e) && (setting->flag & fss_write_flag_object_e))) {