security-and-privacy/gnupg
security-and-privacy/gnupg
1
0
Fork 0
mirror of git://git.gnupg.org/gnupg.git synced 2024-12-22 10:19:57 +01:00
Code Activity

gpg: Create default keyring with .kbx suffix.

Browse Source 
* g10/keydb.c (maybe_create_keyring_or_box): Rename arg for clarity.
(keydb_add_resource): Fix order of args to maybe_create_keyring_or_box
and check and create .kbx.
This commit is contained in:
Werner Koch 2014-09-18 16:00:34 +02:00
parent a4205d5ed0
commit bc2f5c1d1a
1 changed files with 33 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

44
g10/keydb.c
View File

@ -110,11 +110,13 @@ keyblock_cache_clear (void)
/* Handle the creation of a keyring or a keybox if it does not yet /* Handle the creation of a keyring or a keybox if it does not yet
exist. Take into acount that other processes might have the exist. Take into account that other processes might have the
keyring/keybox already locked. This lock check does not work if keyring/keybox already locked. This lock check does not work if
the directory itself is not yet available. */ the directory itself is not yet available. If is IS_BOX is true
the filename is expected to be a keybox. If FORCE_CREATE is true
the keyring or keybox shall be created. */
static int static int
maybe_create_keyring_or_box (char *filename, int is_box, int force) maybe_create_keyring_or_box (char *filename, int is_box, int force_create)
{ {
dotlock_t lockhd = NULL; dotlock_t lockhd = NULL;
IOBUF iobuf; IOBUF iobuf;
@ -129,14 +131,14 @@ maybe_create_keyring_or_box (char *filename, int is_box, int force)
/* If we don't want to create a new file at all, there is no need to /* If we don't want to create a new file at all, there is no need to
go any further - bail out right here. */ go any further - bail out right here. */
if (!force) if (!force_create)
return gpg_error (GPG_ERR_ENOENT); return gpg_error (GPG_ERR_ENOENT);
/* First of all we try to create the home directory. Note, that we /* First of all we try to create the home directory. Note, that we
don't do any locking here because any sane application of gpg don't do any locking here because any sane application of gpg
would create the home directory by itself and not rely on gpg's would create the home directory by itself and not rely on gpg's
tricky auto-creation which is anyway only done for some home tricky auto-creation which is anyway only done for certain home
directory name patterns. */ directory name pattern. */
last_slash_in_filename = strrchr (filename, DIRSEP_C); last_slash_in_filename = strrchr (filename, DIRSEP_C);
#if HAVE_W32_SYSTEM #if HAVE_W32_SYSTEM
{ {
@ -184,8 +186,8 @@ maybe_create_keyring_or_box (char *filename, int is_box, int force)
log_info ("can't allocate lock for '%s': %s\n", log_info ("can't allocate lock for '%s': %s\n",
filename, gpg_strerror (rc)); filename, gpg_strerror (rc));
if (!force) if (!force_create)
return gpg_error (GPG_ERR_ENOENT); return gpg_error (GPG_ERR_ENOENT); /* Won't happen. */
else else
return rc; return rc;
} }
@ -289,6 +291,7 @@ keydb_add_resource (const char *url, unsigned int flags)
char *filename = NULL; char *filename = NULL;
int create; int create;
int read_only = !!(flags&KEYDB_RESOURCE_FLAG_READONLY); int read_only = !!(flags&KEYDB_RESOURCE_FLAG_READONLY);
int is_default = !!(flags&KEYDB_RESOURCE_FLAG_DEFAULT);
int rc = 0; int rc = 0;
KeydbResourceType rt = KEYDB_RESOURCE_TYPE_NONE; KeydbResourceType rt = KEYDB_RESOURCE_TYPE_NONE;
void *token; void *token;
@ -334,8 +337,13 @@ keydb_add_resource (const char *url, unsigned int flags)
/* See whether we can determine the filetype. */ /* See whether we can determine the filetype. */
if (rt == KEYDB_RESOURCE_TYPE_NONE) if (rt == KEYDB_RESOURCE_TYPE_NONE)
{ {
FILE *fp = fopen (filename, "rb"); FILE *fp;
int pass = 0;
size_t filenamelen;
check_again:
filenamelen = strlen (filename);
fp = fopen (filename, "rb");
if (fp) if (fp)
{ {
u32 magic; u32 magic;
@ -357,6 +365,20 @@ keydb_add_resource (const char *url, unsigned int flags)
fclose (fp); fclose (fp);
} }
else if (!pass
&& is_default && create
&& filenamelen > 4 && !strcmp (filename+filenamelen-4, ".gpg"))
{
/* The file does not exist, the default resource has been
requested, the file shall be created, and the file has a
".gpg" suffix. Change the suffix to ".kbx" and try once
more. This way we achieve that we open an existing
".gpg" keyring, but create a new keybox file with an
".kbx" suffix. */
strcpy (filename+filenamelen-4, ".kbx");
pass++;
goto check_again;
}
else /* No file yet: create keybox. */ else /* No file yet: create keybox. */
rt = KEYDB_RESOURCE_TYPE_KEYBOX; rt = KEYDB_RESOURCE_TYPE_KEYBOX;
} }
@ -369,7 +391,7 @@ keydb_add_resource (const char *url, unsigned int flags)
goto leave; goto leave;
case KEYDB_RESOURCE_TYPE_KEYRING: case KEYDB_RESOURCE_TYPE_KEYRING:
rc = maybe_create_keyring_or_box (filename, create, 0); rc = maybe_create_keyring_or_box (filename, 0, create);
if (rc) if (rc)
goto leave; goto leave;
@ -399,7 +421,7 @@ keydb_add_resource (const char *url, unsigned int flags)
case KEYDB_RESOURCE_TYPE_KEYBOX: case KEYDB_RESOURCE_TYPE_KEYBOX:
{ {
rc = maybe_create_keyring_or_box (filename, create, 1); rc = maybe_create_keyring_or_box (filename, 1, create);
if (rc) if (rc)
goto leave; goto leave;