Index: App/Services/SystemCommMessages.c =================================================================== diff -u -r0e1748b0cdc3df627c5f7cf685ff0f4c7bc63d00 -r54ab19a7ac15f6ef2628c96661a7d6cc35ea12d0 --- App/Services/SystemCommMessages.c (.../SystemCommMessages.c) (revision 0e1748b0cdc3df627c5f7cf685ff0f4c7bc63d00) +++ App/Services/SystemCommMessages.c (.../SystemCommMessages.c) (revision 54ab19a7ac15f6ef2628c96661a7d6cc35ea12d0) @@ -264,14 +264,14 @@ void handleTestOffButtonStateOverrideRequest( MESSAGE_T *message ) { TEST_OVERRIDE_CARGO_T cargo; - BOOL result; + BOOL result = FALSE; // verify cargo length - if ( SIZE_OF_TEST_OVERRIDE_CARGO_T == message->hdr.cargoLen ) + if ( sizeof(TEST_OVERRIDE_CARGO_T) == message->hdr.cargoLen ) { memcpy( &cargo, message->cargo, sizeof(TEST_OVERRIDE_CARGO_T) ); - if ( FALSE == (BOOL)(cargo.reset) ) + if ( FALSE == cargo.reset ) { result = testSetOffButtonStateOverride( (BUTTON_STATE_T)(cargo.state) ); } @@ -297,14 +297,14 @@ void handleTestStopButtonStateOverrideRequest( MESSAGE_T *message ) { TEST_OVERRIDE_CARGO_T cargo; - BOOL result; + BOOL result = FALSE; // verify cargo length - if ( SIZE_OF_TEST_OVERRIDE_CARGO_T == message->hdr.cargoLen ) + if ( sizeof(TEST_OVERRIDE_CARGO_T) == message->hdr.cargoLen ) { memcpy( &cargo, message->cargo, sizeof(TEST_OVERRIDE_CARGO_T) ); - if ( FALSE == (BOOL)(cargo.reset) ) + if ( FALSE == cargo.reset ) { result = testSetStopButtonStateOverride( (BUTTON_STATE_T)(cargo.state) ); } @@ -330,14 +330,14 @@ void handleTestAlarmLampPatternOverrideRequest( MESSAGE_T *message ) { TEST_OVERRIDE_CARGO_T cargo; - BOOL result; + BOOL result = FALSE; // verify cargo length - if ( SIZE_OF_TEST_OVERRIDE_CARGO_T == message->hdr.cargoLen ) + if ( sizeof(TEST_OVERRIDE_CARGO_T) == message->hdr.cargoLen ) { memcpy( &cargo, message->cargo, sizeof(TEST_OVERRIDE_CARGO_T) ); - if ( FALSE == (BOOL)(cargo.reset) ) + if ( FALSE == cargo.reset ) { result = testSetCurrentLampPatternOverride( (LAMP_PATTERN_T)(cargo.state) ); } @@ -363,14 +363,14 @@ void handleTestWatchdogCheckInStateOverrideRequest( MESSAGE_T *message ) { TEST_OVERRIDE_ARRAY_CARGO_T cargo; - BOOL result; + BOOL result = FALSE; // verify cargo length - if ( SIZE_OF_TEST_OVERRIDE_ARRAY_CARGO_T == message->hdr.cargoLen ) + if ( sizeof(TEST_OVERRIDE_ARRAY_CARGO_T) == message->hdr.cargoLen ) { memcpy( &cargo, message->cargo, sizeof(TEST_OVERRIDE_ARRAY_CARGO_T) ); - if ( FALSE == (BOOL)(cargo.reset) ) + if ( FALSE == cargo.reset ) { result = testSetWatchdogTaskCheckInOverride( cargo.index, (BOOL)(cargo.state) ); }