aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNIIBE Yutaka <[email protected]>2022-06-20 02:25:44 +0000
committerWerner Koch <[email protected]>2023-06-16 08:05:28 +0000
commitff1f8ee7a8fb9729e08fb51419de9a8c93a4fdb6 (patch)
tree8232f0837354c901267bc3777b4659c48cfa93a2
parentclient: Wipe the inbound buffer when CONFIDENTIAL. (diff)
downloadlibassuan-ff1f8ee7a8fb9729e08fb51419de9a8c93a4fdb6.tar.gz
libassuan-ff1f8ee7a8fb9729e08fb51419de9a8c93a4fdb6.zip
Don't access NULL by wipememory.
* src/assuan-inquire.c (assuan_inquire): Check mb->buf. -- There is a case where mb->buf may be cleared already by get_membuf. Fixes-commit: d812e28af637144d078005887cb7519d7a2d0c58 Signed-off-by: NIIBE Yutaka <[email protected]>
-rw-r--r--src/assuan-inquire.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/assuan-inquire.c b/src/assuan-inquire.c
index 9599a77..7a40c85 100644
--- a/src/assuan-inquire.c
+++ b/src/assuan-inquire.c
@@ -255,7 +255,7 @@ assuan_inquire (assuan_context_t ctx, const char *keyword,
out:
if (!nodataexpected)
{
- if (ctx->flags.confidential)
+ if (ctx->flags.confidential && mb->buf)
wipememory (mb.buf, mb.len);
free_membuf (ctx, &mb);
}