p11-glue / p11-kit / 16698480972 / 1

69%

master: 69%

push

github

Fix running of test teardown on failure

The variable `setup` is set to 1 between `setjmp` and call to `longjmp` in
`p11_test_fail`, so its value is indeterminate upon the second return from
`setjmp`. If the variable is stored in a register, `setjmp` resets it to the
value before the call (0), and the teardown function is not executed. Fix
by making the variable `volatile`.

This bug can be observed, for example, by modifying `test-argv` to fail and
building with LeakSanitizer and optimizations.

13960 of 26997 branches covered (51.71%)

Branch coverage included in aggregate %.

29471 of 35880 relevant lines covered (82.14%)

2646.56 hits per line