|
 |
00e421 |
commit cfe41d6f06af0e7744c1ca30503f93d28aca4d8b
|
|
|
00e421 |
Author: NeilBrown <neilb@suse.de>
|
|
|
00e421 |
Date: Tue Sep 21 12:47:10 2021 -0400
|
|
|
00e421 |
|
|
|
00e421 |
gssd: fix crash in debug message.
|
|
|
00e421 |
|
|
|
00e421 |
A recent cleanup of debug messages added func and tid format specifiers
|
|
|
00e421 |
to a debug message (when full hostname was different), but the func name
|
|
|
00e421 |
and tid were NOT added as arguments.
|
|
|
00e421 |
|
|
|
00e421 |
Consequently there weren't enough args, random bytes of the stack were
|
|
|
00e421 |
interpreted as a pointer, and rpc.gssd crashed (when -v was specified).
|
|
|
00e421 |
|
|
|
00e421 |
Fixes: b538862a5135 ("gssd: Cleaned up debug messages")
|
|
|
00e421 |
Reviewed-by: Petr Vorel <pvorel@suse.cz>
|
|
|
00e421 |
Signed-off-by: NeilBrown <neilb@suse.de>
|
|
|
00e421 |
Signed-off-by: Steve Dickson <steved@redhat.com>
|
|
|
00e421 |
|
|
|
00e421 |
diff --git a/utils/gssd/krb5_util.c b/utils/gssd/krb5_util.c
|
|
|
00e421 |
index 6d059f33..e3f270e9 100644
|
|
|
00e421 |
--- a/utils/gssd/krb5_util.c
|
|
|
00e421 |
+++ b/utils/gssd/krb5_util.c
|
|
|
00e421 |
@@ -673,8 +673,8 @@ get_full_hostname(const char *inhost, char *outhost, int outhostlen)
|
|
|
00e421 |
*c = tolower(*c);
|
|
|
00e421 |
|
|
|
00e421 |
if (get_verbosity() && strcmp(inhost, outhost))
|
|
|
00e421 |
- printerr(1, "%s(0x%0lx): inhost '%s' different than outhost'%s'\n",
|
|
|
00e421 |
- inhost, outhost);
|
|
|
00e421 |
+ printerr(1, "%s(0x%0lx): inhost '%s' different than outhost '%s'\n",
|
|
|
00e421 |
+ __func__, tid, inhost, outhost);
|
|
|
00e421 |
|
|
|
00e421 |
retval = 0;
|
|
|
00e421 |
out:
|