security-and-privacy/gnupg
security-and-privacy/gnupg
1
0
Fork 0
mirror of git://git.gnupg.org/gnupg.git synced 2025-07-02 22:46:30 +02:00
Code Activity

kbx: Implement keybox_lock for use by gpg.

Browse source 
* kbx/keybox-defs.h: Include dotlock.h and logging.h.
(CONST_KB_NAME): Remove.  Replace usage by KB_NAME.
(struct keybox_name): Add field "lockhd".
* kbx/keybox-init.c (keybox_register_file): Init LOCKHD.
(keybox_lock): Chnage to return gpg_error_t.  Implement locking.
--

The keybox locking for gpg was not implemented - This needs to be
fixed of course.

Signed-off-by: Werner Koch <wk@gnupg.org>
This commit is contained in:
Werner Koch 2016-01-13 10:16:27 +01:00
parent 9dc355ad3a
commit 1608629786
No known key found for this signature in database
GPG key ID: E3FDFF218E45B72B
3 changed files with 55 additions and 15 deletions
Download patch file Download diff file Expand all files Collapse all files

58
kbx/keybox-init.c
View file

@ -60,7 +60,7 @@ keybox_register_file (const char *fname, int secret, void **r_token)
kr->handle_table = NULL;
kr->handle_table_size = 0;
/* kr->lockhd = NULL;*/
kr->lockhd = NULL;
kr->is_locked = 0;
kr->did_full_scan = 0;
/* keep a list of all issued pointers */
@ -261,17 +261,55 @@ _keybox_close_file (KEYBOX_HANDLE hd)
/*
* Lock the keybox at handle HD, or unlock if YES is false. Note that
* we currently ignore the handle and lock all registered keyboxes.
* Lock the keybox at handle HD, or unlock if YES is false.
*/
int
gpg_error_t
keybox_lock (KEYBOX_HANDLE hd, int yes)
{
/* FIXME: We need to implement it before we can use it with gpg.
gpgsm does the locking in its local keydb.c driver; this should
be changed as well. */
gpg_error_t err;
KB_NAME kb = hd->kb;
(void)hd;
(void)yes;
return 0;
if (!keybox_is_writable ((void*)kb))
return 0;
/* Make sure the lock handle has been created. */
if (!kb->lockhd)
{
kb->lockhd = dotlock_create (kb->fname, 0);
if (!kb->lockhd)
{
/* Unfortuntaley dotlock_create does not properly set ERRNO. */
log_info ("can't allocate lock for '%s'\n", kb->fname );
return gpg_error (GPG_ERR_GENERAL);
}
}
if (yes) /* Take the lock. */
{
if (kb->is_locked)
;
else if (!dotlock_take (kb->lockhd, -1))
kb->is_locked = 1;
else
{
/* Unfortuntaley dotlock_take does not properly set ERRNO. */
log_info ("can't lock '%s'\n", kb->fname );
err = gpg_error (GPG_ERR_GENERAL);
}
}
else /* Release the lock. */
{
if (!kb->is_locked)
;
else if (!dotlock_release (kb->lockhd))
kb->is_locked = 0;
else
{
/* Unfortuntaley dotlock_release does not properly set ERRNO. */
log_info ("can't unlock '%s'\n", kb->fname );
err = gpg_error (GPG_ERR_GENERAL);
}
}
return err;
}