github-mirror/fish-shell
github-mirror/fish-shell
2
0
Fork 0
mirror of https://github.com/fish-shell/fish-shell.git synced 2025-02-21 08:10:24 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Fix uninitialized sigaction.sa_flags valgrind error

Browse Source 
Valgrind warns that the sometimes uninitialized sigaction.sa_flags field
is sometimes used when passed to the signal handler.

This patch explicitly zeros out the sigaction.sa_flags field at creation
time.
This commit is contained in:
Mahmoud Al-Qudsi 2017-08-26 19:11:39 -05:00
parent 2d08da79e9
commit e67ab57ca3
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/signal.cpp
View File

@ -270,6 +270,8 @@ void signal_reset_handlers() {
static void set_interactive_handlers() {
struct sigaction act, oact;
act.sa_flags = 0;
oact.sa_flags = 0;
sigemptyset(&act.sa_mask);
// Interactive mode. Ignore interactive signals. We are a shell, we know what is best for
@ -315,6 +317,7 @@ static void set_interactive_handlers() {
static void set_non_interactive_handlers() {
struct sigaction act;
act.sa_flags = 0;
sigemptyset(&act.sa_mask);
// Non-interactive. Ignore interrupt, check exit status of processes to determine result
@ -327,6 +330,7 @@ static void set_non_interactive_handlers() {
/// Sets up appropriate signal handlers.
void signal_set_handlers() {
struct sigaction act;
act.sa_flags = 0;
sigemptyset(&act.sa_mask);
// Ignore SIGPIPE. We'll detect failed writes and deal with them appropriately. We don't want
@ -358,6 +362,7 @@ void signal_handle(int sig, int do_handle) {
(sig == SIGTTOU) || (sig == SIGCHLD))
return;
act.sa_flags = 0;
sigemptyset(&act.sa_mask);
if (do_handle) {
act.sa_flags = SA_SIGINFO;