diff options
| author | Dave Reisner <dreisner@archlinux.org> | 2011-08-19 13:42:53 -0400 |
|---|---|---|
| committer | Dan McGee <dan@archlinux.org> | 2011-08-22 09:18:00 -0500 |
| commit | d64c409913720ed3fbf3713de621bbef4ea3bb49 (patch) | |
| tree | 64e7d935b21e9f3de4788c361cefc32a0b5e649e | |
| parent | 592ed13bce83afeeb8deff5a81dc782fdc32ac11 (diff) | |
dload: add open_mode to payload struct
This is a precursor to a following patch which will move the setting of
options to a separate function. With the open mode as part of the
struct, we can avoid modifying stack allocated variables.
Signed-off-by: Dave Reisner <dreisner@archlinux.org>
Signed-off-by: Dan McGee <dan@archlinux.org>
| -rw-r--r-- | lib/libalpm/dload.c | 9 | ||||
| -rw-r--r-- | lib/libalpm/dload.h | 1 |
2 files changed, 6 insertions, 4 deletions
diff --git a/lib/libalpm/dload.c b/lib/libalpm/dload.c index ff3ac6df..14b80557 100644 --- a/lib/libalpm/dload.c +++ b/lib/libalpm/dload.c @@ -182,7 +182,6 @@ static int curl_download_internal(struct dload_payload *payload, int ret = -1; FILE *localf = NULL; const char *useragent; - const char *open_mode = "wb"; char *effective_url; /* RFC1123 states applications should support this length */ char hostname[256]; @@ -195,6 +194,7 @@ static int curl_download_internal(struct dload_payload *payload, alpm_handle_t *handle = payload->handle; handle->pm_errno = 0; + payload->tempfile_openmode = "wb"; if(!payload->remote_name) { payload->remote_name = get_filename(payload->fileurl); } @@ -220,7 +220,8 @@ static int curl_download_internal(struct dload_payload *payload, /* create a random filename, which is opened with O_EXCL */ snprintf(randpath, PATH_MAX, "%salpmtmp.XXXXXX", localpath); - if((fd = mkstemp(randpath)) == -1 || !(localf = fdopen(fd, open_mode))) { + if((fd = mkstemp(randpath)) == -1 || + !(localf = fdopen(fd, payload->tempfile_openmode))) { unlink(randpath); close(fd); _alpm_log(handle, ALPM_LOG_ERROR, @@ -267,14 +268,14 @@ static int curl_download_internal(struct dload_payload *payload, curl_easy_setopt(handle->curl, CURLOPT_TIMEVALUE, (long)st.st_mtime); } else if(stat(payload->tempfile_name, &st) == 0 && payload->allow_resume) { /* a previous partial download exists, resume from end of file. */ - open_mode = "ab"; + payload->tempfile_openmode = "ab"; curl_easy_setopt(handle->curl, CURLOPT_RESUME_FROM, (long)st.st_size); _alpm_log(handle, ALPM_LOG_DEBUG, "tempfile found, attempting continuation\n"); payload->initial_size = (double)st.st_size; } if(localf == NULL) { - localf = fopen(payload->tempfile_name, open_mode); + localf = fopen(payload->tempfile_name, payload->tempfile_openmode); if(localf == NULL) { goto cleanup; } diff --git a/lib/libalpm/dload.h b/lib/libalpm/dload.h index cc6cc240..13034f7b 100644 --- a/lib/libalpm/dload.h +++ b/lib/libalpm/dload.h @@ -28,6 +28,7 @@ struct dload_payload { alpm_handle_t *handle; const char *remote_name; + const char *tempfile_openmode; char *tempfile_name; char *destfile_name; char *content_disp_name; |