2020-08-31 16:14:04

by Steve Dickson

[permalink] [raw]
Subject: [PATCH] rpc.gssd: munmap_chunk(): invalid pointer

Removed an errant call to gss_release_oid()
to try and deal with memory leaks

Signed-off-by: Steve Dickson <[email protected]>
---
utils/gssd/gssd_proc.c | 1 -
1 file changed, 1 deletion(-)

diff --git a/utils/gssd/gssd_proc.c b/utils/gssd/gssd_proc.c
index 2a8b618..e830f49 100644
--- a/utils/gssd/gssd_proc.c
+++ b/utils/gssd/gssd_proc.c
@@ -687,7 +687,6 @@ process_krb5_upcall(struct clnt_info *clp, uid_t uid, int fd, char *srchost,
} else {
get_hostbased_client_buffer(gacceptor, mech, &acceptor);
gss_release_name(&min_stat, &gacceptor);
- gss_release_oid(&min_stat, &mech);
}

/*
--
2.26.2


2020-08-31 16:17:03

by Steve Dickson

[permalink] [raw]
Subject: Re: [PATCH] rpc.gssd: munmap_chunk(): invalid pointer



On 8/31/20 12:11 PM, Steve Dickson wrote:
> Removed an errant call to gss_release_oid()
> to try and deal with memory leaks
>
> Signed-off-by: Steve Dickson <[email protected]>
Committed... (tag: nfs-utils-2-5-2-rc4)

steved.

> ---
> utils/gssd/gssd_proc.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/utils/gssd/gssd_proc.c b/utils/gssd/gssd_proc.c
> index 2a8b618..e830f49 100644
> --- a/utils/gssd/gssd_proc.c
> +++ b/utils/gssd/gssd_proc.c
> @@ -687,7 +687,6 @@ process_krb5_upcall(struct clnt_info *clp, uid_t uid, int fd, char *srchost,
> } else {
> get_hostbased_client_buffer(gacceptor, mech, &acceptor);
> gss_release_name(&min_stat, &gacceptor);
> - gss_release_oid(&min_stat, &mech);
> }
>
> /*
>