idmap-autorid: Slightly simplify idmap_autorid_get_domainrange
authorVolker Lendecke <vl@samba.org>
Tue, 8 Mar 2011 20:55:58 +0000 (21:55 +0100)
committerVolker Lendecke <vl@samba.org>
Fri, 18 Mar 2011 14:46:37 +0000 (15:46 +0100)
Signed-off-by: Christian Ambach <christian.ambach@de.ibm.com>
source3/winbindd/idmap_autorid.c

index 434de25171730e6d103d70d178d47bcac8af706d..b162b623556be2fd68c89f74d826738c8fcd4788 100644 (file)
@@ -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;
 
 }