I write a signal handler in my program, but in this file structure I get a segmentation fault. But if everything is all in one file, then everything works correctly. As I understand it, it can’t access the return address. How can I fix it?
"test.h"
#ifndef TEST_H
#define TEST_H
#include <string.h>
#include <setjmp.h>
#include <signal.h>
static jmp_buf g_env;
void sig_handler(int sig);
endif TEST_H
"test.c"
#include "test.h"
void sig_handler(int sig) {
psignal(sig, "Signal");
siglongjmp(g_env, 1);
}
"main.c"
#include "test.h"
int main(void) {
struct sigaction act;
memset(&act, 0, sizeof(act));
act.sa_handler = sig_handler;
sigemptyset(&act.sa_mask);
sigaddset(&act.sa_mask, SIGSEGV);
sigaction(SIGSEGV, &act, 0);
if (!sigsetjmp(g_env, 1)) {
raise(SIGSEGV);
}
return (1);
}
"Result:"
Signal: Segmentation fault
Segmentation fault (core dumped)
"Valgrind:"
==44698== Command: ./a.out
==44698==
Signal: Segmentation fault
==44698== Warning: client switching stacks? SP change: 0x1ffeffee08 --> 0xf7c1f413060a2966
==44698== to suppress, use: --max-stackframe=593925450715481250 or greater
==44698== Jump to the invalid address stated on the next line
==44698== at 0xF7C1F413060A2966: ???
==44698== Address 0xf7c1f413060a2966 is not stack'd, malloc'd or (recently) free'd
==44698==
==44698==
==44698== Process terminating with default action of signal 11 (SIGSEGV): dumping core
==44698== Bad permissions for mapped region at address 0xF7C1F413060A2966
==44698== at 0xF7C1F413060A2966: ???
==44698== Invalid write of size 8
==44698== at 0x482F120: _vgnU_freeres (vg_preloaded.c:59)
==44698== Address 0xf7c1f413060a295e is not stack'd, malloc'd or (recently) free'd
==44698==
==44698==
==44698== Process terminating with default action of signal 11 (SIGSEGV)
==44698== General Protection Fault
==44698== at 0x482F120: _vgnU_freeres (vg_preloaded.c:59)
It's because you have different instances of g_env
- each translation unit gets a different g_env
. As a result, the g_env
buffer used in sigsetjmp
isn't same as that's used by siglongjmp
.
Just drop the static
from:
static jmp_buf g_env;
Now, there'll be just one global instance of g_env
.