* tests/test-sigsegv-catch-segv1.c: * tests/test-sigsegv-catch-stackoverflow1.c: * tests/test-sigsegv-catch-stackoverflow2.c: Declare some functions and variables static, to pacify GCC when warning about external functions missing declarations. --- ChangeLog | 7 +++++++ tests/test-sigsegv-catch-segv1.c | 4 ++-- tests/test-sigsegv-catch-stackoverflow1.c | 16 ++++++++-------- tests/test-sigsegv-catch-stackoverflow2.c | 18 +++++++++--------- 4 files changed, 26 insertions(+), 19 deletions(-)
diff --git a/ChangeLog b/ChangeLog index f72226809..f908d4ef5 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,5 +1,12 @@ 2021-08-01 Paul Eggert <egg...@cs.ucla.edu> + sigsegv-tests: make more things static + * tests/test-sigsegv-catch-segv1.c: + * tests/test-sigsegv-catch-stackoverflow1.c: + * tests/test-sigsegv-catch-stackoverflow2.c: + Declare some functions and variables static, to pacify GCC when + warning about external functions missing declarations. + maint: improve -fanalyzer malloc checking * lib/backup-internal.h, lib/backupfile.h: * lib/canonicalize.h, lib/dfa.h, lib/dirname.h, lib/exclude.h: diff --git a/tests/test-sigsegv-catch-segv1.c b/tests/test-sigsegv-catch-segv1.c index 68f65c55c..4f6989efe 100644 --- a/tests/test-sigsegv-catch-segv1.c +++ b/tests/test-sigsegv-catch-segv1.c @@ -40,7 +40,7 @@ uintptr_t page; volatile int handler_called = 0; -int +static int handler (void *fault_address, int serious) { handler_called++; @@ -54,7 +54,7 @@ handler (void *fault_address, int serious) return 0; } -void +static void crasher (uintptr_t p) { *(volatile int *) (p + 0x678) = 42; diff --git a/tests/test-sigsegv-catch-stackoverflow1.c b/tests/test-sigsegv-catch-stackoverflow1.c index 2f1e6f487..3c229975b 100644 --- a/tests/test-sigsegv-catch-stackoverflow1.c +++ b/tests/test-sigsegv-catch-stackoverflow1.c @@ -44,13 +44,13 @@ # endif # include "altstack-util.h" -jmp_buf mainloop; -sigset_t mainsigset; +static jmp_buf mainloop; +static sigset_t mainsigset; -volatile int pass = 0; +static volatile int pass = 0; -volatile char *stack_lower_bound; -volatile char *stack_upper_bound; +static volatile char *stack_lower_bound; +static volatile char *stack_upper_bound; static void stackoverflow_handler_continuation (void *arg1, void *arg2, void *arg3) @@ -59,7 +59,7 @@ stackoverflow_handler_continuation (void *arg1, void *arg2, void *arg3) longjmp (mainloop, arg); } -void +static void stackoverflow_handler (int emergency, stackoverflow_context_t scp) { char dummy; @@ -73,7 +73,7 @@ stackoverflow_handler (int emergency, stackoverflow_context_t scp) (void *) (long) (emergency ? -1 : pass), NULL, NULL); } -volatile int * +static volatile int * recurse_1 (int n, volatile int *p) { if (n < INT_MAX) @@ -81,7 +81,7 @@ recurse_1 (int n, volatile int *p) return p; } -int +static int recurse (volatile int n) { return *recurse_1 (n, &n); diff --git a/tests/test-sigsegv-catch-stackoverflow2.c b/tests/test-sigsegv-catch-stackoverflow2.c index 5914e3250..ea79b96e7 100644 --- a/tests/test-sigsegv-catch-stackoverflow2.c +++ b/tests/test-sigsegv-catch-stackoverflow2.c @@ -46,12 +46,12 @@ # endif # include "altstack-util.h" -jmp_buf mainloop; -sigset_t mainsigset; +static jmp_buf mainloop; +static sigset_t mainsigset; -volatile int pass = 0; -uintptr_t page; -volatile int *null_pointer_to_volatile_int /* = NULL */; +static volatile int pass = 0; +static uintptr_t page; +static volatile int *null_pointer_to_volatile_int /* = NULL */; static void stackoverflow_handler_continuation (void *arg1, void *arg2, void *arg3) @@ -60,7 +60,7 @@ stackoverflow_handler_continuation (void *arg1, void *arg2, void *arg3) longjmp (mainloop, arg); } -void +static void stackoverflow_handler (int emergency, stackoverflow_context_t scp) { pass++; @@ -76,7 +76,7 @@ stackoverflow_handler (int emergency, stackoverflow_context_t scp) (void *) (long) (emergency ? -1 : pass), NULL, NULL); } -int +static int sigsegv_handler (void *address, int emergency) { /* This test is necessary to distinguish stack overflow and SIGSEGV. */ @@ -96,7 +96,7 @@ sigsegv_handler (void *address, int emergency) (void *) (long) pass, NULL, NULL); } -volatile int * +static volatile int * recurse_1 (int n, volatile int *p) { if (n < INT_MAX) @@ -104,7 +104,7 @@ recurse_1 (int n, volatile int *p) return p; } -int +static int recurse (volatile int n) { return *recurse_1 (n, &n); -- 2.31.1