On Wed, 2009-11-04 at 15:05 +0100, Sumit Bose wrote: > + if (e != NULL) { > + dummy = talloc_strdup(talloc_parent(e->name), newname); The parent should be "attrs", I would use that and not talloc_parent(), so even if the parent is something "else", we get the new parent right. Simo. -- Simo Sorce * Red Hat, Inc * New York