From 839b32b1d14c8a45d58a75ef45d145d20744604a Mon Sep 17 00:00:00 2001
From: Vinit Agnihotri <vagnihot@redhat.com>
Date: Thu, 12 Dec 2024 19:05:24 +0530
Subject: [PATCH] sharesec: Fix warning frame not freed in order

This change should fix following warning:
Freed frame ../../source3/utils/sharesec.c:515, expected ../../source3/utils/sharesec.c:637

Frame was not getting freed in case of servicename is NULL.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=15780

Signed-off-by: Vinit Agnihotri <vagnihot@redhat.com>
Reviewed-by: John Mulligan <jmulligan@samba.org>
Reviewed-by: Guenther Deschner <gd@samba.org>
(cherry picked from commit 04531e1b1d25d114c470922547bee769b07e5e60)
---
 source3/utils/sharesec.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/source3/utils/sharesec.c b/source3/utils/sharesec.c
index 417572954c8..3c93893e178 100644
--- a/source3/utils/sharesec.c
+++ b/source3/utils/sharesec.c
@@ -563,6 +563,7 @@ int main(int argc, const char *argv[])
 			const char *service = lp_servicename(frame, lp_sub, i);
 
 			if (service == NULL) {
+				TALLOC_FREE(frame);
 				continue;
 			}
 
-- 
2.47.2