summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDave Reisner <dreisner@archlinux.org>2011-08-19 13:51:47 -0400
committerDan McGee <dan@archlinux.org>2011-08-22 09:23:14 -0500
commitc65cea0dcbc2777bd72b13d5a69f89d99fa0b0d5 (patch)
tree9058a0b375f25c79cb82264037ec425ab3d1294d
parent10cbfc956c69db0cc2826e5a9c6b5b5df2a46326 (diff)
downloadpacman-c65cea0dcbc2777bd72b13d5a69f89d99fa0b0d5.tar.xz
dload: move (un)masking of signals to separate functions
Signed-off-by: Dave Reisner <dreisner@archlinux.org> Signed-off-by: Dan McGee <dan@archlinux.org>
-rw-r--r--lib/libalpm/dload.c41
1 files changed, 24 insertions, 17 deletions
diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c
index b8c0f1e6..b99e7d91 100644
--- a/lib/libalpm/dload.c
+++ b/lib/libalpm/dload.c
@@ -66,8 +66,6 @@ static char *get_fullpath(const char *path, const char *filename,
return filepath;
}
-enum sighandlers { OLD = 0, NEW = 1 };
-
static int dload_interrupted;
static void inthandler(int UNUSED signum)
{
@@ -222,6 +220,25 @@ static void curl_set_handle_opts(struct dload_payload *payload,
}
}
+static struct sigaction mask_signal(int signal, void (*handler)(int))
+{
+ struct sigaction newaction, origaction;
+
+ newaction.sa_handler = handler;
+ sigemptyset(&newaction.sa_mask);
+ newaction.sa_flags = 0;
+
+ sigaction(signal, NULL, &origaction);
+ sigaction(signal, &newaction, NULL);
+
+ return origaction;
+}
+
+static void unmask_signal(int signal, struct sigaction sa)
+{
+ sigaction(signal, &sa, NULL);
+}
+
static int curl_download_internal(struct dload_payload *payload,
const char *localpath, char **final_file)
{
@@ -234,7 +251,7 @@ static int curl_download_internal(struct dload_payload *payload,
struct stat st;
long timecond, respcode = 0, remote_time = -1;
double remote_size, bytes_dl;
- struct sigaction sig_pipe[2], sig_int[2];
+ struct sigaction orig_sig_pipe, orig_sig_int;
/* shortcut to our handle within the payload */
alpm_handle_t *handle = payload->handle;
handle->pm_errno = 0;
@@ -291,18 +308,8 @@ static int curl_download_internal(struct dload_payload *payload,
/* ignore any SIGPIPE signals- these may occur if our FTP socket dies or
* something along those lines. Store the old signal handler first. */
- sig_pipe[NEW].sa_handler = SIG_IGN;
- sigemptyset(&sig_pipe[NEW].sa_mask);
- sig_pipe[NEW].sa_flags = 0;
- sigaction(SIGPIPE, NULL, &sig_pipe[OLD]);
- sigaction(SIGPIPE, &sig_pipe[NEW], NULL);
-
- dload_interrupted = 0;
- sig_int[NEW].sa_handler = &inthandler;
- sigemptyset(&sig_int[NEW].sa_mask);
- sig_int[NEW].sa_flags = 0;
- sigaction(SIGINT, NULL, &sig_int[OLD]);
- sigaction(SIGINT, &sig_int[NEW], NULL);
+ orig_sig_pipe = mask_signal(SIGPIPE, SIG_IGN);
+ orig_sig_int = mask_signal(SIGINT, &inthandler);
/* Progress 0 - initialize */
prevprogress = 0;
@@ -418,8 +425,8 @@ cleanup:
}
/* restore the old signal handlers */
- sigaction(SIGINT, &sig_int[OLD], NULL);
- sigaction(SIGPIPE, &sig_pipe[OLD], NULL);
+ unmask_signal(SIGINT, orig_sig_int);
+ unmask_signal(SIGPIPE, orig_sig_pipe);
/* if we were interrupted, trip the old handler */
if(dload_interrupted) {
raise(SIGINT);