diff options
author | Werner Koch <wk@gnupg.org> | 2015-08-07 11:26:00 +0200 |
---|---|---|
committer | Werner Koch <wk@gnupg.org> | 2015-08-07 12:17:34 +0200 |
commit | b5cbf11ccece653819a782a3e8adbb785fe36d7d (patch) | |
tree | a2d4a8b1307f4ee0b2bc3f0e15f3ddf4c37c1a32 | |
parent | 87def94c86d5272c23daf2b5ea446c5553aa1d90 (diff) | |
download | libassuan-b5cbf11ccece653819a782a3e8adbb785fe36d7d.tar.gz |
Wipe the context before releasing as an extra safeguard.
* src/assuan-defs.h (wipememory2, wipememory): New. Taken from GnuPG.
* src/assuan.c (assuan_release): Wipe the context.
--
The assuan context has buffers which may carry senitive information.
These buffers could be wiped out with each flush but that is too
expensive. Thus we only wipe them when freeing the context.
Signed-off-by: Werner Koch <wk@gnupg.org>
-rw-r--r-- | src/assuan-defs.h | 10 | ||||
-rw-r--r-- | src/assuan.c | 6 |
2 files changed, 15 insertions, 1 deletions
diff --git a/src/assuan-defs.h b/src/assuan-defs.h index 68cd810..cf0015e 100644 --- a/src/assuan-defs.h +++ b/src/assuan-defs.h @@ -404,6 +404,16 @@ int _assuan_asprintf (char **buf, const char *fmt, ...); #define DIM(v) (sizeof(v)/sizeof((v)[0])) +/* To avoid that a compiler optimizes memset calls away, these macros + can be used. */ +#define wipememory2(_ptr,_set,_len) do { \ + volatile char *_vptr=(volatile char *)(_ptr); \ + size_t _vlen=(_len); \ + while(_vlen) { *_vptr=(_set); _vptr++; _vlen--; } \ + } while(0) +#define wipememory(_ptr,_len) wipememory2(_ptr,0,_len) + + #if HAVE_W64_SYSTEM # define SOCKET2HANDLE(s) ((void *)(s)) # define HANDLE2SOCKET(h) ((uintptr_t)(h)) diff --git a/src/assuan.c b/src/assuan.c index 5cbb86c..d4c4b56 100644 --- a/src/assuan.c +++ b/src/assuan.c @@ -189,7 +189,11 @@ assuan_release (assuan_context_t ctx) _assuan_reset (ctx); /* None of the members that are our responsibility requires - deallocation. */ + deallocation. To avoid sensitive data in the line buffers we + wipe them out, though. Note that we can't wipe the entire + context because it also has a pointer to the actual free(). */ + wipememory (&ctx->inbound, sizeof ctx->inbound); + wipememory (&ctx->outbound, sizeof ctx->outbound); _assuan_free (ctx, ctx); } |