r9616: Finish ldb_map module (still needs a lot of testing though...)
authorJelmer Vernooij <jelmer@samba.org>
Thu, 25 Aug 2005 15:25:22 +0000 (15:25 +0000)
committerGerald (Jerry) Carter <jerry@samba.org>
Wed, 10 Oct 2007 18:34:35 +0000 (13:34 -0500)
Add initial mapping table for ldb_samba3. Contains most attributes.
(This used to be commit b73441b04d048919e95ee7f7bcae330c4dafefe7)

source4/lib/ldb/ldb_map/ldb_map.c
source4/lib/ldb/ldb_map/ldb_map.h
source4/lib/samba3/ldb_samba3.c

index 39bab2c4c626ea7bf3a3bb5a41585b72b2e78d28..4006ccbbd7d690ea626af5dfd67b1ba8987c5363 100644 (file)
 #include "lib/ldb/include/ldb_private.h"
 #include "lib/ldb/ldb_map/ldb_map.h"
 
+/* TODO:
+ *  - objectclass hint in ldb_map_attribute 
+ *     for use when multiple remote attributes (independant of each other)
+ *     map to one local attribute. E.g.: (uid, gidNumber) -> unixName
+ */
+
 struct map_private {
-       struct ldb_map_mappings *mappings;
+       const struct ldb_map_mappings *mappings;
+       const char *last_err_string;
 };
 
-static struct ldb_dn *ldb_map_dn(struct ldb_module *module, const struct ldb_dn *dn)
+/* find an attribute by the local name */
+static const struct ldb_map_attribute *map_find_attr_local(struct ldb_module *module, const char *attr)
 {
-       /* FIXME */
+       struct map_private *privdat = module->private_data;
+       int i;
+       for (i = 0; privdat->mappings->attribute_maps[i]; i++) {
+               if (!strcmp(privdat->mappings->attribute_maps[i]->local_name, attr)) 
+                       return privdat->mappings->attribute_maps[i];
+       }
+
        return NULL;
 }
 
-static char *ldb_map_expression(struct ldb_module *module, const char *expr)
+/* find an attribute by the remote name */
+static const struct ldb_map_attribute *map_find_attr_remote(struct ldb_module *module, const char *attr)
 {
-       /* FIXME */
+       struct map_private *privdat = module->private_data;
+       int i;
+       for (i = 0; privdat->mappings->attribute_maps[i]; i++) {
+               if (privdat->mappings->attribute_maps[i]->type != MAP_RENAME &&
+                       privdat->mappings->attribute_maps[i]->type != MAP_CONVERT) 
+                       continue;
+
+               if (!strcmp(privdat->mappings->attribute_maps[i]->u.rename.remote_name, attr)) 
+                       return privdat->mappings->attribute_maps[i];
+       }
+
        return NULL;
 }
 
+static struct ldb_parse_tree *ldb_map_parse_tree(struct ldb_module *module, const struct ldb_parse_tree *tree)
+{
+       int i;
+       const struct ldb_map_attribute *attr;
+       struct ldb_parse_tree *new_tree = talloc_memdup(module, tree, sizeof(*tree));
+
+       /* Find attr in question and:
+        *  - if it has a convert_operator function, run that
+        *  - otherwise, replace attr name with required[0] */
+
+       if (tree->operation == LDB_OP_AND || 
+               tree->operation == LDB_OP_OR) {
+               for (i = 0; i < tree->u.list.num_elements; i++) {
+                       new_tree->u.list.elements[i] = ldb_map_parse_tree(module, tree->u.list.elements[i]);
+               }
+
+               return new_tree;
+       }
+               
+       if (tree->operation == LDB_OP_NOT) {
+               new_tree->u.isnot.child = ldb_map_parse_tree(module, tree->u.isnot.child);
+               return new_tree;
+       }
+
+       /* tree->operation is LDB_OP_EQUALITY, LDB_OP_SUBSTRING, LDB_OP_GREATER,
+        * LDB_OP_LESS, LDB_OP_APPROX, LDB_OP_PRESENT or LDB_OP_EXTENDED
+        *
+        * (all have attr as the first element)
+        */
+
+       attr = map_find_attr_local(module, tree->u.equality.attr);
+
+       if (!attr) {
+               DEBUG(0, ("Unable to find local attribute '%s', leaving as is", tree->u.equality.attr));
+               return new_tree;
+       }
+
+       if (attr->type == MAP_IGNORE)
+               return NULL;
+
+       if (attr->convert_operator) {
+               /* Run convert_operator */
+               talloc_free(new_tree);
+               new_tree = attr->convert_operator(module, tree);
+       } else {
+               new_tree->u.equality.attr = talloc_strdup(new_tree, attr->u.rename.remote_name);
+       }
+
+       return new_tree;
+}
+
+/* Remote DN -> Local DN */
+static struct ldb_dn *map_remote_dn(struct ldb_module *module, const struct ldb_dn *dn)
+{
+       struct ldb_dn *newdn;
+       int i;
+
+       newdn = talloc_memdup(module, dn, sizeof(*dn));
+       if (!newdn) 
+               return NULL;
+
+       newdn->components = talloc_memdup(newdn, dn->components, sizeof(struct ldb_dn_component) * newdn->comp_num); 
+
+       if (!newdn->components)
+               return NULL;
+
+       /* For each rdn, map the attribute name and possibly the 
+        * complete rdn */
+       
+       for (i = 0; i < dn->comp_num; i++) {
+               const struct ldb_map_attribute *attr = map_find_attr_remote(module, dn->components[i].name);
+
+               /* Unknown attribute - leave this dn as is and hope the best... */
+               if (!attr)
+                       continue;
+
+               if (attr->type == MAP_IGNORE) {
+                       DEBUG(0, ("Local MAP_IGNORE attribute '%s' used in DN!", dn->components[i].name));
+                       talloc_free(newdn);
+                       return NULL;
+               }
+
+               if (attr->type == MAP_GENERATE) {
+                       DEBUG(0, ("Local MAP_GENERATE attribute '%s' used in DN!", dn->components[i].name));
+                       talloc_free(newdn);
+
+                       return NULL;
+               }
+
+               if (attr->type == MAP_CONVERT) {
+                       struct ldb_message_element elm, *newelm;
+                       struct ldb_val vals[1] = { dn->components[i].value };
+                       elm.flags = 0;
+                       elm.name = attr->u.convert.remote_name;
+                       elm.num_values = 1;
+                       elm.values = vals;
+
+                       newelm = attr->u.convert.convert_remote(module, attr->local_name, &elm);
+
+                       newdn->components[i].name = talloc_strdup(module, newelm->name);
+                       newdn->components[i].value = newelm->values[0];
+               } else if (attr->type == MAP_RENAME) {
+                       newdn->components[i].name = talloc_strdup(module, attr->local_name);
+               }
+       }
+       return newdn;
+}
+
+/* Local DN -> Remote DN */
+static struct ldb_dn *map_local_dn(struct ldb_module *module, const struct ldb_dn *dn)
+{      struct ldb_dn *newdn;
+       int i;
+       struct ldb_parse_tree eqtree, *new_eqtree;
+
+       newdn = talloc_memdup(module, dn, sizeof(*dn));
+       if (!newdn) 
+               return NULL;
+
+       newdn->components = talloc_memdup(newdn, dn->components, sizeof(struct ldb_dn_component) * newdn->comp_num); 
+
+       if (!newdn->components)
+               return NULL;
+
+       /* For each rdn, map the attribute name and possibly the 
+        * complete rdn using an equality convert_operator call */
+       
+       for (i = 0; i < dn->comp_num; i++) {
+               const struct ldb_map_attribute *attr = map_find_attr_local(module, dn->components[i].name);
+
+               /* Unknown attribute - leave this dn as is and hope the best... */
+               if (!attr)
+                       continue;
+
+               if (attr->type == MAP_IGNORE) {
+                       DEBUG(0, ("Local MAP_IGNORE attribute '%s' used in DN!", dn->components[i].name));
+                       talloc_free(newdn);
+                       return NULL;
+               }
+
+               if (attr->type == MAP_GENERATE) {
+                       DEBUG(0, ("Local MAP_GENERATE attribute '%s' used in DN!", dn->components[i].name));
+                       talloc_free(newdn);
+
+                       return NULL;
+               }
+
+               /* Simple rename/convert only */
+               if (attr->convert_operator) {
+                       /* Fancy stuff */
+                       eqtree.operation = LDB_OP_EQUALITY;
+                       eqtree.u.equality.attr = dn->components[i].name;
+                       eqtree.u.equality.value = dn->components[i].value;
+
+                       new_eqtree = ldb_map_parse_tree(module, &eqtree);
+
+                       /* Silently continue for now */
+                       if (!new_eqtree) {
+                               DEBUG(0, ("Unable to convert RDN for attribute %s\n", dn->components[i].name));
+                               continue;
+                       }
+
+                       newdn->components[i].name = new_eqtree->u.equality.attr;
+                       newdn->components[i].value = new_eqtree->u.equality.value;
+               } else if (attr->type == MAP_CONVERT) {
+                       struct ldb_message_element elm, *newelm;
+                       struct ldb_val vals[1] = { dn->components[i].value };
+                       elm.flags = 0;
+                       elm.name = attr->local_name;
+                       elm.num_values = 1;
+                       elm.values = vals;
+
+                       newelm = attr->u.convert.convert_local(module, attr->u.convert.remote_name, &elm);
+
+                       newdn->components[i].name = talloc_strdup(module, newelm->name);
+                       newdn->components[i].value = newelm->values[0];
+               } else if (attr->type == MAP_RENAME) {
+                       newdn->components[i].name = talloc_strdup(module, attr->u.rename.remote_name);
+               }
+       }
+
+       return newdn;
+}
+
+
+
+/* Loop over ldb_map_attribute array and add remote_names */
 static const char **ldb_map_attrs(struct ldb_module *module, const char *const attrs[])
 {
-       /* FIXME */
+       int i;
+       const char **ret;
+       int ar_size = 0, last_element = 0;
+
+       /* Start with good guess of number of elements */
+       for (i = 0; attrs[i]; i++);
+
+       ret = talloc_array(module, const char *, i);
+       ar_size = i;
+
+       for (i = 0; attrs[i]; i++) {
+               int j;
+               const struct ldb_map_attribute *attr = map_find_attr_local(module, attrs[i]);
+
+               if (!attr) {
+                       DEBUG(0, ("Local attribute '%s' does not have a definition!\n", attrs[i]));
+                       continue;
+               }
+
+               switch (attr->type)
+               { 
+                       case MAP_IGNORE: break;
+                       case MAP_KEEP: 
+                               if (last_element >= ar_size) {
+                                       ret = talloc_realloc(module, ret, const char *, ar_size+1);
+                                       ar_size++;
+                               }
+                               ret[last_element] = attr->local_name;
+                               last_element++;
+                               break;
+
+                       case MAP_RENAME:
+                       case MAP_CONVERT:
+                               if (last_element >= ar_size) {
+                                       ret = talloc_realloc(module, ret, const char *, ar_size+1);
+                                       ar_size++;
+                               }
+                               ret[last_element] = attr->u.rename.remote_name;
+                               last_element++;
+                               break;
+
+                       case MAP_GENERATE:
+                               /* Add remote_names[] for this attribute to the list of 
+                                * attributes to request from the remote server */
+                               for (j = 0; attr->u.generate.remote_names[j]; j++) {
+                                       if (last_element >= ar_size) {
+                                               ret = talloc_realloc(module, ret, const char *, ar_size+1);
+                                               ar_size++;
+                                       }
+                                       ret[last_element] = attr->u.generate.remote_names[j];                   
+                                       last_element++;
+                               }
+                               break;
+               } 
+       }
+
        return NULL;
 }
 
-static struct ldb_message *ldb_map_message_incoming(struct ldb_module *module, const struct ldb_message *mi)
+static struct ldb_message *ldb_map_message_incoming(struct ldb_module *module, const char * const*attrs, const struct ldb_message *mi)
 {
-       /* FIXME */
-       return NULL;
+       int i;
+       struct ldb_message *msg = talloc_zero(module, struct ldb_message);
+       struct ldb_message_element *elm, *oldelm;
+
+       msg->dn = map_remote_dn(module, mi->dn);
+
+       /* Loop over attrs, find in ldb_map_attribute array and 
+        * run generate() */
+
+       for (i = 0; attrs[i]; i++) {
+               const struct ldb_map_attribute *attr = map_find_attr_local(module, attrs[i]);
+
+               if (!attr) {
+                       DEBUG(0, ("Unable to find local attribute '%s' when generating incoming message", attrs[i]));
+                       continue;
+               }
+
+               switch (attr->type) {
+                       case MAP_IGNORE:break;
+                       case MAP_RENAME:
+                               oldelm = ldb_msg_find_element(mi, attr->u.rename.remote_name);
+                               elm = talloc_memdup(msg, oldelm, sizeof(*oldelm));
+                               elm->name = talloc_strdup(elm, attr->local_name);
+
+                               ldb_msg_add(module->ldb, msg, elm, 0);
+                               break;
+                               
+                       case MAP_CONVERT:
+                               oldelm = ldb_msg_find_element(mi, attr->u.rename.remote_name);
+                               elm = attr->u.convert.convert_local(msg, attr->local_name, oldelm);
+
+                               ldb_msg_add(module->ldb, msg, elm, 0);
+                               break;
+
+                       case MAP_KEEP:
+                               ldb_msg_add(module->ldb, msg, ldb_msg_find_element(mi, attr->local_name), 0);
+                               break;
+
+                       case MAP_GENERATE:
+                               elm = attr->u.generate.generate_local(msg, attr->local_name, mi);
+                               ldb_msg_add(module->ldb, msg, elm, 0);
+                               break;
+                       default: 
+                               DEBUG(0, ("Unknown attr->type for %s", attr->local_name));
+                               break;
+               }
+       }
+
+       return msg;
 }
 
