diff options
| author | Samuel Cabrero <scabrero@samba.org> | 2022-06-15 12:29:15 +0200 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2022-06-27 15:50:30 +0000 |
| commit | 8ea0dd1a38cd04d85527b7147618256af6c52b2d (patch) | |
| tree | 727a01c5f794b1f8ec921800272022fd810fa353 /source3/libads | |
| parent | 6ca5eaccfa3169a85917d86540145a447ddf2340 (diff) | |
| download | samba-8ea0dd1a38cd04d85527b7147618256af6c52b2d.tar.gz | |
s3:libads: Allocate ads->config.config_path under ADS_STRUCT talloc context
Signed-off-by: Samuel Cabrero <scabrero@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Diffstat (limited to 'source3/libads')
| -rw-r--r-- | source3/libads/ads_struct.c | 1 | ||||
| -rw-r--r-- | source3/libads/disp_sec.c | 5 |
2 files changed, 4 insertions, 2 deletions
diff --git a/source3/libads/ads_struct.c b/source3/libads/ads_struct.c index 56883e8496b..ce07f2a89dd 100644 --- a/source3/libads/ads_struct.c +++ b/source3/libads/ads_struct.c @@ -153,7 +153,6 @@ static void ads_destroy(ADS_STRUCT **ads) #ifdef HAVE_LDAP ads_disconnect(*ads); #endif - SAFE_FREE((*ads)->config.config_path); } } diff --git a/source3/libads/disp_sec.c b/source3/libads/disp_sec.c index 74311aac289..a193c5bb3e9 100644 --- a/source3/libads/disp_sec.c +++ b/source3/libads/disp_sec.c @@ -218,7 +218,10 @@ void ads_disp_sd(ADS_STRUCT *ads, TALLOC_CTX *mem_ctx, struct security_descripto if (ads && !ads->config.config_path) { if (ADS_ERR_OK(ads_config_path(ads, mem_ctx, &tmp_path))) { - ads->config.config_path = SMB_STRDUP(tmp_path); + ads->config.config_path = talloc_strdup(ads, tmp_path); + if (ads->config.config_path == NULL) { + DBG_WARNING("Out of memory\n"); + } } } |