From 68333be63042be281a497a5751474e9e3d119dc3 Mon Sep 17 00:00:00 2001 From: Werner Koch Date: Tue, 21 Mar 2023 10:33:34 +0100 Subject: Avoid segv in logging with improper use of the "socket://" . * src/logging.c (fun_writer): Protect calling socket_dir_cb if not set. -- If no callback has been registered the use of the "socket://" shortcut caused a segv. --- src/sysutils.c | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'src/sysutils.c') diff --git a/src/sysutils.c b/src/sysutils.c index 1251ed6..fac72bf 100644 --- a/src/sysutils.c +++ b/src/sysutils.c @@ -141,6 +141,10 @@ _gpgrt_setenv (const char *name, const char *value, int overwrite) if (!value && overwrite) { + /* Deleting an envvar. Although the Microsoft specs for + * putenv tell us that one may use "NAME=" to unset an envvar, + * this seems not to be correct. Thus we do the same as what + * we do in Unix (or well in GNU libc) and use just "NAME". */ if (!SetEnvironmentVariable (name, NULL)) return GPG_ERR_EINVAL; if (getenv (name)) -- cgit v1.2.3