From c1d6cec2e277eea7affa43601086c523311b58c0 Mon Sep 17 00:00:00 2001 From: Andrew Gregory Date: Sat, 11 Oct 2014 04:08:20 -0400 Subject: sync.c: download packages before starting commit download_files sets the transaction state to STATE_DOWNLOADING. Modifying the state after it has already been set to STATE_COMMITTING created a brief window where SIGINT would fail to interrupt the process and caused interrupted downloads to result in a 'transaction started' message in the log with no matching transaction end. Signed-off-by: Andrew Gregory --- lib/libalpm/sync.c | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) (limited to 'lib/libalpm/sync.c') diff --git a/lib/libalpm/sync.c b/lib/libalpm/sync.c index 9fe4eff5..c513b6b5 100644 --- a/lib/libalpm/sync.c +++ b/lib/libalpm/sync.c @@ -1263,13 +1263,12 @@ static int load_packages(alpm_handle_t *handle, alpm_list_t **data, return 0; } -int _alpm_sync_commit(alpm_handle_t *handle, alpm_list_t **data) +int _alpm_sync_load(alpm_handle_t *handle, alpm_list_t **data) { alpm_list_t *i, *deltas = NULL; size_t total = 0; uint64_t total_bytes = 0; alpm_trans_t *trans = handle->trans; - alpm_event_t event; if(download_files(handle, &deltas)) { alpm_list_free(deltas); @@ -1318,7 +1317,13 @@ int _alpm_sync_commit(alpm_handle_t *handle, alpm_list_t **data) return -1; } - trans->state = STATE_COMMITING; + return 0; +} + +int _alpm_sync_commit(alpm_handle_t *handle, alpm_list_t **data) +{ + alpm_trans_t *trans = handle->trans; + alpm_event_t event; /* fileconflict check */ if(!(trans->flags & ALPM_TRANS_FLAG_DBONLY)) { -- cgit v1.2.3-24-g4f1b