diff options
| author | Günther Deschner <gd@samba.org> | 2007-07-24 09:46:21 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:28:57 -0500 |
| commit | 57aa0acb84a41a24ea89950bcb51b5ffc68b4564 (patch) | |
| tree | fac0d1b0d5e73cfa7c54a37e58239ba5666acbcf /source | |
| parent | 6a3c44fd99c91beddd9d6a04a30c35d429d0b9a5 (diff) | |
| download | samba-57aa0acb84a41a24ea89950bcb51b5ffc68b4564.tar.gz samba-57aa0acb84a41a24ea89950bcb51b5ffc68b4564.tar.xz samba-57aa0acb84a41a24ea89950bcb51b5ffc68b4564.zip | |
r24024: Minor cleanup in ads_parse_gp_ext().
Guenther
Diffstat (limited to 'source')
| -rw-r--r-- | source/libgpo/gpo_ldap.c | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/source/libgpo/gpo_ldap.c b/source/libgpo/gpo_ldap.c index 4f983b261d8..e32522ab5c8 100644 --- a/source/libgpo/gpo_ldap.c +++ b/source/libgpo/gpo_ldap.c @@ -29,6 +29,7 @@ ADS_STATUS ads_parse_gp_ext(TALLOC_CTX *mem_ctx, const char *extension_raw, struct GP_EXT **gp_ext) { + ADS_STATUS status = ADS_ERROR_NT(NT_STATUS_NO_MEMORY); struct GP_EXT *ext = NULL; char **ext_list; char **ext_strings = NULL; @@ -131,16 +132,9 @@ ADS_STATUS ads_parse_gp_ext(TALLOC_CTX *mem_ctx, } } - if (ext_list) { - str_list_free_talloc(mem_ctx, &ext_list); - } - if (ext_strings) { - str_list_free_talloc(mem_ctx, &ext_strings); - } - *gp_ext = ext; - return ADS_ERROR(LDAP_SUCCESS); + status = ADS_ERROR_NT(NT_STATUS_OK); parse_error: if (ext_list) { @@ -150,7 +144,7 @@ parse_error: str_list_free_talloc(mem_ctx, &ext_strings); } - return ADS_ERROR(LDAP_NO_MEMORY); + return status; } /**************************************************************** |