moparisthebest/pacman
1
0
Fork 0
mirror of https://github.com/moparisthebest/pacman synced 2024-11-13 21:05:05 -05:00
Code Issues Releases Wiki Activity

Add _alpm_process_siglist() logic to failed package validation

Browse Source 
This moves the result processing out of the validation check loop itself
and into a new loop. Errors will be presented to the user one-by-one
after we fully complete the validation loop, so they no longer overlap
the progress bar.

Unlike the database validation, we may have several errors to process in
sequence here, so we use a function-scoped struct to track all the
necessary information between seeing an error and asking the user about
it.

The older prompt_to_delete() callback logic is still kept, but only for
checksum failures. It is debatable whether we should do this at all or
just delegate said actions to the user.

Signed-off-by: Dan McGee <dan@archlinux.org>
This commit is contained in:
Dan McGee 2011-09-20 16:52:18 -05:00
parent 0a4a5cea97
commit 4849a4596d
2 changed files with 59 additions and 24 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
lib/libalpm/be_package.c
View File

@ -287,6 +287,7 @@ int _alpm_pkg_validate_internal(alpm_handle_t *handle,
alpm_siglist_t **sigdata) alpm_siglist_t **sigdata)
{ {
int has_sig; int has_sig;
handle->pm_errno = 0;
if(pkgfile == NULL || strlen(pkgfile) == 0) { if(pkgfile == NULL || strlen(pkgfile) == 0) {
RET_ERR(handle, ALPM_ERR_WRONG_ARGS, -1); RET_ERR(handle, ALPM_ERR_WRONG_ARGS, -1);

82
lib/libalpm/sync.c
View File

@ -909,42 +909,47 @@ static int download_files(alpm_handle_t *handle, alpm_list_t **deltas)
return errors; return errors;
} }
static int check_validity(alpm_handle_t *handle, alpm_list_t **data, static int check_validity(alpm_handle_t *handle,
size_t total, size_t total_bytes) size_t total, size_t total_bytes)
{ {
struct validity {
alpm_pkg_t *pkg;
char *path;
alpm_siglist_t *siglist;
alpm_siglevel_t level;
enum _alpm_errno_t error;
};
size_t current = 0, current_bytes = 0; size_t current = 0, current_bytes = 0;
int errors = 0; alpm_list_t *i, *errors = NULL;
alpm_list_t *i;
/* Check integrity of packages */ /* Check integrity of packages */
EVENT(handle, ALPM_EVENT_INTEGRITY_START, NULL, NULL); EVENT(handle, ALPM_EVENT_INTEGRITY_START, NULL, NULL);
for(i = handle->trans->add; i; i = i->next, current++) { for(i = handle->trans->add; i; i = i->next, current++) {
alpm_pkg_t *spkg = i->data; struct validity v = { i->data, NULL, NULL, 0, 0 };
char *filepath;
alpm_siglevel_t level;
alpm_siglist_t *siglist = NULL;
int percent = (int)(((double)current_bytes / total_bytes) * 100); int percent = (int)(((double)current_bytes / total_bytes) * 100);
PROGRESS(handle, ALPM_PROGRESS_INTEGRITY_START, "", percent, PROGRESS(handle, ALPM_PROGRESS_INTEGRITY_START, "", percent,
total, current); total, current);
if(spkg->origin == PKG_FROM_FILE) { if(v.pkg->origin == PKG_FROM_FILE) {
continue; /* pkg_load() has been already called, this package is valid */ continue; /* pkg_load() has been already called, this package is valid */
} }
current_bytes += spkg->size; current_bytes += v.pkg->size;
filepath = _alpm_filecache_find(handle, spkg->filename); v.path = _alpm_filecache_find(handle, v.pkg->filename);
alpm_db_t *sdb = alpm_pkg_get_db(spkg); v.level = alpm_db_get_siglevel(alpm_pkg_get_db(v.pkg));
level = alpm_db_get_siglevel(sdb);
if(_alpm_pkg_validate_internal(handle, filepath, spkg, level, &siglist) == -1) { if(_alpm_pkg_validate_internal(handle, v.path, v.pkg,
prompt_to_delete(handle, filepath, handle->pm_errno); v.level, &v.siglist) == -1) {
errors++; v.error = handle->pm_errno;
*data = alpm_list_add(*data, strdup(spkg->filename)); struct validity *invalid = malloc(sizeof(struct validity));
memcpy(invalid, &v, sizeof(struct validity));
errors = alpm_list_add(errors, invalid);
} else {
alpm_siglist_cleanup(v.siglist);
free(v.siglist);
free(v.path);
} }
alpm_siglist_cleanup(siglist);
free(siglist);
free(filepath);
} }
PROGRESS(handle, ALPM_PROGRESS_INTEGRITY_START, "", 100, PROGRESS(handle, ALPM_PROGRESS_INTEGRITY_START, "", 100,
@ -952,10 +957,33 @@ static int check_validity(alpm_handle_t *handle, alpm_list_t **data,
EVENT(handle, ALPM_EVENT_INTEGRITY_DONE, NULL, NULL); EVENT(handle, ALPM_EVENT_INTEGRITY_DONE, NULL, NULL);
if(errors) { if(errors) {
if(!handle->pm_errno) { int tryagain = 0;
RET_ERR(handle, ALPM_ERR_PKG_INVALID, -1); for(i = errors; i; i = i->next) {
struct validity *v = i->data;
if(v->error == ALPM_ERR_PKG_INVALID_SIG) {
int retry = _alpm_process_siglist(handle, v->pkg->name, v->siglist,
v->level & ALPM_SIG_PACKAGE_OPTIONAL,
v->level & ALPM_SIG_PACKAGE_MARGINAL_OK,
v->level & ALPM_SIG_PACKAGE_UNKNOWN_OK);
tryagain += retry;
} else if(v->error == ALPM_ERR_PKG_INVALID_CHECKSUM) {
prompt_to_delete(handle, v->path, v->error);
}
alpm_siglist_cleanup(v->siglist);
free(v->siglist);
free(v->path);
free(v);
} }
return -1; alpm_list_free(errors);
if(tryagain == 0) {
if(!handle->pm_errno) {
RET_ERR(handle, ALPM_ERR_PKG_INVALID, -1);
}
return -1;
}
/* we were told at least once we can try again */
return 1;
} }
return 0; return 0;
@ -1051,8 +1079,14 @@ int _alpm_sync_commit(alpm_handle_t *handle, alpm_list_t **data)
/* this can only happen maliciously */ /* this can only happen maliciously */
total_bytes = total_bytes ? total_bytes : 1; total_bytes = total_bytes ? total_bytes : 1;
if(check_validity(handle, data, total, total_bytes)) { /* this one is special: -1 is failure, 1 is retry, 0 is success */
return -1; while(1) {
int ret = check_validity(handle, total, total_bytes);
if(ret == 0) {
break;
} else if(ret < 0) {
return -1;
}
} }
if(trans->flags & ALPM_TRANS_FLAG_DOWNLOADONLY) { if(trans->flags & ALPM_TRANS_FLAG_DOWNLOADONLY) {