Some cleanup for CppcheckSignalHandler, also some cosmetics for the output
This commit is contained in:
parent
6fe2f8b01b
commit
08c209d4d9
|
@ -211,6 +211,19 @@ size_t GetArrayLength(const T(&)[size])
|
|||
return size;
|
||||
}
|
||||
|
||||
/*
|
||||
* Simple mapping
|
||||
*/
|
||||
static const char *signal_name(int signo)
|
||||
{
|
||||
for (size_t s=0; s<GetArrayLength(listofsignals); ++s) {
|
||||
if (listofsignals[s].signalnumber==signo)
|
||||
return listofsignals[s].signalname;
|
||||
}
|
||||
return "";
|
||||
}
|
||||
|
||||
|
||||
// 32 vs. 64bit
|
||||
#define ADDRESSDISPLAYLENGTH ((sizeof(long)==8)?12:8)
|
||||
|
||||
|
@ -229,7 +242,6 @@ static void print_stacktrace(FILE* f, bool demangling)
|
|||
const int offset=3; // the first two entries are simply within our own exception handling code, third is within libc
|
||||
for (int i = offset; i < depth; ++i) {
|
||||
const char * const symbol = symbolstrings[i];
|
||||
//fprintf(f, "\"%s\"\n", symbol);
|
||||
char * realname = nullptr;
|
||||
const char * const firstBracketName = strchr(symbol, '(');
|
||||
const char * const firstBracketAddress = strchr(symbol, '[');
|
||||
|
@ -271,55 +283,38 @@ static void print_stacktrace(FILE* f, bool demangling)
|
|||
#endif
|
||||
}
|
||||
|
||||
/*
|
||||
* Simple mapping
|
||||
*/
|
||||
static const char *signal_name(int signo)
|
||||
{
|
||||
for (size_t s=0; s<GetArrayLength(listofsignals); ++s) {
|
||||
if (listofsignals[s].signalnumber==signo)
|
||||
return listofsignals[s].signalname;
|
||||
}
|
||||
return "";
|
||||
}
|
||||
|
||||
/*
|
||||
* Entry pointer for signal handlers
|
||||
*/
|
||||
static void CppcheckSignalHandler(int signo, siginfo_t * info, void * /*context*/)
|
||||
{
|
||||
const char * const signame=signal_name(signo);
|
||||
const char * const signame = signal_name(signo);
|
||||
const char * const sigtext = strsignal(signo);
|
||||
bool bPrintCallstack=true;
|
||||
FILE* f=stderr;
|
||||
fputs("Internal error: cppcheck received signal ", f);
|
||||
fputs(signame, f);
|
||||
fputs(", ", f);
|
||||
fputs(sigtext, f);
|
||||
switch (signo) {
|
||||
case SIGILL:
|
||||
fprintf(f, "Internal error (caught signal %d=%s at 0x%p)\n",
|
||||
signo, signame, info->si_addr);
|
||||
fprintf(f, " (at 0x%p).\n",
|
||||
info->si_addr);
|
||||
break;
|
||||
case SIGFPE:
|
||||
fprintf(f, "Internal error (caught signal %d=%s at 0x%p)\n",
|
||||
signo, signame, info->si_addr);
|
||||
fprintf(f, " (at 0x%p).\n",
|
||||
info->si_addr);
|
||||
break;
|
||||
case SIGSEGV:
|
||||
fprintf(f, "Internal error (caught signal %d=%s at 0x%p)\n",
|
||||
signo, signame, info->si_addr);
|
||||
fprintf(f, " (at 0x%p).\n",
|
||||
info->si_addr);
|
||||
break;
|
||||
/*
|
||||
case SIGBUS:
|
||||
fprintf(f, "Internal error (caught signal %d=%s at 0x%p)\n",
|
||||
signo, signame, info->si_addr);
|
||||
break;
|
||||
case SIGTRAP:
|
||||
fprintf(f, "Internal error (caught signal %d=%s at 0x%p)\n",
|
||||
signo, signame, info->si_addr);
|
||||
break;
|
||||
*/
|
||||
case SIGINT:
|
||||
bPrintCallstack=false;
|
||||
fprintf(f, ".\n");
|
||||
break;
|
||||
default:
|
||||
fprintf(f, "Internal error (caught signal %d)\n",
|
||||
signo);
|
||||
fputs(".\n", f);
|
||||
break;
|
||||
}
|
||||
if (bPrintCallstack) {
|
||||
|
|
Loading…
Reference in New Issue