* [PATCH] rpc.idmapd: Do not free config varibles
@ 2020-09-04 18:19 Steve Dickson
2020-09-08 15:27 ` Steve Dickson
0 siblings, 1 reply; 2+ messages in thread
From: Steve Dickson @ 2020-09-04 18:19 UTC (permalink / raw)
To: Linux NFS Mailing list
Commit 93e8f092e added a conf_cleanup() call to clean
up memory after the config file was parsed. It turns
out that memory still needed and it is not very much
so the call is removed.
Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1873965
Signed-off-by: Steve Dickson <steved@redhat.com>
---
utils/idmapd/idmapd.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/utils/idmapd/idmapd.c b/utils/idmapd/idmapd.c
index f3d2314..51c71fb 100644
--- a/utils/idmapd/idmapd.c
+++ b/utils/idmapd/idmapd.c
@@ -306,9 +306,6 @@ main(int argc, char **argv)
serverstart = 0;
}
- /* Config memory is no longer needed */
- conf_cleanup();
-
while ((opt = getopt(argc, argv, GETOPTSTR)) != -1)
switch (opt) {
case 'v':
--
2.26.2
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] rpc.idmapd: Do not free config varibles
2020-09-04 18:19 [PATCH] rpc.idmapd: Do not free config varibles Steve Dickson
@ 2020-09-08 15:27 ` Steve Dickson
0 siblings, 0 replies; 2+ messages in thread
From: Steve Dickson @ 2020-09-08 15:27 UTC (permalink / raw)
To: Linux NFS Mailing list
On 9/4/20 2:19 PM, Steve Dickson wrote:
> Commit 93e8f092e added a conf_cleanup() call to clean
> up memory after the config file was parsed. It turns
> out that memory still needed and it is not very much
> so the call is removed.
>
> Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=1873965
>
> Signed-off-by: Steve Dickson <steved@redhat.com>
Committed...
steved.
> ---
> utils/idmapd/idmapd.c | 3 ---
> 1 file changed, 3 deletions(-)
>
> diff --git a/utils/idmapd/idmapd.c b/utils/idmapd/idmapd.c
> index f3d2314..51c71fb 100644
> --- a/utils/idmapd/idmapd.c
> +++ b/utils/idmapd/idmapd.c
> @@ -306,9 +306,6 @@ main(int argc, char **argv)
> serverstart = 0;
> }
>
> - /* Config memory is no longer needed */
> - conf_cleanup();
> -
> while ((opt = getopt(argc, argv, GETOPTSTR)) != -1)
> switch (opt) {
> case 'v':
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-09-08 19:58 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-09-04 18:19 [PATCH] rpc.idmapd: Do not free config varibles Steve Dickson
2020-09-08 15:27 ` Steve Dickson
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox