s3/s4 build: Fix Py_RETURN_NONE to work with python versions < 2.4
[tprouty/samba.git] / source4 / lib / ldb / pyldb.c
index 53b4fcef3dd49149aeb54004128800c7011ba8bc..b7a78e8f531607ff48a05e287d0af776eca865da 100644 (file)
    License along with this library; if not, see <http://www.gnu.org/licenses/>.
 */
 
-#include <stdint.h>
-#include <stdbool.h>
+#include "ldb_includes.h"
+#include "../lib/util/python_util.h"
 #include "pyldb.h"
-#include "events.h"
-#include "ldb.h"
-#include "ldb_errors.h"
-#include "ldb_private.h"
 
-PyObject *PyExc_LdbError;
+/* There's no Py_ssize_t in 2.4, apparently */
+#if PY_MAJOR_VERSION == 2 && PY_MINOR_VERSION < 5
+typedef int Py_ssize_t;
+typedef inquiry lenfunc;
+typedef intargfunc ssizeargfunc;
+#endif
+
+/* Picked out of thin air. To do this properly, we should probably have some part of the 
+ * errors in LDB be allocated to bindings ? */
+#define LDB_ERR_PYTHON_EXCEPTION       142
+
+static PyObject *PyExc_LdbError;
 
 void PyErr_SetLdbError(int ret, struct ldb_context *ldb_ctx)
 {
+       if (ret == LDB_ERR_PYTHON_EXCEPTION)
+               return; /* Python exception should already be set, just keep that */
        PyErr_SetObject(PyExc_LdbError, Py_BuildValue((char *)"(i,s)", ret, ldb_ctx == NULL?ldb_strerror(ret):ldb_errstring(ldb_ctx)));
 }
