diff options
| author | Eric Wong <normalperson@yhbt.net> | 2008-01-26 22:16:46 +0000 |
|---|---|---|
| committer | Eric Wong <normalperson@yhbt.net> | 2008-01-26 22:16:46 +0000 |
| commit | eda485afab60e0f4913ccd8ae7b886d081356b1f (patch) | |
| tree | 5396cc0c2fb4975b59c652ab91801613ffaa5049 /src/player.c | |
| parent | 4c16b2a4fbb11803c29e0686873589af8262dfbb (diff) | |
| download | mpd-eda485afab60e0f4913ccd8ae7b886d081356b1f.tar.gz mpd-eda485afab60e0f4913ccd8ae7b886d081356b1f.tar.xz mpd-eda485afab60e0f4913ccd8ae7b886d081356b1f.zip | |
removing debug messages from signal handlers
As unfortunate as it is to remove such useful debugging messages, it's
necessary to fix a potential deadlock with signal handling. A bunch of
functions the debug functions call aren't safe to call from a signal
handler. There are some alternate solutions, but they're neither pretty
nor simple. So just remove them entirely for now.
From: jat, r6828 in trunk
git-svn-id: https://svn.musicpd.org/mpd/branches/branch-0.13.0-fixes@7160 09075e82-0dd4-0310-85a5-a0d7c8717e4f
Diffstat (limited to '')
| -rw-r--r-- | src/player.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/src/player.c b/src/player.c index 7c92d088e..6d9f1e693 100644 --- a/src/player.c +++ b/src/player.c @@ -85,6 +85,7 @@ void player_sigChldHandler(int pid, int status) { if (player_pid == pid) { + /* DEBUG("SIGCHLD caused by player process\n"); if (WIFSIGNALED(status) && WTERMSIG(status) != SIGTERM && @@ -93,17 +94,20 @@ void player_sigChldHandler(int pid, int status) ERROR("player process died from signal: %i\n", WTERMSIG(status)); } + */ resetPlayer(); } else if (pid == getPlayerData()->playerControl.decode_pid && player_pid <= 0) { + /* if (WIFSIGNALED(status) && WTERMSIG(status) != SIGTERM) { ERROR("(caught by master parent) " "decode process died from a " "non-TERM signal: %i\n", WTERMSIG(status)); } + */ getPlayerData()->playerControl.decode_pid = 0; } } |