aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMatthias Andree <matthias.andree@gmx.de>2006-07-31 23:27:50 +0000
committerMatthias Andree <matthias.andree@gmx.de>2006-07-31 23:27:50 +0000
commit3a27df341c39eaf7b2181d3f036c68b080e07a74 (patch)
treeb5f99f1117fd6e1799e64cd6fefd5c828835b7d0
parent5b9086a2c5040711cb9a5feb7158146af42b70ab (diff)
downloadfetchmail-3a27df341c39eaf7b2181d3f036c68b080e07a74.tar.gz
fetchmail-3a27df341c39eaf7b2181d3f036c68b080e07a74.tar.bz2
fetchmail-3a27df341c39eaf7b2181d3f036c68b080e07a74.zip
Fix SIGHUP behavior, broken since 5.9.13 (r3692).
svn path=/branches/BRANCH_6-3/; revision=4871
-rw-r--r--NEWS7
-rw-r--r--fetchmail.c2
2 files changed, 8 insertions, 1 deletions
diff --git a/NEWS b/NEWS
index fc00b061..ff107336 100644
--- a/NEWS
+++ b/NEWS
@@ -32,6 +32,8 @@ change. MA = Matthias Andree, ESR = Eric S. Raymond, RF = Rob Funk.)
* The "protocol auto" default inside fetchmail may be removed from a future
fetchmail release. Explicit configuration of the protocol is recommended.
* Kerberos IV support may be removed from a future fetchmail release.
+* SIGHUP wakeup may be removed from a future fetchmail release and cause it
+ to terminate.
--------------------------------------------------------------------------------
@@ -89,6 +91,11 @@ fetchmail 6.3.5 (not yet released):
* Avoid compiling empty files, to avoid diagnostics from strict compilers.
* If the lockfile ends before the process ID, treat it as stale and unlink it.
Reported by Justin Pryzby, Debian Bug #376603.
+* SIGHUP wake-up behavior was broken since 5.9.13's Cygwin changes, in that for
+ non-root users, SIGHUP would abort the first poll and subsequently interfere
+ with new polls, and SIGHUP would be ignored for root users. SIGHUP now matches
+ documented behavior. SIGUSR1 has always been a wakeup signal for both root
+ (undocumented) and non-root users. See also the deprecation warning above.
# CHANGES:
* Rename all fetchmail-internal lock_* functions to fm_lock_*. Obsoletes
diff --git a/fetchmail.c b/fetchmail.c
index 1c895a0a..ad351dff 100644
--- a/fetchmail.c
+++ b/fetchmail.c
@@ -808,7 +808,7 @@ int main(int argc, char **argv)
* for people who think all system daemons wake up on SIGHUP.
*/
set_signal_handler(SIGUSR1, donothing);
- if (getuid() != ROOT_UID)
+ if (getuid() == ROOT_UID)
set_signal_handler(SIGHUP, donothing);
/*