-PyObject *PyObject_FromLdbValue(struct ldb_context *ldb_ctx, 
+static PyObject *PyObject_FromLdbValue(struct ldb_context *ldb_ctx, 
                                                           struct ldb_message_element *el, 
                                                           struct ldb_val *val)
 {
@@ -93,12 +102,11 @@ static PyObject *PyLdbResult_FromResult(struct ldb_result *result)
        PyObject *ret;
        int i;
        if (result == NULL) {
-               return Py_None;
+               Py_RETURN_NONE;
        } 
        ret = PyList_New(result->count);
        for (i = 0; i < result->count; i++) {
-               PyList_SetItem(ret, i, 
-                                          PyLdbMessage_FromMessage(result->msgs[i])
+               PyList_SetItem(ret, i, PyLdbMessage_FromMessage(result->msgs[i])
                );
        }
        return ret;
@@ -124,47 +132,47 @@ static struct ldb_result *PyLdbResult_AsResult(TALLOC_CTX *mem_ctx, PyObject *ob
 
 static PyObject *py_ldb_dn_validate(PyLdbDnObject *self)
 {
-       return PyBool_FromLong(ldb_dn_validate(self->ptr));
+       return PyBool_FromLong(ldb_dn_validate(self->dn));
 }
 
 static PyObject *py_ldb_dn_is_valid(PyLdbDnObject *self)
 {
-       return PyBool_FromLong(ldb_dn_is_valid(self->ptr));
+       return PyBool_FromLong(ldb_dn_is_valid(self->dn));
 }
 
 static PyObject *py_ldb_dn_is_special(PyLdbDnObject *self)
 {
-       return PyBool_FromLong(ldb_dn_is_special(self->ptr));
+       return PyBool_FromLong(ldb_dn_is_special(self->dn));
 }
 
 static PyObject *py_ldb_dn_is_null(PyLdbDnObject *self)
 {
-       return PyBool_FromLong(ldb_dn_is_null(self->ptr));
+       return PyBool_FromLong(ldb_dn_is_null(self->dn));
 }
  
 static PyObject *py_ldb_dn_get_casefold(PyLdbDnObject *self)
 {
-       return PyString_FromString(ldb_dn_get_casefold(self->ptr));
+       return PyString_FromString(ldb_dn_get_casefold(self->dn));
 }
 
 static PyObject *py_ldb_dn_get_linearized(PyLdbDnObject *self)
 {
-       return PyString_FromString(ldb_dn_get_linearized(self->ptr));
+       return PyString_FromString(ldb_dn_get_linearized(self->dn));
 }
 
 static PyObject *py_ldb_dn_canonical_str(PyLdbDnObject *self)
 {
-       return PyString_FromString(ldb_dn_canonical_string(self->ptr, self->ptr));
+       return PyString_FromString(ldb_dn_canonical_string(self->dn, self->dn));
 }
 
 static PyObject *py_ldb_dn_canonical_ex_str(PyLdbDnObject *self)
 {
-       return PyString_FromString(ldb_dn_canonical_ex_string(self->ptr, self->ptr));
+       return PyString_FromString(ldb_dn_canonical_ex_string(self->dn, self->dn));
 }
 
 static PyObject *py_ldb_dn_repr(PyLdbDnObject *self)
 {
-       return PyString_FromFormat("Dn(%s)", PyObject_REPR(PyString_FromString(ldb_dn_get_linearized(self->ptr))));
+       return PyString_FromFormat("Dn(%s)", PyObject_REPR(PyString_FromString(ldb_dn_get_linearized(self->dn))));
 }
 
 static PyObject *py_ldb_dn_check_special(PyLdbDnObject *self, PyObject *args)
@@ -174,12 +182,12 @@ static PyObject *py_ldb_dn_check_special(PyLdbDnObject *self, PyObject *args)
        if (!PyArg_ParseTuple(args, "s", &name))
                return NULL;
 
-       return ldb_dn_check_special(self->ptr, name)?Py_True:Py_False;
+       return ldb_dn_check_special(self->dn, name)?Py_True:Py_False;
 }
 
 static int py_ldb_dn_compare(PyLdbDnObject *dn1, PyLdbDnObject *dn2)
 {
-       return ldb_dn_compare(dn1->ptr, dn2->ptr);
+       return ldb_dn_compare(dn1->dn, dn2->dn);
 }
 
 static PyObject *py_ldb_dn_get_parent(PyLdbDnObject *self)
@@ -284,6 +292,7 @@ static PyObject *py_ldb_dn_new(PyTypeObject *type, PyObject *args, PyObject *kwa
        char *str;
        PyObject *py_ldb;
        struct ldb_context *ldb_ctx;
+       PyLdbDnObject *py_ret;
        const char *kwnames[] = { "ldb", "dn", NULL };
 
        if (!PyArg_ParseTupleAndKeywords(args, kwargs, "Os", (char **)kwnames, &py_ldb, &str))
@@ -301,12 +310,32 @@ static PyObject *py_ldb_dn_new(PyTypeObject *type, PyObject *args, PyObject *kwa
                return NULL;
        }
 
-       return py_talloc_import(&PyLdbDn, ret);
+       py_ret = (PyLdbDnObject *)type->tp_alloc(type, 0);
+       if (ret == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+       py_ret->dn = ret;
+       return (PyObject *)py_ret;
 }
 
 PyObject *PyLdbDn_FromDn(struct ldb_dn *dn)
 {
-       return py_talloc_import(&PyLdbDn, dn);
+       PyLdbDnObject *py_ret;
+       py_ret = (PyLdbDnObject *)PyLdbDn.tp_alloc(&PyLdbDn, 0);
+       if (py_ret == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+       py_ret->mem_ctx = talloc_new(NULL);
+       py_ret->dn = talloc_reference(py_ret->mem_ctx, dn);
+       return (PyObject *)py_ret;
+}
+
+static void py_ldb_dn_dealloc(PyLdbDnObject *self)
+{
+       talloc_free(self->mem_ctx);
+       self->ob_type->tp_free(self);
 }
 
 PyTypeObject PyLdbDn = {
@@ -318,8 +347,9 @@ PyTypeObject PyLdbDn = {
        .tp_as_sequence = &py_ldb_dn_seq,
        .tp_doc = "A LDB distinguished name.",
        .tp_new = py_ldb_dn_new,
-       .tp_dealloc = py_talloc_dealloc,
+       .tp_dealloc = (destructor)py_ldb_dn_dealloc,
        .tp_basicsize = sizeof(PyLdbObject),
+       .tp_flags = Py_TPFLAGS_DEFAULT,
 };
 
 /* Debug */
@@ -339,9 +369,9 @@ static PyObject *py_ldb_set_debug(PyLdbObject *self, PyObject *args)
 
        Py_INCREF(cb);
        /* FIXME: Where do we DECREF cb ? */
-       PyErr_LDB_ERROR_IS_ERR_RAISE(ldb_set_debug(self->ptr, py_ldb_debug, cb), PyLdb_AsLdbContext(self));
+       PyErr_LDB_ERROR_IS_ERR_RAISE(ldb_set_debug(self->ldb_ctx, py_ldb_debug, cb), PyLdb_AsLdbContext(self));
        
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_set_create_perms(PyTypeObject *self, PyObject *args)
@@ -352,7 +382,7 @@ static PyObject *py_ldb_set_create_perms(PyTypeObject *self, PyObject *args)
 
        ldb_set_create_perms(PyLdb_AsLdbContext(self), perms);
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_set_modules_dir(PyTypeObject *self, PyObject *args)
@@ -363,31 +393,31 @@ static PyObject *py_ldb_set_modules_dir(PyTypeObject *self, PyObject *args)
 
        ldb_set_modules_dir(PyLdb_AsLdbContext(self), modules_dir);
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_transaction_start(PyLdbObject *self)
 {
        PyErr_LDB_ERROR_IS_ERR_RAISE(ldb_transaction_start(PyLdb_AsLdbContext(self)), PyLdb_AsLdbContext(self));
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_transaction_commit(PyLdbObject *self)
 {
        PyErr_LDB_ERROR_IS_ERR_RAISE(ldb_transaction_commit(PyLdb_AsLdbContext(self)), PyLdb_AsLdbContext(self));
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_transaction_cancel(PyLdbObject *self)
 {
        PyErr_LDB_ERROR_IS_ERR_RAISE(ldb_transaction_cancel(PyLdb_AsLdbContext(self)), PyLdb_AsLdbContext(self));
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_setup_wellknown_attributes(PyLdbObject *self)
 {
        PyErr_LDB_ERROR_IS_ERR_RAISE(ldb_setup_wellknown_attributes(PyLdb_AsLdbContext(self)), PyLdb_AsLdbContext(self));
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_repr(PyLdbObject *self)
@@ -399,7 +429,7 @@ static PyObject *py_ldb_get_root_basedn(PyLdbObject *self)
 {
        struct ldb_dn *dn = ldb_get_root_basedn(PyLdb_AsLdbContext(self));
        if (dn == NULL)
-               return Py_None;
+               Py_RETURN_NONE;
        return PyLdbDn_FromDn(dn);
 }
 
@@ -408,7 +438,7 @@ static PyObject *py_ldb_get_schema_basedn(PyLdbObject *self)
 {
        struct ldb_dn *dn = ldb_get_schema_basedn(PyLdb_AsLdbContext(self));
        if (dn == NULL)
-               return Py_None;
+               Py_RETURN_NONE;
        return PyLdbDn_FromDn(dn);
 }
 
@@ -417,7 +447,7 @@ static PyObject *py_ldb_get_config_basedn(PyLdbObject *self)
 {
        struct ldb_dn *dn = ldb_get_config_basedn(PyLdb_AsLdbContext(self));
        if (dn == NULL)
-               return Py_None;
+               Py_RETURN_NONE;
        return PyLdbDn_FromDn(dn);
 }
 
@@ -426,7 +456,7 @@ static PyObject *py_ldb_get_default_basedn(PyLdbObject *self)
 {
        struct ldb_dn *dn = ldb_get_default_basedn(PyLdb_AsLdbContext(self));
        if (dn == NULL)
-               return Py_None;
+               Py_RETURN_NONE;
        return PyLdbDn_FromDn(dn);
 }
 
@@ -489,6 +519,7 @@ static int py_ldb_init(PyLdbObject *self, PyObject *args, PyObject *kwargs)
 
 static PyObject *py_ldb_new(PyTypeObject *type, PyObject *args, PyObject *kwargs)
 {
+       PyLdbObject *ret;
        struct ldb_context *ldb;
        ldb = ldb_init(NULL, event_context_init(NULL)); 
        if (ldb == NULL) {
@@ -496,13 +527,19 @@ static PyObject *py_ldb_new(PyTypeObject *type, PyObject *args, PyObject *kwargs
                return NULL;
        }
 
-       return py_talloc_import(type, ldb);
+       ret = (PyLdbObject *)type->tp_alloc(type, 0);
+       if (ret == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+       ret->ldb_ctx = ldb;
+       return (PyObject *)ret;
 }
 
 static PyObject *py_ldb_connect(PyLdbObject *self, PyObject *args, PyObject *kwargs)
 {
        char *url;
-       int flags;
+       int flags = 0;
        PyObject *py_options = Py_None;
        int ret;
        const char **options;
@@ -524,7 +561,7 @@ static PyObject *py_ldb_connect(PyLdbObject *self, PyObject *args, PyObject *kwa
 
        PyErr_LDB_ERROR_IS_ERR_RAISE(ret, PyLdb_AsLdbContext(self));
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_modify(PyLdbObject *self, PyObject *args)
@@ -542,7 +579,7 @@ static PyObject *py_ldb_modify(PyLdbObject *self, PyObject *args)
        ret = ldb_modify(PyLdb_AsLdbContext(self), PyLdbMessage_AsMessage(py_msg));
        PyErr_LDB_ERROR_IS_ERR_RAISE(ret, PyLdb_AsLdbContext(self));
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_add(PyLdbObject *self, PyObject *args)
@@ -599,7 +636,7 @@ static PyObject *py_ldb_add(PyLdbObject *self, PyObject *args)
        ret = ldb_add(PyLdb_AsLdbContext(self), msg);
        PyErr_LDB_ERROR_IS_ERR_RAISE(ret, PyLdb_AsLdbContext(self));
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 
@@ -609,16 +646,19 @@ static PyObject *py_ldb_delete(PyLdbObject *self, PyObject *args)
        PyObject *py_dn;
        struct ldb_dn *dn;
        int ret;
+       struct ldb_context *ldb;
        if (!PyArg_ParseTuple(args, "O", &py_dn))
                return NULL;
 
-       if (!PyObject_AsDn(NULL, py_dn, PyLdb_AsLdbContext(self), &dn))
+       ldb = PyLdb_AsLdbContext(self);
+
+       if (!PyObject_AsDn(NULL, py_dn, ldb, &dn))
                return NULL;
 
-       ret = ldb_delete(PyLdb_AsLdbContext(self), dn);
-       PyErr_LDB_ERROR_IS_ERR_RAISE(ret, PyLdb_AsLdbContext(self));
+       ret = ldb_delete(ldb, dn);
+       PyErr_LDB_ERROR_IS_ERR_RAISE(ret, ldb);
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_rename(PyLdbObject *self, PyObject *args)
@@ -626,24 +666,21 @@ static PyObject *py_ldb_rename(PyLdbObject *self, PyObject *args)
        PyObject *py_dn1, *py_dn2;
        struct ldb_dn *dn1, *dn2;
        int ret;
+       struct ldb_context *ldb;
        if (!PyArg_ParseTuple(args, "OO", &py_dn1, &py_dn2))
                return NULL;
 
-       if (!PyLdbDn_Check(py_dn1) || !PyLdbDn_Check(py_dn2)) {
-               PyErr_SetString(PyExc_TypeError, "Expected Ldb Dn");
-               return NULL;
-       }
-
-       if (!PyObject_AsDn(NULL, py_dn1, PyLdb_AsLdbContext(self), &dn1))
+       ldb = PyLdb_AsLdbContext(self);
+       if (!PyObject_AsDn(NULL, py_dn1, ldb, &dn1))
                return NULL;
 
-       if (!PyObject_AsDn(NULL, py_dn2, PyLdb_AsLdbContext(self), &dn2))
+       if (!PyObject_AsDn(NULL, py_dn2, ldb, &dn2))
                return NULL;
 
-       ret = ldb_rename(PyLdb_AsLdbContext(self), dn1, dn2);
-       PyErr_LDB_ERROR_IS_ERR_RAISE(ret, PyLdb_AsLdbContext(self));
+       ret = ldb_rename(ldb, dn1, dn2);
+       PyErr_LDB_ERROR_IS_ERR_RAISE(ret, ldb);
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_schema_attribute_remove(PyLdbObject *self, PyObject *args)
@@ -654,7 +691,7 @@ static PyObject *py_ldb_schema_attribute_remove(PyLdbObject *self, PyObject *arg
 
        ldb_schema_attribute_remove(PyLdb_AsLdbContext(self), name);
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_schema_attribute_add(PyLdbObject *self, PyObject *args)
@@ -669,13 +706,13 @@ static PyObject *py_ldb_schema_attribute_add(PyLdbObject *self, PyObject *args)
 
        PyErr_LDB_ERROR_IS_ERR_RAISE(ret, PyLdb_AsLdbContext(self));
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *ldb_ldif_to_pyobject(struct ldb_ldif *ldif)
 {
        if (ldif == NULL) {
-               return Py_None;
+               Py_RETURN_NONE;
        } else {
        /* We don't want this attached to the 'ldb' any more */
                talloc_steal(NULL, ldif);
@@ -695,7 +732,7 @@ static PyObject *py_ldb_parse_ldif(PyLdbObject *self, PyObject *args)
                return NULL;
 
        list = PyList_New(0);
-       while ((ldif = ldb_ldif_read_string(self->ptr, &s)) != NULL) {
+       while ((ldif = ldb_ldif_read_string(self->ldb_ctx, &s)) != NULL) {
                PyList_Append(list, ldb_ldif_to_pyobject(ldif));
        }
        return PyObject_GetIter(list);
@@ -722,12 +759,12 @@ static PyObject *py_ldb_schema_format_value(PyLdbObject *self, PyObject *args)
        a = ldb_schema_attribute_by_name(PyLdb_AsLdbContext(self), element_name);
 
        if (a == NULL) {
-               return Py_None;
+               Py_RETURN_NONE;
        }
        
        if (a->syntax->ldif_write_fn(PyLdb_AsLdbContext(self), mem_ctx, &old_val, &new_val) != 0) {
                talloc_free(mem_ctx);
-               return Py_None;
+               Py_RETURN_NONE;
        }
 
        ret = PyString_FromStringAndSize((const char *)new_val.data, new_val.length);
@@ -753,7 +790,7 @@ static PyObject *py_ldb_search(PyLdbObject *self, PyObject *args, PyObject *kwar
        struct ldb_control **parsed_controls;
        struct ldb_dn *base;
 
-       if (!PyArg_ParseTupleAndKeywords(args, kwargs, "|OisOO", (char **)kwnames,
+       if (!PyArg_ParseTupleAndKeywords(args, kwargs, "|OizOO", (char **)kwnames,
                                                                         &py_base, &scope, &expr, &py_attrs, &py_controls))
                return NULL;
 
@@ -812,6 +849,12 @@ static PyObject *py_ldb_search(PyLdbObject *self, PyObject *args, PyObject *kwar
 
        talloc_free(req);
 
+       if (ret != LDB_SUCCESS) {
+               talloc_free(res);
+               PyErr_LDB_ERROR_IS_ERR_RAISE(ret, ldb_ctx);
+               return NULL;
+       }
+
        return PyLdbResult_FromResult(res);
 }
 
@@ -825,9 +868,12 @@ static PyObject *py_ldb_get_opaque(PyLdbObject *self, PyObject *args)
 
        data = ldb_get_opaque(PyLdb_AsLdbContext(self), name);
 
+       if (data == NULL)
+               Py_RETURN_NONE;
+
        /* FIXME: More interpretation */
 
-       return Py_None; 
+       return Py_True;
 }
 
 static PyObject *py_ldb_set_opaque(PyLdbObject *self, PyObject *args)
@@ -842,7 +888,20 @@ static PyObject *py_ldb_set_opaque(PyLdbObject *self, PyObject *args)
 
        ldb_set_opaque(PyLdb_AsLdbContext(self), name, data);
 
-       return Py_None;
+       Py_RETURN_NONE;
+}
+
+static PyObject *py_ldb_modules(PyLdbObject *self)
+{
+       struct ldb_context *ldb = PyLdb_AsLdbContext(self);
+       PyObject *ret = PyList_New(0);
+       struct ldb_module *mod;
+
+       for (mod = ldb->modules; mod; mod = mod->next) {
+               PyList_Append(ret, PyLdbModule_FromModule(mod));
+       }
+
+       return ret;
 }
 
 static PyMethodDef py_ldb_methods[] = {
@@ -910,21 +969,33 @@ static PyMethodDef py_ldb_methods[] = {
        { "parse_ldif", (PyCFunction)py_ldb_parse_ldif, METH_VARARGS,
                "S.parse_ldif(ldif) -> iter(messages)\n"
                "Parse a string formatted using LDIF." },
-       { "get_opaque", (PyCFunction)py_ldb_set_opaque, METH_VARARGS,
+       { "get_opaque", (PyCFunction)py_ldb_get_opaque, METH_VARARGS,
                "S.get_opaque(name) -> value\n"
                "Get an opaque value set on this LDB connection. \n"
                ":note: The returned value may not be useful in Python."
        },
-       { "set_opaque", (PyCFunction)py_ldb_get_opaque, METH_VARARGS,
+       { "set_opaque", (PyCFunction)py_ldb_set_opaque, METH_VARARGS,
                "S.set_opaque(name, value) -> None\n"
                "Set an opaque value on this LDB connection. \n"
                ":note: Passing incorrect values may cause crashes." },
+       { "modules", (PyCFunction)py_ldb_modules, METH_NOARGS,
+               "S.modules() -> list\n"
+               "Return the list of modules on this LDB connection " },
        { NULL },
 };
 
 PyObject *PyLdbModule_FromModule(struct ldb_module *mod)
 {
-       return py_talloc_import(&PyLdbModule, mod);
+       PyLdbModuleObject *ret;
+
+       ret = (PyLdbModuleObject *)PyLdbModule.tp_alloc(&PyLdbModule, 0);
+       if (ret == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+       ret->mem_ctx = talloc_new(NULL);
+       ret->mod = talloc_reference(ret->mem_ctx, mod);
+       return (PyObject *)ret;
 }
 
 static PyObject *py_ldb_get_firstmodule(PyLdbObject *self, void *closure)
@@ -967,17 +1038,31 @@ static PySequenceMethods py_ldb_seq = {
 
 PyObject *PyLdb_FromLdbContext(struct ldb_context *ldb_ctx)
 {
-       return py_talloc_import(&PyLdb, ldb_ctx);
+       PyLdbObject *ret;
+
+       ret = (PyLdbObject *)PyLdb.tp_alloc(&PyLdb, 0);
+       if (ret == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+       ret->mem_ctx = talloc_new(NULL);
+       ret->ldb_ctx = talloc_reference(ret->mem_ctx, ldb_ctx);
+       return (PyObject *)ret;
+}
+
+static void py_ldb_dealloc(PyLdbObject *self)
+{
+       talloc_free(self->mem_ctx);
+       self->ob_type->tp_free(self);
 }
 
 PyTypeObject PyLdb = {
-       PyObject_HEAD_INIT(NULL)
        .tp_name = "Ldb",
        .tp_methods = py_ldb_methods,
        .tp_repr = (reprfunc)py_ldb_repr,
        .tp_new = py_ldb_new,
        .tp_init = (initproc)py_ldb_init,
-       .tp_dealloc = py_talloc_dealloc,
+       .tp_dealloc = (destructor)py_ldb_dealloc,
        .tp_getset = py_ldb_getset,
        .tp_getattro = PyObject_GenericGetAttr,
        .tp_basicsize = sizeof(PyLdbObject),
@@ -999,19 +1084,19 @@ static PyObject *py_ldb_module_str(PyLdbModuleObject *self)
 static PyObject *py_ldb_module_start_transaction(PyLdbModuleObject *self)
 {
        PyLdbModule_AsModule(self)->ops->start_transaction(PyLdbModule_AsModule(self));
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_module_end_transaction(PyLdbModuleObject *self)
 {
        PyLdbModule_AsModule(self)->ops->end_transaction(PyLdbModule_AsModule(self));
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_module_del_transaction(PyLdbModuleObject *self)
 {
        PyLdbModule_AsModule(self)->ops->del_transaction(PyLdbModule_AsModule(self));
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_module_search(PyLdbModuleObject *self, PyObject *args, PyObject *kwargs)
@@ -1025,26 +1110,17 @@ static PyObject *py_ldb_module_search(PyLdbModuleObject *self, PyObject *args, P
                                                                         &py_base, &scope, &py_tree, &py_attrs))
                return NULL;
 
-       req = talloc_zero(NULL, struct ldb_request);
+       mod = self->mod;
 
-       req->operation = LDB_SEARCH;
-       req->op.search.base = PyLdbDn_AsDn(py_base);
-       req->op.search.scope = scope;
-       req->op.search.tree = PyLdbTree_AsTree(py_tree);
-       if (py_attrs == Py_None) {
-               req->op.search.attrs = NULL;
-       } else {
-               req->op.search.attrs = PyList_AsStringList(req, py_attrs);
-       }
-
-       req->op.search.res = talloc_zero(NULL, struct ldb_result);
-
-       mod = self->ptr;
+       ret = ldb_build_search_req(&req, mod->ldb, NULL, PyLdbDn_AsDn(py_base), 
+                            scope, NULL /* expr */, py_attrs == Py_None?NULL:PyList_AsStringList(req, py_attrs),
+                            NULL /* controls */, NULL, NULL, NULL);
+       PyErr_LDB_ERROR_IS_ERR_RAISE(ret, mod->ldb);
 
        ret = mod->ops->search(mod, req);
        talloc_free(req);
 
-       PyErr_LDB_ERROR_IS_ERR_RAISE(ret, NULL);
+       PyErr_LDB_ERROR_IS_ERR_RAISE(ret, mod->ldb);
 
        return PyLdbResult_FromResult(req->op.search.res);
 }
@@ -1055,6 +1131,7 @@ static PyObject *py_ldb_module_add(PyLdbModuleObject *self, PyObject *args)
        struct ldb_request *req;
        PyObject *py_message;
        int ret;
+       struct ldb_module *mod;
 
        if (!PyArg_ParseTuple(args, "O", &py_message))
                return NULL;
@@ -1063,11 +1140,12 @@ static PyObject *py_ldb_module_add(PyLdbModuleObject *self, PyObject *args)
        req->operation = LDB_ADD;
        req->op.add.message = PyLdbMessage_AsMessage(py_message);
 
-       ret = PyLdbModule_AsModule(self)->ops->add(PyLdbModule_AsModule(self), req);
+       mod = PyLdbModule_AsModule(self);
+       ret = mod->ops->add(mod, req);
 
-       PyErr_LDB_ERROR_IS_ERR_RAISE(ret, NULL);
+       PyErr_LDB_ERROR_IS_ERR_RAISE(ret, mod->ldb);
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_module_modify(PyLdbModuleObject *self, PyObject *args) 
@@ -1075,6 +1153,7 @@ static PyObject *py_ldb_module_modify(PyLdbModuleObject *self, PyObject *args)
        int ret;
        struct ldb_request *req;
        PyObject *py_message;
+       struct ldb_module *mod;
 
        if (!PyArg_ParseTuple(args, "O", &py_message))
                return NULL;
@@ -1083,11 +1162,12 @@ static PyObject *py_ldb_module_modify(PyLdbModuleObject *self, PyObject *args)
        req->operation = LDB_MODIFY;
        req->op.mod.message = PyLdbMessage_AsMessage(py_message);
        
-       ret = PyLdbModule_AsModule(self)->ops->modify(PyLdbModule_AsModule(self), req);
+       mod = PyLdbModule_AsModule(self);
+       ret = mod->ops->modify(mod, req);
 
-       PyErr_LDB_ERROR_IS_ERR_RAISE(ret, NULL);
+       PyErr_LDB_ERROR_IS_ERR_RAISE(ret, mod->ldb);
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_module_delete(PyLdbModuleObject *self, PyObject *args) 
@@ -1107,7 +1187,7 @@ static PyObject *py_ldb_module_delete(PyLdbModuleObject *self, PyObject *args)
 
        PyErr_LDB_ERROR_IS_ERR_RAISE(ret, NULL);
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_module_rename(PyLdbModuleObject *self, PyObject *args)
@@ -1129,7 +1209,7 @@ static PyObject *py_ldb_module_rename(PyLdbModuleObject *self, PyObject *args)
 
        PyErr_LDB_ERROR_IS_ERR_RAISE(ret, NULL);
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyMethodDef py_ldb_module_methods[] = {
@@ -1144,13 +1224,20 @@ static PyMethodDef py_ldb_module_methods[] = {
        { NULL },
 };
 
+static void py_ldb_module_dealloc(PyLdbModuleObject *self)
+{
+       talloc_free(self->mem_ctx);
+       self->ob_type->tp_free(self);
+}
+
 PyTypeObject PyLdbModule = {
        .tp_name = "LdbModule",
        .tp_methods = py_ldb_module_methods,
        .tp_repr = (reprfunc)py_ldb_module_repr,
        .tp_str = (reprfunc)py_ldb_module_str,
-       .tp_basicsize = sizeof(py_talloc_Object),
-       .tp_dealloc = py_talloc_dealloc,
+       .tp_basicsize = sizeof(PyLdbModuleObject),
+       .tp_dealloc = (destructor)py_ldb_module_dealloc,
+       .tp_flags = Py_TPFLAGS_DEFAULT,
 };
 
 struct ldb_message_element *PyObject_AsMessageElement(TALLOC_CTX *mem_ctx,
@@ -1170,14 +1257,16 @@ struct ldb_message_element *PyObject_AsMessageElement(TALLOC_CTX *mem_ctx,
                me->num_values = 1;
                me->values = talloc_array(me, struct ldb_val, me->num_values);
                me->values[0].length = PyString_Size(set_obj);
-               me->values[0].data = (uint8_t *)talloc_strdup(me->values, 
-                                                                                  PyString_AsString(set_obj));
+               me->values[0].data = (uint8_t *)talloc_strndup(me->values,
+                                       PyString_AsString(set_obj),
+                                       me->values[0].length);
        } else if (PySequence_Check(set_obj)) {
                int i;
                me->num_values = PySequence_Size(set_obj);
                me->values = talloc_array(me, struct ldb_val, me->num_values);
                for (i = 0; i < me->num_values; i++) {
                        PyObject *obj = PySequence_GetItem(set_obj, i);
+
                        me->values[i].length = PyString_Size(obj);
                        me->values[i].data = (uint8_t *)PyString_AsString(obj);
                }
@@ -1190,7 +1279,7 @@ struct ldb_message_element *PyObject_AsMessageElement(TALLOC_CTX *mem_ctx,
 }
 
 
-PyObject *ldb_msg_element_to_set(struct ldb_context *ldb_ctx, 
+static PyObject *ldb_msg_element_to_set(struct ldb_context *ldb_ctx, 
                                                                 struct ldb_message_element *me)
 {
        int i;
@@ -1207,13 +1296,13 @@ PyObject *ldb_msg_element_to_set(struct ldb_context *ldb_ctx,
        return result;
 }
 
-PyObject *py_ldb_msg_element_get(PyLdbMessageElementObject *self, PyObject *args)
+static PyObject *py_ldb_msg_element_get(PyLdbMessageElementObject *self, PyObject *args)
 {
        int i;
        if (!PyArg_ParseTuple(args, "i", &i))
                return NULL;
        if (i < 0 || i >= PyLdbMessageElement_AsMessageElement(self)->num_values)
-               return Py_None;
+               Py_RETURN_NONE;
 
        return PyObject_FromLdbValue(NULL, PyLdbMessageElement_AsMessageElement(self), 
                                                                 &(PyLdbMessageElement_AsMessageElement(self)->values[i]));
@@ -1255,9 +1344,21 @@ static PyObject *py_ldb_msg_element_iter(PyLdbMessageElementObject *self)
        return PyObject_GetIter(ldb_msg_element_to_set(NULL, PyLdbMessageElement_AsMessageElement(self)));
 }
 
-PyObject *PyLdbMessageElement_FromMessageElement(struct ldb_message_element *el)
+PyObject *PyLdbMessageElement_FromMessageElement(struct ldb_message_element *el, TALLOC_CTX *mem_ctx)
 {
-       return py_talloc_import(&PyLdbMessageElement, el);
+       PyLdbMessageElementObject *ret;
+       ret = (PyLdbMessageElementObject *)PyLdbMessageElement.tp_alloc(&PyLdbMessageElement, 0);
+       if (ret == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+       ret->mem_ctx = talloc_new(NULL);
+       if (talloc_reference(ret->mem_ctx, mem_ctx) == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+       ret->el = el;
+       return (PyObject *)ret;
 }
 
 static PyObject *py_ldb_msg_element_new(PyTypeObject *type, PyObject *args, PyObject *kwargs)
@@ -1267,6 +1368,7 @@ static PyObject *py_ldb_msg_element_new(PyTypeObject *type, PyObject *args, PyOb
        int flags = 0;
        char *name = NULL;
        const char *kwnames[] = { "elements", "flags", "name", NULL };
+       PyLdbMessageElementObject *ret;
        if (!PyArg_ParseTupleAndKeywords(args, kwargs, "|Ois", (char **)kwnames, &py_elements, &flags, &name))
                return NULL;
 
@@ -1274,12 +1376,12 @@ static PyObject *py_ldb_msg_element_new(PyTypeObject *type, PyObject *args, PyOb
 
        if (py_elements != NULL) {
                int i;
-               if (!PySequence_Check(py_elements)) {
+               if (PyString_Check(py_elements)) {
                        el->num_values = 1;
                        el->values = talloc_array(el, struct ldb_val, 1);
                        el->values[0].data = (uint8_t *)PyString_AsString(py_elements);
                        el->values[0].length = PyString_Size(py_elements);
-               } else {
+               } else if (PySequence_Check(py_elements)) {
                        el->num_values = PySequence_Size(py_elements);
                        el->values = talloc_array(el, struct ldb_val, el->num_values);
                        for (i = 0; i < el->num_values; i++) {
@@ -1287,13 +1389,27 @@ static PyObject *py_ldb_msg_element_new(PyTypeObject *type, PyObject *args, PyOb
                                el->values[i].data = (uint8_t *)PyString_AsString(item);
                                el->values[i].length = PyString_Size(item);
                        }
+               } else {
+                       PyErr_SetString(PyExc_TypeError, 
+                                       "Expected string or list");
+                       talloc_free(el);
+                       return NULL;
                }
        }
 
        el->flags = flags;
        el->name = talloc_strdup(el, name);
 
-       return py_talloc_import(&PyLdbMessageElement, el);
+       ret = (PyLdbMessageElementObject *)PyLdbMessageElement.tp_alloc(&PyLdbMessageElement, 0);
+       if (ret == NULL) {
+               PyErr_NoMemory();
+               talloc_free(el);
+               return NULL;
+       }
+
+       ret->mem_ctx = talloc_new(NULL);
+       ret->el = talloc_reference(ret->mem_ctx, el);
+       return (PyObject *)ret;
 }
 
 static PyObject *py_ldb_msg_element_repr(PyLdbMessageElementObject *self)
@@ -1318,16 +1434,34 @@ static PyObject *py_ldb_msg_element_repr(PyLdbMessageElementObject *self)
        return ret;
 }
 
+static PyObject *py_ldb_msg_element_str(PyLdbMessageElementObject *self)
+{
+       struct ldb_message_element *el = PyLdbMessageElement_AsMessageElement(self);
+
+       if (el->num_values == 1)
+               return PyString_FromStringAndSize((char *)el->values[0].data, el->values[0].length);
+       else 
+               Py_RETURN_NONE;
+}
+
+static void py_ldb_msg_element_dealloc(PyLdbMessageElementObject *self)
+{
+       talloc_free(self->mem_ctx);
+       self->ob_type->tp_free(self);
+}
+
 PyTypeObject PyLdbMessageElement = {
        .tp_name = "MessageElement",
        .tp_basicsize = sizeof(PyLdbMessageElementObject),
-       .tp_dealloc = py_talloc_dealloc,
+       .tp_dealloc = (destructor)py_ldb_msg_element_dealloc,
        .tp_repr = (reprfunc)py_ldb_msg_element_repr,
+       .tp_str = (reprfunc)py_ldb_msg_element_str,
        .tp_methods = py_ldb_msg_element_methods,
        .tp_compare = (cmpfunc)py_ldb_msg_element_cmp,
        .tp_iter = (getiterfunc)py_ldb_msg_element_iter,
        .tp_as_sequence = &py_ldb_msg_element_seq,
        .tp_new = py_ldb_msg_element_new,
+       .tp_flags = Py_TPFLAGS_DEFAULT,
 };
 
 static PyObject *py_ldb_msg_remove_attr(PyLdbMessageObject *self, PyObject *args)
@@ -1336,9 +1470,9 @@ static PyObject *py_ldb_msg_remove_attr(PyLdbMessageObject *self, PyObject *args
        if (!PyArg_ParseTuple(args, "s", &name))
                return NULL;
 
-       ldb_msg_remove_attr(self->ptr, name);
+       ldb_msg_remove_attr(self->msg, name);
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_ldb_msg_keys(PyLdbMessageObject *self)
@@ -1361,13 +1495,14 @@ static PyObject *py_ldb_msg_getitem_helper(PyLdbMessageObject *self, PyObject *p
 {
        struct ldb_message_element *el;
        char *name = PyString_AsString(py_name);
+       struct ldb_message *msg = PyLdbMessage_AsMessage(self);
        if (!strcmp(name, "dn"))
-               return PyLdbDn_FromDn(PyLdbMessage_AsMessage(self)->dn);
-       el = ldb_msg_find_element(PyLdbMessage_AsMessage(self), name);
+               return PyLdbDn_FromDn(msg->dn);
+       el = ldb_msg_find_element(msg, name);
        if (el == NULL) {
                return NULL;
        }
-       return (PyObject *)PyLdbMessageElement_FromMessageElement(el);
+       return (PyObject *)PyLdbMessageElement_FromMessageElement(el, msg);
 }
 
 static PyObject *py_ldb_msg_getitem(PyLdbMessageObject *self, PyObject *py_name)
@@ -1388,7 +1523,7 @@ static PyObject *py_ldb_msg_get(PyLdbMessageObject *self, PyObject *args)
 
        ret = py_ldb_msg_getitem_helper(self, name);
        if (ret == NULL)
-               return Py_None;
+               Py_RETURN_NONE;
        return ret;
 }
 
@@ -1403,7 +1538,7 @@ static PyObject *py_ldb_msg_items(PyLdbMessageObject *self)
                j++;
        }
        for (i = 0; i < msg->num_elements; i++, j++) {
-               PyList_SetItem(l, j, Py_BuildValue("(sO)", msg->elements[i].name, PyLdbMessageElement_FromMessageElement(&msg->elements[i])));
+               PyList_SetItem(l, j, Py_BuildValue("(sO)", msg->elements[i].name, PyLdbMessageElement_FromMessageElement(&msg->elements[i], self->msg)));
        }
        return l;
 }
@@ -1416,17 +1551,27 @@ static PyMethodDef py_ldb_msg_methods[] = {
        { NULL },
 };
 
+static PyObject *py_ldb_msg_iter(PyLdbMessageObject *self)
+{
+       PyObject *list, *iter;
+
+       list = py_ldb_msg_keys(self);
+       iter = PyObject_GetIter(list);
+       Py_DECREF(list);
+       return iter;
+}
+
 static int py_ldb_msg_setitem(PyLdbMessageObject *self, PyObject *name, PyObject *value)
 {
        char *attr_name = PyString_AsString(name);
        if (value == NULL) {
-               ldb_msg_remove_attr(self->ptr, attr_name);
+               ldb_msg_remove_attr(self->msg, attr_name);
        } else {
                struct ldb_message_element *el = PyObject_AsMessageElement(NULL,
                                                                                        value, 0, attr_name);
                if (el == NULL)
                        return -1;
-               talloc_steal(self->ptr, el);
+               talloc_steal(self->msg, el);
                ldb_msg_remove_attr(PyLdbMessage_AsMessage(self), attr_name);
                ldb_msg_add(PyLdbMessage_AsMessage(self), el, el->flags);
        }
@@ -1449,6 +1594,7 @@ static PyObject *py_ldb_msg_new(PyTypeObject *type, PyObject *args, PyObject *kw
        const char *kwnames[] = { "dn", NULL };
        struct ldb_message *ret;
        PyObject *pydn = NULL;
+       PyLdbMessageObject *py_ret;
        if (!PyArg_ParseTupleAndKeywords(args, kwargs, "|O", (char **)kwnames, &pydn))
                return NULL;
 
@@ -1462,12 +1608,29 @@ static PyObject *py_ldb_msg_new(PyTypeObject *type, PyObject *args, PyObject *kw
                if (!PyObject_AsDn(NULL, pydn, NULL, &ret->dn))
                        return NULL;
 
-       return py_talloc_import(&PyLdbMessage, ret); 
+       py_ret = (PyLdbMessageObject *)type->tp_alloc(type, 0);
+       if (py_ret == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+
+       py_ret->mem_ctx = talloc_new(NULL);
+       py_ret->msg = talloc_reference(py_ret->mem_ctx, ret);
+       return (PyObject *)py_ret;
 }
 
 PyObject *PyLdbMessage_FromMessage(struct ldb_message *msg)
 {
-       return py_talloc_import(&PyLdbMessage, msg);
+       PyLdbMessageObject *ret;
+
+       ret = (PyLdbMessageObject *)PyLdbMessage.tp_alloc(&PyLdbMessage, 0);
+       if (ret == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+       ret->mem_ctx = talloc_new(NULL);
+       ret->msg = talloc_reference(ret->mem_ctx, msg);
+       return (PyObject *)ret;
 }
 
 static PyObject *py_ldb_msg_get_dn(PyLdbMessageObject *self, void *closure)
@@ -1496,30 +1659,55 @@ static PyObject *py_ldb_msg_repr(PyLdbMessageObject *self)
        return ret;
 }
 
+static void py_ldb_msg_dealloc(PyLdbMessageObject *self)
+{
+       talloc_free(self->mem_ctx);
+       self->ob_type->tp_free(self);
+}
+
 PyTypeObject PyLdbMessage = {
        .tp_name = "Message",
        .tp_methods = py_ldb_msg_methods,
        .tp_getset = py_ldb_msg_getset,
        .tp_as_mapping = &py_ldb_msg_mapping,
        .tp_basicsize = sizeof(PyLdbMessageObject),
-       .tp_dealloc = py_talloc_dealloc,
+       .tp_dealloc = (destructor)py_ldb_msg_dealloc,
        .tp_new = py_ldb_msg_new,
        .tp_repr = (reprfunc)py_ldb_msg_repr,
+       .tp_flags = Py_TPFLAGS_DEFAULT,
+       .tp_iter = (getiterfunc)py_ldb_msg_iter,
 };
 
 PyObject *PyLdbTree_FromTree(struct ldb_parse_tree *tree)
 {
-       return py_talloc_import(&PyLdbTree, tree);
+       PyLdbTreeObject *ret;
+
+       ret = (PyLdbTreeObject *)PyLdbTree.tp_alloc(&PyLdbTree, 0);
+       if (ret == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+       
+       ret->mem_ctx = talloc_new(NULL);
+       ret->tree = talloc_reference(ret->mem_ctx, tree);
+       return (PyObject *)ret;
+}
+
+static void py_ldb_tree_dealloc(PyLdbTreeObject *self)
+{
+       talloc_free(self->mem_ctx);
+       self->ob_type->tp_free(self);
 }
 
 PyTypeObject PyLdbTree = {
        .tp_name = "Tree",
        .tp_basicsize = sizeof(PyLdbTreeObject),
-       .tp_dealloc = py_talloc_dealloc,
+       .tp_dealloc = (destructor)py_ldb_tree_dealloc,
+       .tp_flags = Py_TPFLAGS_DEFAULT,
 };
 
 /* Ldb_module */
-int py_module_search(struct ldb_module *mod, struct ldb_request *req)
+static int py_module_search(struct ldb_module *mod, struct ldb_request *req)
 {
        PyObject *py_ldb = mod->private_data;
        PyObject *py_result, *py_base, *py_attrs, *py_tree;
@@ -1551,12 +1739,12 @@ int py_module_search(struct ldb_module *mod, struct ldb_request *req)
        Py_DECREF(py_base);
 
        if (py_result == NULL) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return LDB_ERR_PYTHON_EXCEPTION;
        }
 
        req->op.search.res = PyLdbResult_AsResult(NULL, py_result);
        if (req->op.search.res == NULL) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return LDB_ERR_PYTHON_EXCEPTION;
        }
 
        Py_DECREF(py_result);
@@ -1564,7 +1752,7 @@ int py_module_search(struct ldb_module *mod, struct ldb_request *req)
        return LDB_SUCCESS;
 }
 
-int py_module_add(struct ldb_module *mod, struct ldb_request *req)
+static int py_module_add(struct ldb_module *mod, struct ldb_request *req)
 {
        PyObject *py_ldb = mod->private_data;
        PyObject *py_result, *py_msg;
@@ -1580,7 +1768,7 @@ int py_module_add(struct ldb_module *mod, struct ldb_request *req)
        Py_DECREF(py_msg);
 
        if (py_result == NULL) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return LDB_ERR_PYTHON_EXCEPTION;
        }
 
        Py_DECREF(py_result);
@@ -1588,7 +1776,7 @@ int py_module_add(struct ldb_module *mod, struct ldb_request *req)
        return LDB_SUCCESS;
 }
 
-int py_module_modify(struct ldb_module *mod, struct ldb_request *req)
+static int py_module_modify(struct ldb_module *mod, struct ldb_request *req)
 {
        PyObject *py_ldb = mod->private_data;
        PyObject *py_result, *py_msg;
@@ -1604,7 +1792,7 @@ int py_module_modify(struct ldb_module *mod, struct ldb_request *req)
        Py_DECREF(py_msg);
 
        if (py_result == NULL) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return LDB_ERR_PYTHON_EXCEPTION;
        }
 
        Py_DECREF(py_result);
@@ -1612,7 +1800,7 @@ int py_module_modify(struct ldb_module *mod, struct ldb_request *req)
        return LDB_SUCCESS;
 }
 
-int py_module_del(struct ldb_module *mod, struct ldb_request *req)
+static int py_module_del(struct ldb_module *mod, struct ldb_request *req)
 {
        PyObject *py_ldb = mod->private_data;
        PyObject *py_result, *py_dn;
@@ -1625,7 +1813,7 @@ int py_module_del(struct ldb_module *mod, struct ldb_request *req)
        py_result = PyObject_CallMethod(py_ldb, (char *)"delete", (char *)"O", py_dn);
 
        if (py_result == NULL) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return LDB_ERR_PYTHON_EXCEPTION;
        }
 
        Py_DECREF(py_result);
@@ -1633,7 +1821,7 @@ int py_module_del(struct ldb_module *mod, struct ldb_request *req)
        return LDB_SUCCESS;
 }
 
-int py_module_rename(struct ldb_module *mod, struct ldb_request *req)
+static int py_module_rename(struct ldb_module *mod, struct ldb_request *req)
 {
        PyObject *py_ldb = mod->private_data;
        PyObject *py_result, *py_olddn, *py_newdn;
@@ -1654,7 +1842,7 @@ int py_module_rename(struct ldb_module *mod, struct ldb_request *req)
        Py_DECREF(py_newdn);
 
        if (py_result == NULL) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return LDB_ERR_PYTHON_EXCEPTION;
        }
 
        Py_DECREF(py_result);
@@ -1662,7 +1850,7 @@ int py_module_rename(struct ldb_module *mod, struct ldb_request *req)
        return LDB_SUCCESS;
 }
 
-int py_module_request(struct ldb_module *mod, struct ldb_request *req)
+static int py_module_request(struct ldb_module *mod, struct ldb_request *req)
 {
        PyObject *py_ldb = mod->private_data;
        PyObject *py_result;
@@ -1672,7 +1860,7 @@ int py_module_request(struct ldb_module *mod, struct ldb_request *req)
        return LDB_ERR_OPERATIONS_ERROR;
 }
 
-int py_module_extended(struct ldb_module *mod, struct ldb_request *req)
+static int py_module_extended(struct ldb_module *mod, struct ldb_request *req)
 {
        PyObject *py_ldb = mod->private_data;
        PyObject *py_result;
@@ -1682,7 +1870,7 @@ int py_module_extended(struct ldb_module *mod, struct ldb_request *req)
        return LDB_ERR_OPERATIONS_ERROR;
 }
 
-int py_module_start_transaction(struct ldb_module *mod)
+static int py_module_start_transaction(struct ldb_module *mod)
 {
        PyObject *py_ldb = mod->private_data;
        PyObject *py_result;
@@ -1690,7 +1878,7 @@ int py_module_start_transaction(struct ldb_module *mod)
        py_result = PyObject_CallMethod(py_ldb, (char *)"start_transaction", (char *)"");
 
        if (py_result == NULL) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return LDB_ERR_PYTHON_EXCEPTION;
        }
 
        Py_DECREF(py_result);
@@ -1698,7 +1886,7 @@ int py_module_start_transaction(struct ldb_module *mod)
        return LDB_SUCCESS;
 }
 
-int py_module_end_transaction(struct ldb_module *mod)
+static int py_module_end_transaction(struct ldb_module *mod)
 {
        PyObject *py_ldb = mod->private_data;
        PyObject *py_result;
@@ -1706,7 +1894,7 @@ int py_module_end_transaction(struct ldb_module *mod)
        py_result = PyObject_CallMethod(py_ldb, (char *)"end_transaction", (char *)"");
 
        if (py_result == NULL) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return LDB_ERR_PYTHON_EXCEPTION;
        }
 
        Py_DECREF(py_result);
@@ -1714,7 +1902,7 @@ int py_module_end_transaction(struct ldb_module *mod)
        return LDB_SUCCESS;
 }
 
-int py_module_del_transaction(struct ldb_module *mod)
+static int py_module_del_transaction(struct ldb_module *mod)
 {
        PyObject *py_ldb = mod->private_data;
        PyObject *py_result;
@@ -1722,7 +1910,7 @@ int py_module_del_transaction(struct ldb_module *mod)
        py_result = PyObject_CallMethod(py_ldb, (char *)"del_transaction", (char *)"");
 
        if (py_result == NULL) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return LDB_ERR_PYTHON_EXCEPTION;
        }
 
        Py_DECREF(py_result);
@@ -1736,7 +1924,7 @@ static int py_module_destructor(struct ldb_module *mod)
        return 0;
 }
 
-int py_module_init (struct ldb_module *mod)
+static int py_module_init (struct ldb_module *mod)
 {
        PyObject *py_class = mod->ops->private_data;
        PyObject *py_result, *py_next, *py_ldb;
@@ -1754,7 +1942,7 @@ int py_module_init (struct ldb_module *mod)
        py_result = PyObject_CallFunction(py_class, (char *)"OO", py_ldb, py_next);
 
        if (py_result == NULL) {
-               return LDB_ERR_OPERATIONS_ERROR;
+               return LDB_ERR_PYTHON_EXCEPTION;
        }
 
        mod->private_data = py_result;
@@ -1799,7 +1987,7 @@ static PyObject *py_register_module(PyObject *module, PyObject *args)
 
        PyErr_LDB_ERROR_IS_ERR_RAISE(ret, NULL);
 
-       return Py_None;
+       Py_RETURN_NONE;
 }
 
 static PyObject *py_timestring(PyObject *module, PyObject *args)
@@ -1922,7 +2110,7 @@ void initldb(void)
        PyModule_AddObject(m, "ERR_OBJECT_CLASS_VIOLATION", PyInt_FromLong(LDB_ERR_OBJECT_CLASS_VIOLATION));
        PyModule_AddObject(m, "ERR_NOT_ALLOWED_ON_NON_LEAF", PyInt_FromLong(LDB_ERR_NOT_ALLOWED_ON_NON_LEAF));
        PyModule_AddObject(m, "ERR_NOT_ALLOWED_ON_RDN", PyInt_FromLong(LDB_ERR_NOT_ALLOWED_ON_RDN));
-       PyModule_AddObject(m, "ERR_ENTYR_ALREADY_EXISTS", PyInt_FromLong(LDB_ERR_ENTRY_ALREADY_EXISTS));
+       PyModule_AddObject(m, "ERR_ENTRY_ALREADY_EXISTS", PyInt_FromLong(LDB_ERR_ENTRY_ALREADY_EXISTS));
        PyModule_AddObject(m, "ERR_OBJECT_CLASS_MODS_PROHIBITED", PyInt_FromLong(LDB_ERR_OBJECT_CLASS_MODS_PROHIBITED));
        PyModule_AddObject(m, "ERR_AFFECTS_MULTIPLE_DSAS", PyInt_FromLong(LDB_ERR_AFFECTS_MULTIPLE_DSAS));
 
@@ -1947,94 +2135,3 @@ void initldb(void)
        PyModule_AddObject(m, "Module", (PyObject *)&PyLdbModule);
        PyModule_AddObject(m, "Tree", (PyObject *)&PyLdbTree);
 }
-
-#if 0
-
-/* ldb_message_element */
-%rename(MessageElement) ldb_message_element;
-%feature("docstring") ldb_message_element "Message element.";
-typedef struct ldb_message_element {
-       %extend {
-                       PyObject *__set__(void)
-               {
-                       return ldb_msg_element_to_set(NULL, $self);
-               }
-
-               ldb_message_element(PyObject *set_obj, int flags=0, const char *name = NULL)
-               {
-                       return PyObject_AsMessageElement(NULL, set_obj, flags, name);
-               }
-
-       }
-       %pythoncode {
-               def __getitem__(self, i):
-                       ret = self.get(i)
-                       if ret is None:
-                               raise KeyError("no such value")
-                       return ret
-
-               def __repr__(self):
-                       return "MessageElement([%s])" % (",".join(repr(x) for x in self.__set__()))
-       }
-} ldb_message_element;
-
-/* ldb_message */
-
-typedef struct ldb_message {
-       ldb_dn *dn;
-
-       %extend {
-               ldb_msg(ldb_dn *dn = NULL) { 
-                       ret->dn = talloc_reference(ret, dn);
-                       return ret;
-               }
-               
-               void __setitem__(const char *attr_name, ldb_message_element *val)
-               {
-                       struct ldb_message_element *el;
-                       
-                       ldb_msg_remove_attr($self, attr_name);
-
-                       el = talloc($self, struct ldb_message_element);
-                       el->name = talloc_strdup(el, attr_name);
-                       el->num_values = val->num_values;
-                       el->values = talloc_reference(el, val->values);
-
-                       ldb_msg_add($self, el, val->flags);
-               }
-
-               void __setitem__(const char *attr_name, PyObject *val)
-               {
-               }
-%pythoncode {
-       def get(self, key, default=None):
-               if key == "dn":
-                       return self.dn
-               return self.find_element(key)
-
-       def iteritems(self):
-               for k in self.keys():
-                       yield k, self[k]
-       
-       def items(self):
-               return list(self.iteritems())
-
-       def __repr__(self):
-               return "Message(%s)" % repr(dict(self.iteritems()))
-}
-       }
-} ldb_msg;
-
-typedef struct ldb_context {
-       %pythoncode {
-               def itermodules(self):
-                       m = self.firstmodule
-                       while m is not None:
-                               yield m
-                               m = m.next
-
-               def modules(self):
-                       return list(self.itermodules())
-       }
-} ldb;
-#endif