Module Name: src Committed By: agc Date: Tue Jun 1 05:22:38 UTC 2010
Modified Files: src/crypto/external/bsd/netpgp/dist/src/lib: netpgp.c Log Message: make sure we have created a directory when generating a new key. don't try to re-read the key after writing it - that's done by a separate function. Problem found by Tyler Retzlaff, fixed in a different way. check that keyrings are non-NULL before attempting to free them - from a nudge by Tyler Retzlaff. To generate a diff of this commit: cvs rdiff -u -r1.56 -r1.57 \ src/crypto/external/bsd/netpgp/dist/src/lib/netpgp.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/crypto/external/bsd/netpgp/dist/src/lib/netpgp.c diff -u src/crypto/external/bsd/netpgp/dist/src/lib/netpgp.c:1.56 src/crypto/external/bsd/netpgp/dist/src/lib/netpgp.c:1.57 --- src/crypto/external/bsd/netpgp/dist/src/lib/netpgp.c:1.56 Tue May 25 01:05:10 2010 +++ src/crypto/external/bsd/netpgp/dist/src/lib/netpgp.c Tue Jun 1 05:22:38 2010 @@ -34,7 +34,7 @@ #if defined(__NetBSD__) __COPYRIGHT("@(#) Copyright (c) 2009 The NetBSD Foundation, Inc. All rights reserved."); -__RCSID("$NetBSD: netpgp.c,v 1.56 2010/05/25 01:05:10 agc Exp $"); +__RCSID("$NetBSD: netpgp.c,v 1.57 2010/06/01 05:22:38 agc Exp $"); #endif #include <sys/types.h> @@ -755,9 +755,12 @@ cp = NULL; __ops_sprint_keydata(netpgp->io, NULL, key, &cp, "pub", &key->key.seckey.pubkey, 0); (void) fprintf(stdout, "%s", cp); - /* write public key, and try to re-read it */ + /* write public key */ (void) snprintf(dir, sizeof(dir), "%s/%.16s", netpgp_getvar(netpgp, "homedir"), &cp[31]); - (void) mkdir(dir, 0700); + if (mkdir(dir, 0700) < 0) { + (void) fprintf(io->errs, "can't mkdir '%s'\n", dir); + return 0; + } (void) fprintf(io->errs, "netpgp: generated keys in directory %s\n", dir); (void) snprintf(ringfile = filename, sizeof(filename), "%s/pubring.gpg", dir); if ((fd = __ops_setup_file_append(&create, ringfile)) < 0) { @@ -772,12 +775,10 @@ return 0; } __ops_teardown_file_write(create, fd); - __ops_keyring_free(netpgp->pubring); - if (!__ops_keyring_fileread(netpgp->pubring, noarmor, ringfile)) { - (void) fprintf(io->errs, "Cannot read pubring %s\n", ringfile); - return 0; + if (netpgp->pubring != NULL) { + __ops_keyring_free(netpgp->pubring); } - /* write secret key, and try to re-read it */ + /* write secret key */ (void) snprintf(ringfile = filename, sizeof(filename), "%s/secring.gpg", dir); if ((fd = __ops_setup_file_append(&create, ringfile)) < 0) { fd = __ops_setup_file_write(&create, ringfile, 0); @@ -791,10 +792,8 @@ return 0; } __ops_teardown_file_write(create, fd); - __ops_keyring_free(netpgp->secring); - if (!__ops_keyring_fileread(netpgp->secring, noarmor, ringfile)) { - (void) fprintf(io->errs, "Can't read secring %s\n", ringfile); - return 0; + if (netpgp->secring != NULL) { + __ops_keyring_free(netpgp->secring); } __ops_keydata_free(key); free(cp);