Add SIGABRT monitoring support to segfault_analyzer#51
Merged
Conversation
- Extended segfault_analyzer to monitor both SIGSEGV and SIGABRT signals - Added support for detecting abort signals including stack smashing errors - Updated data structures to include signal_number and event_type fields - Modified BPF program to capture SIGABRT events with stack traces - Enhanced output to display abort events with diagnostic information - Stack traces for SIGABRT marked as potentially unreliable due to corruption - Added test_stack_smashing.c test program compiled with -fstack-protector-strong - Updated Makefile to build stack smashing test program - Updated requirements.md and design.md to reflect simplified abort handling - Users can identify stack smashing by examining stack trace for __stack_chk_fail Key features: - Captures complete stack traces for SIGABRT events - Provides instruction pointer, registers, and VMA information - Marks abort stack traces as potentially unreliable - Simplified approach: all SIGABRT treated as 'ABORT' events - Stack trace reveals abort cause (e.g., __stack_chk_fail for stack smashing)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What's changed
Key features: