Skip to content

Commit

Permalink
Merge branch 'feature/run_stack_smash_check_in_ci' into 'master'
Browse files Browse the repository at this point in the history
system: enable stack smashing protection test

Closes IDF-6612

See merge request espressif/esp-idf!22894
  • Loading branch information
zikalino committed Mar 24, 2023
2 parents 8d90249 + 783d8c2 commit bd4995c
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,10 @@ static void recur_and_smash(void)
}
}


/* Test is marked as ignore, since we cannot run it as a regular unity test case
Instead we run it in a seperate test case and check output with pytest,
see test_stack_smash_protection in the pytest file.
*/
TEST_CASE("stack smashing protection", "[stack_check] [ignore]")
{
recur_and_smash();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,3 +16,17 @@
)
def test_esp_system(dut: Dut) -> None:
dut.run_all_single_board_cases()


@pytest.mark.generic
@pytest.mark.parametrize(
'config',
[
pytest.param('default', marks=[pytest.mark.supported_targets]),
]
)
def test_stack_smash_protection(dut: Dut) -> None:
dut.expect_exact('Press ENTER to see the list of tests')
dut.write('"stack smashing protection"')
dut.expect_exact('Stack smashing protect failure!')
dut.expect_exact('Rebooting...')
Original file line number Diff line number Diff line change
@@ -1 +1,3 @@
# Default configuration
# Used for testing stack smashing protection
CONFIG_COMPILER_STACK_CHECK=y

0 comments on commit bd4995c

Please sign in to comment.