-static struct ldb_message *ldb_map_message_outgoing(struct ldb_module *module, const struct ldb_message *mi)
+static struct ldb_message *ldb_map_message_outgoing(struct ldb_module *module, const struct ldb_message *mo)
 {
-       /* FIXME */
-       return NULL;
+       struct ldb_message *msg = talloc_zero(module, struct ldb_message);
+       struct ldb_message_element *elm;
+       int i;
+       
+       msg->private_data = mo->private_data;
+       
+       msg->dn = map_local_dn(module, mo->dn);
+
+       /* Loop over mi and call generate_remote for each attribute */
+       for (i = 0; i < mo->num_elements; i++) {
+               const struct ldb_map_attribute *attr = map_find_attr_local(module, mo->elements[i].name);
+
+               if (!attr) {
+                       DEBUG(0, ("Undefined local attribute '%s', ignoring\n", mo->elements[i].name));
+                       continue;
+               }
+
+               switch (attr->type) {
+               case MAP_IGNORE: break;
+               case MAP_RENAME:
+                       elm = talloc_memdup(msg, &msg->elements[i], sizeof(*elm));
+                       elm->name = talloc_strdup(elm, attr->u.rename.remote_name);
+
+                       ldb_msg_add(module->ldb, msg, elm, 0);
+                       break;
+
+               case MAP_CONVERT:
+                       elm = attr->u.convert.convert_remote(msg, attr->local_name, &msg->elements[i]);
+                       ldb_msg_add(module->ldb, msg, elm, 0);
+                       break;
+
+               case MAP_KEEP:
+                       ldb_msg_add(module->ldb, msg, &msg->elements[i], 0);    
+                       break;
+
+               case MAP_GENERATE:
+                       attr->u.generate.generate_remote(attr->local_name, mo, msg);
+                       break;
+               } 
+       }
+
+       return msg;
 }
 
 /*
@@ -69,8 +422,8 @@ static int map_rename(struct ldb_module *module, const struct ldb_dn *olddn, con
        struct ldb_dn *n_olddn, *n_newdn;
        int ret;
        
-       n_olddn = ldb_map_dn(module, olddn);
-       n_newdn = ldb_map_dn(module, newdn);
+       n_olddn = map_local_dn(module, olddn);
+       n_newdn = map_local_dn(module, newdn);
 
        ret = ldb_next_rename_record(module, n_olddn, n_newdn);
 
@@ -88,7 +441,7 @@ static int map_delete(struct ldb_module *module, const struct ldb_dn *dn)
        struct ldb_dn *newdn;
        int ret;
 
-       newdn = ldb_map_dn(module, dn);
+       newdn = map_local_dn(module, dn);
 
        ret = ldb_next_delete_record(module, newdn);
 
@@ -97,38 +450,58 @@ static int map_delete(struct ldb_module *module, const struct ldb_dn *dn)
        return ret;
 }
 
-
 /*
-  search for matching records
+  search for matching records using a ldb_parse_tree
 */
