From: Volker Lendecke <vl@samba.org>
Date: Tue, 8 Mar 2011 20:55:58 +0000 (+0100)
Subject: idmap-autorid: Slightly simplify idmap_autorid_get_domainrange
X-Git-Tag: ldb-1.1.0~684
X-Git-Url: http://git.samba.org/samba.git/?a=commitdiff_plain;h=5006dcff4e15b7f840093e0c6c4e6f46a4088120;p=samba.git

idmap-autorid: Slightly simplify idmap_autorid_get_domainrange

Signed-off-by: Christian Ambach <christian.ambach@de.ibm.com>
---

diff --git a/source3/winbindd/idmap_autorid.c b/source3/winbindd/idmap_autorid.c
index 434de251717..b162b623556 100644
--- a/source3/winbindd/idmap_autorid.c
+++ b/source3/winbindd/idmap_autorid.c
@@ -54,15 +54,12 @@ static NTSTATUS idmap_autorid_get_domainrange(struct db_context *db,
 {
 	NTSTATUS ret;
 	uint32_t domainnum, hwm;
-	char *sidstr, *numstr;
+	fstring sidstr;
+	char *numstr;
 	struct autorid_domain_config *cfg;
 
 	cfg = (struct autorid_domain_config *)private_data;
-	sidstr = dom_sid_string(talloc_tos(), &(cfg->sid));
-
-	if (!sidstr) {
-		return NT_STATUS_NO_MEMORY;
-	}
+	dom_sid_string_buf(&(cfg->sid), sidstr, sizeof(sidstr));
 
 	if (!dbwrap_fetch_uint32(db, sidstr, &domainnum)) {
 		DEBUG(10, ("Acquiring new range for domain %s\n", sidstr));
@@ -120,11 +117,9 @@ static NTSTATUS idmap_autorid_get_domainrange(struct db_context *db,
 	DEBUG(10, ("Using range #%d for domain %s\n", domainnum, sidstr));
 	cfg->domainnum = domainnum;
 
-	talloc_free(sidstr);
 	return NT_STATUS_OK;
 
       error:
-	talloc_free(sidstr);
 	return ret;
 
 }