csegmentation-faultsignalssetjmp

Why do I get a segmentation fault in this code with a signal handler?


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)

Solution

  • 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.