-static int map_search(struct ldb_module *module, const struct ldb_dn *base,
-                      enum ldb_scope scope, const char *expression,
-                      const char * const *attrs, struct ldb_message ***res)
+static int map_search_bytree(struct ldb_module *module, const struct ldb_dn *base,
+                             enum ldb_scope scope, struct ldb_parse_tree *tree,
+                             const char * const *attrs, struct ldb_message ***res)
 {
-       char *newexpr;
        int ret;
        const char **newattrs;
+       struct ldb_parse_tree *new_tree;
        struct ldb_dn *new_base;
        struct ldb_message **newres;
        int i;
 
-       newexpr = ldb_map_expression(module, expression);
+       new_tree = ldb_map_parse_tree(module, tree);
        newattrs = ldb_map_attrs(module, attrs); 
-       new_base = ldb_map_dn(module, base);
+       new_base = map_local_dn(module, base);
 
-       ret = ldb_next_search(module, new_base, scope, newexpr, newattrs, &newres);
+       ret = ldb_next_search_bytree(module, new_base, scope, new_tree, newattrs, &newres);
 
        talloc_free(new_base);
-       talloc_free(newexpr);
+       talloc_free(new_tree);
        talloc_free(newattrs);
 
        for (i = 0; i < ret; i++) {
-               *res[i] = ldb_map_message_incoming(module, newres[i]);
+               *res[i] = ldb_map_message_incoming(module, attrs, newres[i]);
                talloc_free(newres[i]);
        }
 
        return ret;
 }
+/*
+  search for matching records
+*/
+static int map_search(struct ldb_module *module, const struct ldb_dn *base,
+                      enum ldb_scope scope, const char *expression,
+                      const char * const *attrs, struct ldb_message ***res)
+{
+       struct map_private *map = module->private_data;
+       struct ldb_parse_tree *tree;
+       int ret;
+
+       tree = ldb_parse_tree(map, expression);
+       if (tree == NULL) {
+               map->last_err_string = "expression parse failed";
+               return -1;
+       }
+
+       ret = map_search_bytree(module, base, scope, tree, attrs, res);
+       talloc_free(tree);
+       return ret;
+}
 
 /*
   add a record
@@ -146,25 +519,7 @@ static int map_add(struct ldb_module *module, const struct ldb_message *msg)
 }
 
 
-/*
-  search for matching records using a ldb_parse_tree
-*/
-static int map_search_bytree(struct ldb_module *module, const struct ldb_dn *base,
-                             enum ldb_scope scope, struct ldb_parse_tree *tree,
-                             const char * const *attrs, struct ldb_message ***res)
-{
-       struct map_private *privdat = module->private_data;
-       char *expression;
-       int ret;
 
-       expression = ldb_filter_from_tree(privdat, tree);
-       if (expression == NULL) {
-               return -1;
-       }
-       ret = map_search(module, base, scope, expression, attrs, res);
-       talloc_free(expression);
-       return ret;
-}
 
 /*
   modify a record
@@ -196,6 +551,11 @@ static int map_unlock(struct ldb_module *module, const char *lockname)
 */
 static const char *map_errstring(struct ldb_module *module)
 {
+       struct map_private *map = module->private_data;
+       
+       if (map->last_err_string)
+               return map->last_err_string;
+
        return ldb_next_errstring(module);
 }
 
