diff options
Diffstat (limited to '')
| -rw-r--r-- | g10/ChangeLog | 15 | ||||
| -rw-r--r-- | g10/export.c | 4 | ||||
| -rw-r--r-- | g10/import.c | 9 | ||||
| -rw-r--r-- | g10/trustdb.c | 9 |
4 files changed, 28 insertions, 9 deletions
diff --git a/g10/ChangeLog b/g10/ChangeLog index 8dd57c7f6..af238dc56 100644 --- a/g10/ChangeLog +++ b/g10/ChangeLog @@ -1,3 +1,18 @@ +2005-11-01 David Shaw <[email protected]> + + * import.c (import_one): Do collapse_uids() before we do any + cleaning so keyserver mangled keys with doubled user IDs can be + properly cleaned - possibly sigs on the different user IDs cancel + each other out. + + * import.c (parse_import_options), export.c + (parse_export_options): List "xxx-clean" before the longer options + so we don't end up with a partial match on the longer options. + + * trustdb.c (clean_uids_from_key): Return proper number of cleaned + user IDs. Don't count user IDs as cleaned unless we actually + delete something. + 2005-10-27 David Shaw <[email protected]> * keyedit.c (menu_addrevoker), getkey.c (finish_lookup): Fix diff --git a/g10/export.c b/g10/export.c index ab5bd0f28..053e1c185 100644 --- a/g10/export.c +++ b/g10/export.c @@ -64,12 +64,12 @@ parse_export_options(char *str,unsigned int *options,int noisy) N_("export attribute user IDs (generally photo IDs)")}, {"export-sensitive-revkeys",EXPORT_SENSITIVE_REVKEYS,NULL, N_("export revocation keys that are marked as \"sensitive\"")}, + {"export-clean",EXPORT_CLEAN_SIGS|EXPORT_CLEAN_UIDS,NULL, + N_("all export-clean-* options from above")}, {"export-clean-sigs",EXPORT_CLEAN_SIGS,NULL, N_("remove unusable signatures during export")}, {"export-clean-uids",EXPORT_CLEAN_UIDS,NULL, N_("remove unusable user IDs during export")}, - {"export-clean",EXPORT_CLEAN_SIGS|EXPORT_CLEAN_UIDS,NULL, - N_("all export-clean-* options from above")}, {"export-minimal", EXPORT_MINIMAL|EXPORT_CLEAN_SIGS|EXPORT_CLEAN_UIDS,NULL, N_("export the smallest key possible")}, diff --git a/g10/import.c b/g10/import.c index 7eeccbad4..d8a7b2a61 100644 --- a/g10/import.c +++ b/g10/import.c @@ -101,12 +101,12 @@ parse_import_options(char *str,unsigned int *options,int noisy) N_("create a public key when importing a secret key")}, {"merge-only",IMPORT_MERGE_ONLY,NULL, N_("only accept updates to existing keys")}, + {"import-clean",IMPORT_CLEAN_SIGS|IMPORT_CLEAN_UIDS,NULL, + N_("all import-clean-* options from above")}, {"import-clean-sigs",IMPORT_CLEAN_SIGS,NULL, N_("remove unusable signatures after import")}, {"import-clean-uids",IMPORT_CLEAN_UIDS,NULL, N_("remove unusable user IDs after import")}, - {"import-clean",IMPORT_CLEAN_SIGS|IMPORT_CLEAN_UIDS,NULL, - N_("all import-clean-* options from above")}, /* Aliases for backward compatibility */ {"allow-local-sigs",IMPORT_LOCAL_SIGS,NULL,NULL}, {"repair-hkp-subkey-bug",IMPORT_REPAIR_PKS_SUBKEY_BUG,NULL,NULL}, @@ -739,6 +739,8 @@ import_one( const char *fname, KBNODE keyblock, return 0; } + collapse_uids(&keyblock); + /* Clean the key that we're about to import, to cut down on things that we have to clean later. This has no practical impact on the end result, but does result in less logging which might @@ -808,8 +810,6 @@ import_one( const char *fname, KBNODE keyblock, if( opt.verbose > 1 ) log_info (_("writing to `%s'\n"), keydb_get_resource_name (hd) ); - collapse_uids(&keyblock); - rc = keydb_insert_keyblock (hd, keyblock ); if (rc) log_error (_("error writing keyring `%s': %s\n"), @@ -886,7 +886,6 @@ import_one( const char *fname, KBNODE keyblock, goto leave; } - collapse_uids( &keyblock ); /* and try to merge the block */ clear_kbnode_flags( keyblock_orig ); clear_kbnode_flags( keyblock ); diff --git a/g10/trustdb.c b/g10/trustdb.c index de39b54de..f2e109621 100644 --- a/g10/trustdb.c +++ b/g10/trustdb.c @@ -1654,7 +1654,7 @@ clean_sigs_from_uid(KBNODE keyblock,KBNODE uidnode,int noisy) int clean_uids_from_key(KBNODE keyblock,int noisy) { - int delete_until_next=0,deleted=0; + int delete_until_next=0,deleting=0,deleted=0; KBNODE node,signode=NULL; u32 keyid[2],sigdate=0; @@ -1684,6 +1684,7 @@ clean_uids_from_key(KBNODE keyblock,int noisy) else { delete_until_next=1; + deleting=1; if(noisy) { @@ -1722,7 +1723,11 @@ clean_uids_from_key(KBNODE keyblock,int noisy) if(delete_until_next && !sig->flags.chosen_selfsig) { delete_kbnode(node); - deleted++; + if(deleting) + { + deleted++; + deleting=0; + } } } } |