diff --git a/framework/src/fwk_io.c b/framework/src/fwk_io.c index f0aed888ea185deeeec62b3192c5522c8bb40e57..b5b2c7604dbc382e1498f236c74f9c64c3c1b9e1 100644 --- a/framework/src/fwk_io.c +++ b/framework/src/fwk_io.c @@ -388,7 +388,7 @@ int fwk_io_vprintf( return FWK_E_STATE; } - buffer = fwk_mm_alloc_notrap(sizeof(buffer[0]), (size_t)(length + 1)); + buffer = fwk_mm_alloc_notrap((size_t)(length + 1), sizeof(buffer[0])); if (buffer == NULL) { /* Not enough memory for the string buffer */ return FWK_E_NOMEM; } diff --git a/module/power_capping/test/mod_power_capping_unit_test.c b/module/power_capping/test/mod_power_capping_unit_test.c index 48c373594a06df560efb1197c007830aaa0b8efa..a6380ac39a470590cc7ec88e7e74814157f85259 100644 --- a/module/power_capping/test/mod_power_capping_unit_test.c +++ b/module/power_capping/test/mod_power_capping_unit_test.c @@ -217,7 +217,7 @@ void utest_pcapping_domain_init_success(void) int status; fwk_id_t element_id; - for (unsigned int index; index < TEST_DOMAIN_COUNT; index++) { + for (unsigned int index = 0U; index < TEST_DOMAIN_COUNT; index++) { element_id = FWK_ID_ELEMENT(FWK_MODULE_IDX_POWER_CAPPING, index); void *data = (void *)(uintptr_t)(345 + index); diff --git a/module/scmi_power_capping/src/mod_scmi_power_capping.c b/module/scmi_power_capping/src/mod_scmi_power_capping.c index 1d9574dd66f8906f9539c8370a697afc0aea6f4c..5aecea23839eab6ce4990326a3815e7525b2d046 100644 --- a/module/scmi_power_capping/src/mod_scmi_power_capping.c +++ b/module/scmi_power_capping/src/mod_scmi_power_capping.c @@ -81,7 +81,7 @@ static int scmi_power_capping_init( return FWK_E_SUPPORT; } - struct mod_scmi_power_capping_context ctx; + struct mod_scmi_power_capping_context ctx = { 0 }; ctx.power_capping_domain_ctx_table = fwk_mm_calloc( element_count, sizeof(struct mod_scmi_power_capping_domain_context)); diff --git a/module/scmi_power_capping/test/mod_scmi_power_capping_unit_test.c b/module/scmi_power_capping/test/mod_scmi_power_capping_unit_test.c index c574a187e20bd6a200161e6e3eae8a59e0d6ad03..2164f148e71acb4b56194e089a1664f7f7fec632 100644 --- a/module/scmi_power_capping/test/mod_scmi_power_capping_unit_test.c +++ b/module/scmi_power_capping/test/mod_scmi_power_capping_unit_test.c @@ -50,17 +50,17 @@ void utest_scmi_power_capping_init_success(void) struct mod_scmi_power_capping_domain_context *domain_table = (struct mod_scmi_power_capping_domain_context *)__LINE__; - struct mod_scmi_power_capping_context expected_ctx = { 0 }; - - expected_ctx.power_capping_domain_ctx_table = domain_table, - expected_ctx.domain_count = domain_count, - fwk_mm_calloc_ExpectAndReturn( domain_count, sizeof(struct mod_scmi_power_capping_domain_context), domain_table); #ifdef BUILD_HAS_SCMI_POWER_CAPPING_STD_COMMANDS + struct mod_scmi_power_capping_context expected_ctx = { 0 }; + + expected_ctx.power_capping_domain_ctx_table = domain_table, + expected_ctx.domain_count = domain_count, + pcapping_protocol_init_ExpectWithArray(&expected_ctx, 1); #endif