Compare commits
No commits in common. "c8" and "c9-beta" have entirely different histories.
2
.gitignore
vendored
2
.gitignore
vendored
@ -1,2 +1,2 @@
|
||||
SOURCES/samba-4.19.4.tar.xz
|
||||
SOURCES/samba-4.24.5.tar.xz
|
||||
SOURCES/samba-pubkey_AA99442FB680B620.gpg
|
||||
|
||||
@ -1,2 +1,2 @@
|
||||
6a164128df94dd89e785ca9f42d7be5714f16bed SOURCES/samba-4.19.4.tar.xz
|
||||
d8bc9b1cb04f3d03fcf96f5af4b959f7ec26cefd SOURCES/samba-4.24.5.tar.xz
|
||||
971f563c447eda8d144d6c9e743cd0f0488c0d9e SOURCES/samba-pubkey_AA99442FB680B620.gpg
|
||||
|
||||
80
SOURCES/redhat-4.24.patch
Normal file
80
SOURCES/redhat-4.24.patch
Normal file
@ -0,0 +1,80 @@
|
||||
From 1de6565c25272b536607e7e787d65765a3ad5220 Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Pavel=20Filipensk=C3=BD?= <pfilipensky@samba.org>
|
||||
Date: Thu, 22 Jan 2026 14:27:09 +0100
|
||||
Subject: [PATCH] s3:libads: Allocate cli_credentials on a stackframe
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
This fixes:
|
||||
ERROR: talloc_free with references at ../../source3/libads/ldap_utils.c:158
|
||||
|
||||
What happens:
|
||||
|
||||
* `struct cli_credentials *creds` is allocated on `ads` talloc context
|
||||
* gensec_set_credentials() creates a talloc_reference to `creds`
|
||||
* TALLOC_FREE(creds) sees two parents and complains
|
||||
|
||||
All other code is using temporary talloc_stackframe() for `creds`.
|
||||
Do it here as well.
|
||||
|
||||
Signed-off-by: Pavel Filipenský <pfilipensky@samba.org>
|
||||
Reviewed-by: Stefan Metzmacher <metze@samba.org>
|
||||
|
||||
Autobuild-User(master): Stefan Metzmacher <metze@samba.org>
|
||||
Autobuild-Date(master): Fri Jan 23 11:20:28 UTC 2026 on atb-devel-224
|
||||
---
|
||||
source3/libads/ldap_utils.c | 9 ++++++---
|
||||
1 file changed, 6 insertions(+), 3 deletions(-)
|
||||
|
||||
diff --git a/source3/libads/ldap_utils.c b/source3/libads/ldap_utils.c
|
||||
index 9d6d962a2bc..d01afa69697 100644
|
||||
--- a/source3/libads/ldap_utils.c
|
||||
+++ b/source3/libads/ldap_utils.c
|
||||
@@ -99,6 +99,7 @@ static ADS_STATUS ads_do_search_retry_internal(ADS_STRUCT *ads, const char *bind
|
||||
struct cli_credentials *creds = NULL;
|
||||
char *cred_name = NULL;
|
||||
NTSTATUS ntstatus;
|
||||
+ TALLOC_CTX *frame = talloc_stackframe();
|
||||
|
||||
if (NT_STATUS_EQUAL(ads_ntstatus(status), NT_STATUS_IO_TIMEOUT) &&
|
||||
ads->config.ldap_page_size >= (lp_ldap_page_size() / 4) &&
|
||||
@@ -119,18 +120,20 @@ static ADS_STATUS ads_do_search_retry_internal(ADS_STRUCT *ads, const char *bind
|
||||
DBG_NOTICE("Search for %s in <%s> failed: %s\n",
|
||||
expr, bp, ads_errstr(status));
|
||||
SAFE_FREE(bp);
|
||||
+ TALLOC_FREE(frame);
|
||||
return status;
|
||||
}
|
||||
|
||||
ntstatus = ads->auth.reconnect_state->fn(ads,
|
||||
ads->auth.reconnect_state->private_data,
|
||||
- ads, &creds);
|
||||
+ frame, &creds);
|
||||
if (!NT_STATUS_IS_OK(ntstatus)) {
|
||||
DBG_WARNING("Failed to get creds for realm(%s): %s\n",
|
||||
ads->server.realm, nt_errstr(ntstatus));
|
||||
DBG_WARNING("Search for %s in <%s> failed: %s\n",
|
||||
expr, bp, ads_errstr(status));
|
||||
SAFE_FREE(bp);
|
||||
+ TALLOC_FREE(frame);
|
||||
return status;
|
||||
}
|
||||
|
||||
@@ -151,11 +154,11 @@ static ADS_STATUS ads_do_search_retry_internal(ADS_STRUCT *ads, const char *bind
|
||||
* callers depend on it being around.
|
||||
*/
|
||||
ads_disconnect(ads);
|
||||
- TALLOC_FREE(creds);
|
||||
+ TALLOC_FREE(frame);
|
||||
SAFE_FREE(bp);
|
||||
return status;
|
||||
}
|
||||
- TALLOC_FREE(creds);
|
||||
+ TALLOC_FREE(frame);
|
||||
|
||||
*res = NULL;
|
||||
|
||||
--
|
||||
2.53.0
|
||||
|
||||
File diff suppressed because it is too large
Load Diff
@ -1,16 +0,0 @@
|
||||
-----BEGIN PGP SIGNATURE-----
|
||||
|
||||
iQIzBAABCgAdFiEEgfXigyvSVFoYl7cTqplEL7aAtiAFAmWcCFAACgkQqplEL7aA
|
||||
tiDKSBAAuWA9jT6xCfFACIlme7DbEoUm/Bsbf+GM2Somd3pgajekiNxo7CsW9Xub
|
||||
Vmpj0Q5OKiri81XTqA8LlqMCBliqfw/rnP48kCH0YqXzjqD6aYuwmk0Q4G3wWBTJ
|
||||
2ZT/wOpbM3YooFfE9Iffz6uNgAiQ/8kpBt2m6Zzfy8n1ThfztyGAGaSmrUWxgUlq
|
||||
XjRjtgTw4isZBm+RzCFSGuPxvWvxRlfD5JCe2gc221rI3kbaQE2GSxdZ6D0635Ln
|
||||
iy64SLIAKkQCrrFFckudSCCLKgLNdIClEwzamhhCbmCxnWMDufzN+BQZhq3axQ+x
|
||||
svPfZqltVSQztr4nPGvKdebtVLL2Zyf/LtXWQP/s66quHlHFoEAC7MuD6tEMQVar
|
||||
JQUCN51Gs0Yk12iReQFm6/Uo35aPAlai1e2uOkNzS5FnagRObYt6FYeQripks4I8
|
||||
ZW5VvF4cE0zqdjrlG+Ttqmpbj7i6AUJj9wSbrEOFDUhTL+QPPOfJ05yr1BHmS6nJ
|
||||
vuuUs+ei/DnYEFS91P81h5NuOdpRHIBTG6LUOLz5KOoNdIgvzjD/Ugyscj4AFTBo
|
||||
+NTG9nNr6gkLV/6dxDRR2/sbU6P+FZBL+JVUoDR7XQ7oHG7sFV+/8Dtu8RivEw++
|
||||
1sNGqxvGkwu7JunMkJO5YZRwXi81v3nmHkWKgb0+52iYXgmdesY=
|
||||
=kOPP
|
||||
-----END PGP SIGNATURE-----
|
||||
16
SOURCES/samba-4.24.5.tar.asc
Normal file
16
SOURCES/samba-4.24.5.tar.asc
Normal file
@ -0,0 +1,16 @@
|
||||
-----BEGIN PGP SIGNATURE-----
|
||||
|
||||
iQIzBAABCgAdFiEEgfXigyvSVFoYl7cTqplEL7aAtiAFAmpg4boACgkQqplEL7aA
|
||||
tiDZrBAAiBI8SST7hZeDLH5NYqyduC8PWrgOm6ljVrsqwPOofIt1HVbJNmeujQx4
|
||||
+9EuVATaf5S2+RS94hZmnJOxY2Bqd5LfXM3i+uaKAYkXx+qjBzFhT0qEGW0L+ZQl
|
||||
x4fA0X5T0TdGF/022KRyZDt6LsgKl+O+0FiminuHDgqE8EsMNZ1jwjtbtn4cZVfn
|
||||
OU0Cq0w0TP38QDMkDxsLu+ZrrHR0gxmp0KVR+pLjUXITwXsLYtXmOnforCUKhdIt
|
||||
aIAg9B7uHrUMXPdF68s0msIFAA9aW9Y2UQPN7xRCvhlkKRwJKqLrFPJeRsaNrUkB
|
||||
beOXqjMCs3WUtnBSCmJGKWaWAiQhazxSfBdo1obNI6CYNOy47ibwUzDUBGfBeeJ2
|
||||
1kcEtQRKSo1I6yzTYptj7LlOHBaN9eB/myIrmaTyzn9v/RHgZ2dx9lCa12h+kkjS
|
||||
SnrTHJUpsNry26vtbV/5yph0cYRLo8A4bjoSWTINioa3KtN71384rQjgeaIK9Szb
|
||||
Du4Cs2cQzcLZ6Fj5TertAnL+sW0WnI+kmN5EVxzSffqLSzrBI18dTG13mFNweFV2
|
||||
mC72vW1mPdAE5rqjzwMahcxril2zKvezD8F2mP77gGs+oWxkEM9Wiuz2AfSEToZQ
|
||||
bg2oli+FIG9aGhRRjpxFaiYWq6gsws9h0a8as87YbAd+a2futPQ=
|
||||
=onxY
|
||||
-----END PGP SIGNATURE-----
|
||||
@ -1,4 +1,4 @@
|
||||
/var/log/samba/log.* {
|
||||
/var/log/samba/*log* {
|
||||
compress
|
||||
dateext
|
||||
maxage 365
|
||||
|
||||
@ -38,7 +38,8 @@
|
||||
[print$]
|
||||
comment = Printer Drivers
|
||||
path = /var/lib/samba/drivers
|
||||
write list = @printadmin root
|
||||
force group = @printadmin
|
||||
# printadmin is a local group
|
||||
write list = printadmin root
|
||||
force group = printadmin
|
||||
create mask = 0664
|
||||
directory mask = 0775
|
||||
|
||||
3435
SPECS/samba.spec
3435
SPECS/samba.spec
File diff suppressed because it is too large
Load Diff
Loading…
Reference in New Issue
Block a user