aboutsummaryrefslogtreecommitdiffstats
path: root/g10
diff options
context:
space:
mode:
Diffstat (limited to 'g10')
-rw-r--r--g10/ChangeLog11
-rw-r--r--g10/import.c7
-rw-r--r--g10/keygen.c2
-rw-r--r--g10/sign.c4
4 files changed, 16 insertions, 8 deletions
diff --git a/g10/ChangeLog b/g10/ChangeLog
index 7e09fb77e..92bfde68d 100644
--- a/g10/ChangeLog
+++ b/g10/ChangeLog
@@ -1,3 +1,14 @@
+2011-01-10 Werner Koch <[email protected]>
+
+ * keygen.c (ask_user_id): Fix duplicate test for AMAIL by correct
+ ANAME. See bug#1307. Reported by Steve Grubb.
+
+ * import.c (import_keys_internal): Make loop code a bit more
+ readable. See bug#1307. Reported by Steve Grubb.
+
+ * sign.c (sign_file): Fix TEMP_HASHLEN computation. See bug#1307.
+ Reported by Steve Grubb.
+
2010-10-29 David Shaw <[email protected]>
* pkclist.c (select_algo_from_prefs): Make sure the scores can't
diff --git a/g10/import.c b/g10/import.c
index e2d77b019..ba2439dcc 100644
--- a/g10/import.c
+++ b/g10/import.c
@@ -175,10 +175,9 @@ import_keys_internal( IOBUF inp, char **fnames, int nnames,
rc = import( inp, "[stream]", stats, fpr, fpr_len, options);
}
else {
- if( !fnames && !nnames )
- nnames = 1; /* Ohh what a ugly hack to jump into the loop */
+ int once = (!fnames && !nnames);
- for(i=0; i < nnames; i++ ) {
+ for(i=0; once || i < nnames; once=0, i++ ) {
const char *fname = fnames? fnames[i] : NULL;
IOBUF inp2 = iobuf_open(fname);
if( !fname )
@@ -201,8 +200,6 @@ import_keys_internal( IOBUF inp, char **fnames, int nnames,
log_error("import from `%s' failed: %s\n", fname,
g10_errstr(rc) );
}
- if( !fname )
- break;
}
}
if (!stats_handle) {
diff --git a/g10/keygen.c b/g10/keygen.c
index 3aaae9c16..8c3e9f617 100644
--- a/g10/keygen.c
+++ b/g10/keygen.c
@@ -2247,7 +2247,7 @@ ask_user_id (int mode, KBNODE keyblock)
xfree(answer);
}
xfree(answer);
- if( !amail && !acomment && !amail )
+ if( !aname && !acomment && !amail )
break;
xfree(uid); uid = NULL;
}
diff --git a/g10/sign.c b/g10/sign.c
index f54e0d9d5..94e107cf2 100644
--- a/g10/sign.c
+++ b/g10/sign.c
@@ -867,8 +867,8 @@ sign_file( strlist_t filenames, int detached, strlist_t locusr,
{
if (sk_rover->sk->pubkey_algo == PUBKEY_ALGO_DSA)
{
- int temp_hashlen = gcry_mpi_get_nbits
- (sk_rover->sk->skey[1])+7/8;
+ int temp_hashlen = (gcry_mpi_get_nbits
+ (sk_rover->sk->skey[1])+7)/8;
/* Pick a hash that is large enough for our
largest q */