@@ -213,7 +573,7 @@ static const struct ldb_module_ops map_ops = {
 };
 
 /* the init function */
-struct ldb_module *ldb_map_init(struct ldb_context *ldb, struct ldb_map_mappings *mappings, const char *options[])
+struct ldb_module *ldb_map_init(struct ldb_context *ldb, const struct ldb_map_mappings *mappings, const char *options[])
 {
        struct ldb_module *ctx;
        struct map_private *data;
index 3ebf2fa9d120a75abd611728d7d46af1afd29f5f..da3198429df488650cf4c8e4336fab5aefafea61 100644 (file)
 #ifndef __LDB_MAP_H__
 #define __LDB_MAP_H__
 
-struct ldb_map_mappings 
+/* ldb_map is a skeleton LDB module that can be used for any other modules
+ * that need to map attributes.
+ *
+ * The term 'remote' in this header refers to the connection where the 
+ * original schema is used on while 'local' means the local connection 
+ * that any upper layers will use.
+ *
+ * All local attributes will have to have a definition. Not all remote 
+ * attributes need a definition as LDB is a lot less stricter then LDAP 
+ * (in other words, sending unknown attributes to an LDAP server hurts us, 
+ * returning too much attributes in ldb_search() doesn't)
+ */
+
+struct ldb_map_attribute 
 {
+       const char *local_name; /* local name */
+
+       enum { 
+               MAP_IGNORE, /* Ignore this local attribute. Doesn't exist remotely.  */
+               MAP_KEEP,   /* Keep as is */
+               MAP_RENAME, /* Simply rename the attribute. Name changes, data is the same */
+               MAP_CONVERT, /* Rename + convert data */
+               MAP_GENERATE /* Use generate function for generating new name/data. 
+                                               Used for generating attributes based on 
+                                               multiple remote attributes. */
+       } type;
+       
+       /* if set, will be called for expressions that contain this attribute */
+       struct ldb_parse_tree *(*convert_operator) (TALLOC_CTX *ctx, const struct ldb_parse_tree *);    
+
+       union { 
+               struct {
+                       const char *remote_name;
+               } rename;
+               
+               struct {
+                       const char *remote_name;
 
+                       struct ldb_message_element *(*convert_local) (
+                               TALLOC_CTX *ctx, 
+                               const char *remote_attr,
+                               const struct ldb_message_element *);
+
+                       struct ldb_message_element *(*convert_remote) (
+                               TALLOC_CTX *ctx,
+                               const char *local_attr,
+                               const struct ldb_message_element *);
+               } convert;
+       
+               struct {
+                       /* Generate the local attribute from remote message */
+                       struct ldb_message_element *(*generate_local) (
+                                       TALLOC_CTX *ctx, 
+                                       const char *attr, 
+                                       const struct ldb_message *remote);
+
+                       /* Update remote message with information from local message */
+                       void (*generate_remote) (
+                                       const char *local_attr,
+                                       const struct ldb_message *local, 
+                                       struct ldb_message *remote);
+
+                       /* Name(s) for this attribute on the remote server. This is an array since 
+                        * one local attribute's data can be split up into several attributes 
+                        * remotely */
+                       const char *remote_names[];
+               } generate;
+       } u;
 };
 
+struct ldb_map_objectclass 
+{
+       const char *local_name;
+       const char *remote_name;
+};
+
+/* Base ldb_map struct. Fill this in to create a mapping backend */
+struct ldb_map_mappings 
+{
+       const char *name;
+       const struct ldb_map_attribute *attribute_maps[];
+};
+
+
 #endif /* __LDB_MAP_H__ */
index 030b0519387b467ee6041fa12be96499b11387fd..53d8964d0db613bb2ec38897b131e5a7a72dd77b 100644 (file)
 #include "ldb/include/ldb_private.h"
 
 /* 
- * sambaGroupMapping -> group
  *     gidNumber -> ???
  *     sambaSID -> member
- *     sambaGroupType -> groupType
- *     displayName -> name
- *     description -> description
  *     sambaSIDList -> member (special!)
  */
 
-/*
- * sambaTrustPassword
- */
+/* sambaTrustPassword */
 
-/* sambaDomain
- *     sambaDomainName
- *     sambaSID
- *     sambaNextRid 
- *     sambaNextGroupRid 
- *     sambaNextUserRid
- *     sambaAlgorithmicRidBase
- */
+/*     sambaDomainName -> name */
 
-/* sambaUnixIdPool
- */
+/* sambaUnixIdPool */
 
 /* sambaIdmapEntry */
 
-/* sambaAccountPolicy */
+/* sambaAccountPolicy: FIXME */
 
 /* sambaSidEntry: FIXME */
 
-/* sambaSamAccount -> user:
- * uid -> unixName (magic!)
- * sambaSID -> objectSid
- * cn -> cn
- * sambaLMPassword -> lmPwdHash
- * sambaNTPassword -> ntPwdHash
- * sambaPwdLastSet -> pwdLastSet
- * sambaLogonTime -> lastLogon
- * sambaLogoffTime -> lastLogoff
- * sambaKickoffTime -> ???
- * sambaPwdCanChange  -> ???
- * sambaPwdMustChange  -> ???
- * sambaAcctFlags -> systemFlags ?
- * displayName  -> name
- * sambaHomePath  -> ???
- * sambaHomeDrive  -> ???
- * sambaLogonScript  -> ???
- * sambaProfilePath  -> ???
- * description -> description
- * sambaUserWorkstations  -> ???
- * sambaPrimaryGroupSID -> primaryGroupID
- * sambaDomainName  -> ???
- * sambaMungedDial -> ???
- * sambaBadPasswordCount -> badPwdcount
- * sambaBadPasswordTime -> badPasswordtime
- * sambaPasswordHistory  -> ntPwdHistory
- * sambaLogonHours -> ???
- */
-
 /* Not necessary:
  * sambaConfig
  * sambaShare
  * sambaConfigOption 
+ * sambaNextGroupRid
+ * sambaNextUserRid
+ * sambaAlgorithmicRidBase
  */
 
-
-struct ldb_map_mappings samba3_mappings;
+/* sambaKickoffTime -> ???*/
+/* sambaPwdCanChange  -> ???*/
+/* sambaPwdMustChange  -> ???*/
+/* sambaAcctFlags -> systemFlags ?*/
+/* sambaHomePath  -> ???*/
+/* sambaHomeDrive  -> ???*/
+/* sambaLogonScript  -> ???*/
+/* sambaProfilePath  -> ???*/
+/* sambaUserWorkstations  -> ???*/
+/* sambaDomainName  -> ???*/
+/* sambaMungedDial -> ???*/
+/* sambaPasswordHistory  -> ntPwdHistory*/
+/* sambaLogonHours -> ???*/
+
+/* sambaNextRid -> nextRid */
+const struct ldb_map_attribute attr_nextRid = {
+       .local_name = "nextRid",
+       .type = MAP_RENAME,
+       .u.rename.remote_name = "sambaNextRid",
+};
+
+/* sambaBadPasswordTime -> badPasswordtime*/
+const struct ldb_map_attribute attr_badPasswordTime = {
+       .local_name = "badPasswordTime",
+       .type = MAP_RENAME,
+       .u.rename.remote_name = "sambaBadPasswordTime",
+};
+
+/* sambaLMPassword -> lmPwdHash*/
+const struct ldb_map_attribute attr_lmPwdHash = {
+       .local_name = "lmPwdHash",
+       .type = MAP_RENAME,
+       .u.rename.remote_name = "sambaLMPassword",
+};
+
+/* sambaGroupType -> groupType */
+const struct ldb_map_attribute attr_groupType = {
+       .local_name = "groupType",
+       .type = MAP_RENAME,
+       .u.rename.remote_name = "sambaGroupType",
+};
+
+/* sambaNTPassword -> ntPwdHash*/
+const struct ldb_map_attribute attr_ntPwdHash = {
+       .local_name = "badPwdCount",
+       .type = MAP_RENAME,
+       .u.rename.remote_name = "sambaNTPassword",
+};
+
+/* sambaPrimaryGroupSID -> primaryGroupID */
+const struct ldb_map_attribute attr_primaryGroupID = {
+       .local_name = "primaryGroupID",
+       .type = MAP_CONVERT,
+       .u.convert.remote_name = "sambaPrimaryGroupSID",
+       .u.convert.convert_local = NULL, /* FIXME: Add domain SID */
+       .u.convert.convert_remote = NULL, /* FIXME: Extract RID */
+};
+
+/* sambaBadPasswordCount -> badPwdCount */
+const struct ldb_map_attribute attr_badPwdCount = {
+       .local_name = "badPwdCount",
+       .type = MAP_RENAME,
+       .u.rename.remote_name = "sambaBadPasswordCount",
+};
+
+/* sambaLogonTime -> lastLogon*/
+const struct ldb_map_attribute attr_lastLogon = {
+       .local_name = "lastLogon",
+       .type = MAP_RENAME,
+       .u.rename.remote_name = "sambaLogonTime",
+};
+
+/* sambaLogoffTime -> lastLogoff*/
+const struct ldb_map_attribute attr_lastLogoff = {
+       .local_name = "lastLogoff",
+       .type = MAP_RENAME,
+       .u.rename.remote_name = "sambaLogoffTime",
+};
+
+/* gidNumber -> unixName */
+const struct ldb_map_attribute attr_unixName_gid = {
+       .local_name = "unixName",
+       .type = MAP_CONVERT,
+       .u.convert.remote_name = "gidNumber",
+       .u.convert.convert_local = NULL, /* FIXME: Lookup gid */
+       .u.convert.convert_remote = NULL, /* FIXME: Lookup groupname */
+};
+
+/* uid -> unixName */
+const struct ldb_map_attribute attr_unixName_uid = {
+       .local_name = "unixName",
+       .type = MAP_CONVERT,
+       .u.convert.remote_name = "uid",
+       .u.convert.convert_local = NULL, /* FIXME: Lookup uid */
+       .u.convert.convert_remote = NULL, /* FIXME: Lookup username */
+};
+
+/* displayName -> name */
+const struct ldb_map_attribute attr_name = {
+       .local_name = "name",
+       .type = MAP_RENAME,
+       .u.rename.remote_name = "displayName",
+};
+
+/* cn */
+const struct ldb_map_attribute attr_cn = {
+       .local_name = "cn",
+       .type = MAP_KEEP,
+};
+
+/* description */
+const struct ldb_map_attribute attr_description = {
+       .local_name = "description",
+       .type = MAP_KEEP,
+};
+
+/* sambaSID -> objectSid*/
+const struct ldb_map_attribute attr_objectSid = {
+       .local_name = "objectSid",
+       .type = MAP_RENAME,
+       .u.rename.remote_name = "sambaSID", 
+};
+
+/* sambaPwdLastSet -> pwdLastSet*/
+const struct ldb_map_attribute attr_pwdLastSet = {
+       .local_name = "pwdLastSet",
+       .type = MAP_RENAME,
+       .u.rename.remote_name = "sambaPwdLastSet",
+};
+
+const struct ldb_map_objectclass samba3_objectclasses[] = {
+       { "group", "sambaGroupMapping" },
+       { "user", "sambaSAMAccount" },
+       { "domain", "sambaDomain" },
+};
+
+const struct ldb_map_mappings samba3_mappings = 
+{
+       .name = "samba3",
+       {
+               &attr_objectSid,
+               &attr_pwdLastSet,
+               &attr_description,
+               &attr_cn,
+               &attr_unixName_uid,
+               &attr_unixName_gid,
+               &attr_name,
+               &attr_lastLogoff,
+               &attr_lastLogon,
+               &attr_primaryGroupID,
+               &attr_badPwdCount,
+               &attr_ntPwdHash,
+               &attr_lmPwdHash,
+               &attr_groupType,
+               &attr_badPasswordTime,
+               &attr_nextRid,
+       }
+};     
 
 /* the init function */
 #ifdef HAVE_DLOPEN_DISABLED