s4/registry/py: use unsigned ParseTuple format for unsigned value
[sfrench/samba-autobuild/.git] / source4 / lib / registry / pyregistry.c
index 58894feb3a53cc3e9f85be408c7a0947b94a8d8e..78b47b8286d8e03fe69ed0489ce49a78b1ebd891 100644 (file)
@@ -2,6 +2,7 @@
    Unix SMB/CIFS implementation.
    Samba utility functions
    Copyright (C) Jelmer Vernooij <jelmer@samba.org> 2008
+   Copyright (C) Wilco Baan Hofman <wilco@baanhofman.nl> 2010
    
    This program is free software; you can redistribute it and/or modify
    it under the terms of the GNU General Public License as published by
 */
 
 #include <Python.h>
+#include "python/py3compat.h"
 #include "includes.h"
-#include <tevent.h>
 #include "libcli/util/pyerrors.h"
 #include "lib/registry/registry.h"
-#include "lib/talloc/pytalloc.h"
+#include <pytalloc.h>
+#include "lib/events/events.h"
 #include "auth/credentials/pycredentials.h"
 #include "param/pyparam.h"
 
-PyAPI_DATA(PyTypeObject) PyRegistryKey;
-PyAPI_DATA(PyTypeObject) PyRegistry;
-PyAPI_DATA(PyTypeObject) PyHiveKey;
+extern PyTypeObject PyRegistryKey;
+extern PyTypeObject PyRegistry;
+extern PyTypeObject PyHiveKey;
 
-/*#define PyRegistryKey_AsRegistryKey(obj) py_talloc_get_type(obj, struct registry_key)*/
-#define PyRegistry_AsRegistryContext(obj) ((struct registry_context *)py_talloc_get_ptr(obj))
-#define PyHiveKey_AsHiveKey(obj) ((struct hive_key*)py_talloc_get_ptr(obj))
+/*#define PyRegistryKey_AsRegistryKey(obj) pytalloc_get_type(obj, struct registry_key)*/
+#define PyRegistry_AsRegistryContext(obj) ((struct registry_context *)pytalloc_get_ptr(obj))
+#define PyHiveKey_AsHiveKey(obj) ((struct hive_key*)pytalloc_get_ptr(obj))
 
 
 static PyObject *py_get_predefined_key_by_name(PyObject *self, PyObject *args)
@@ -46,9 +48,9 @@ static PyObject *py_get_predefined_key_by_name(PyObject *self, PyObject *args)
                return NULL;
 
        result = reg_get_predefined_key_by_name(ctx, name, &key);
-       PyErr_WERROR_IS_ERR_RAISE(result);
+       PyErr_WERROR_NOT_OK_RAISE(result);
 
-       return py_talloc_steal(&PyRegistryKey, key);
+       return pytalloc_steal(&PyRegistryKey, key);
 }
 
 static PyObject *py_key_del_abs(PyObject *self, PyObject *args)
@@ -61,7 +63,7 @@ static PyObject *py_key_del_abs(PyObject *self, PyObject *args)
                return NULL;
 
        result = reg_key_del_abs(ctx, path);
-       PyErr_WERROR_IS_ERR_RAISE(result);
+       PyErr_WERROR_NOT_OK_RAISE(result);
 
        Py_RETURN_NONE;
 }
@@ -77,9 +79,9 @@ static PyObject *py_get_predefined_key(PyObject *self, PyObject *args)
                return NULL;
 
        result = reg_get_predefined_key(ctx, hkey, &key);
-       PyErr_WERROR_IS_ERR_RAISE(result);
+       PyErr_WERROR_NOT_OK_RAISE(result);
 
-       return py_talloc_steal(&PyRegistryKey, key);
+       return pytalloc_steal(&PyRegistryKey, key);
 }
 
 static PyObject *py_diff_apply(PyObject *self, PyObject *args)
@@ -91,7 +93,7 @@ static PyObject *py_diff_apply(PyObject *self, PyObject *args)
                return NULL;
 
        result = reg_diff_apply(ctx, filename);
-       PyErr_WERROR_IS_ERR_RAISE(result);
+       PyErr_WERROR_NOT_OK_RAISE(result);
 
        Py_RETURN_NONE; 
 }
@@ -118,13 +120,13 @@ static PyObject *py_mount_hive(PyObject *self, PyObject *args)
                int i;
                elements = talloc_array(NULL, const char *, PyList_Size(py_elements));
                for (i = 0; i < PyList_Size(py_elements); i++)
-                       elements[i] = PyString_AsString(PyList_GetItem(py_elements, i));
+                       elements[i] = PyStr_AsString(PyList_GetItem(py_elements, i));
        }
 
        SMB_ASSERT(ctx != NULL);
 
        result = reg_mount_hive(ctx, PyHiveKey_AsHiveKey(py_hivekey), hkey, elements);
-       PyErr_WERROR_IS_ERR_RAISE(result);
+       PyErr_WERROR_NOT_OK_RAISE(result);
 
        Py_RETURN_NONE;
 }
@@ -134,8 +136,8 @@ static PyObject *registry_new(PyTypeObject *type, PyObject *args, PyObject *kwar
        WERROR result;
        struct registry_context *ctx;
        result = reg_open_local(NULL, &ctx);
-       PyErr_WERROR_IS_ERR_RAISE(result);
-       return py_talloc_steal(&PyRegistry, ctx);
+       PyErr_WERROR_NOT_OK_RAISE(result);
+       return pytalloc_steal(&PyRegistry, ctx);
 }
 
 static PyMethodDef registry_methods[] = {
@@ -157,8 +159,6 @@ PyTypeObject PyRegistry = {
        .tp_name = "Registry",
        .tp_methods = registry_methods,
        .tp_new = registry_new,
-       .tp_basicsize = sizeof(py_talloc_Object),
-       .tp_dealloc = py_talloc_dealloc,
        .tp_flags = Py_TPFLAGS_DEFAULT,
 };
 
@@ -173,7 +173,7 @@ static PyObject *py_hive_key_del(PyObject *self, PyObject *args)
 
        result = hive_key_del(NULL, key, name);
 
-       PyErr_WERROR_IS_ERR_RAISE(result);
+       PyErr_WERROR_NOT_OK_RAISE(result);
 
        Py_RETURN_NONE; 
 }
@@ -184,7 +184,7 @@ static PyObject *py_hive_key_flush(PyObject *self)
        struct hive_key *key = PyHiveKey_AsHiveKey(self);
 
        result = hive_key_flush(key);
-       PyErr_WERROR_IS_ERR_RAISE(result);
+       PyErr_WERROR_NOT_OK_RAISE(result);
 
        Py_RETURN_NONE;
 }
@@ -200,7 +200,7 @@ static PyObject *py_hive_key_del_value(PyObject *self, PyObject *args)
 
        result = hive_key_del_value(NULL, key, name);
 
-       PyErr_WERROR_IS_ERR_RAISE(result);
+       PyErr_WERROR_NOT_OK_RAISE(result);
 
        Py_RETURN_NONE; 
 }
@@ -210,18 +210,21 @@ static PyObject *py_hive_key_set_value(PyObject *self, PyObject *args)
        char *name;
        uint32_t type;
        DATA_BLOB value;
+       Py_ssize_t value_length = 0;
        WERROR result;
        struct hive_key *key = PyHiveKey_AsHiveKey(self);
 
-       if (!PyArg_ParseTuple(args, "siz#", &name, &type, &value.data, &value.length))
+       if (!PyArg_ParseTuple(args, "sIz#", &name, &type, &value.data, &value_length)) {
                return NULL;
+       }
+       value.length = value_length;
 
        if (value.data != NULL)
                result = hive_key_set_value(key, name, type, value);
        else
                result = hive_key_del_value(NULL, key, name);
 
-       PyErr_WERROR_IS_ERR_RAISE(result);
+       PyErr_WERROR_NOT_OK_RAISE(result);
 
        Py_RETURN_NONE; 
 }
@@ -238,25 +241,70 @@ static PyMethodDef hive_key_methods[] = {
        { NULL }
 };
 
-static PyObject *hive_open(PyTypeObject *type, PyObject *args, PyObject *kwargs)
-{
-       /* reg_open_hive */
+static PyObject *hive_new(PyTypeObject *type, PyObject *args, PyObject *kwargs) {
        Py_RETURN_NONE;
 }
 
+static PyObject *py_open_hive(PyTypeObject *type, PyObject *args, PyObject *kwargs)
+{
+       const char *kwnames[] = { "location", "lp_ctx", "session_info", "credentials", NULL };
+       WERROR result;
+       struct loadparm_context *lp_ctx;
+       PyObject *py_lp_ctx = Py_None;
+       PyObject *py_session_info = Py_None;
+       PyObject *py_credentials = Py_None;
+       struct auth_session_info *session_info;
+       struct cli_credentials *credentials;
+       char *location;
+       struct hive_key *hive_key;
+       TALLOC_CTX *mem_ctx;
+
+       if (!PyArg_ParseTupleAndKeywords(args, kwargs, "s|OOO",
+                                        discard_const_p(char *, kwnames),
+                                        &location,
+                                        &py_lp_ctx, &py_session_info,
+                                        &py_credentials))
+               return NULL;
+
+       mem_ctx = talloc_new(NULL);
+       if (mem_ctx == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+
+       lp_ctx = lpcfg_from_py_object(mem_ctx, py_lp_ctx);
+       if (lp_ctx == NULL) {
+               PyErr_SetString(PyExc_TypeError, "Expected loadparm context");
+               talloc_free(mem_ctx);
+               return NULL;
+       }
+
+       credentials = cli_credentials_from_py_object(py_credentials);
+       if (credentials == NULL) {
+               PyErr_SetString(PyExc_TypeError, "Expected credentials");
+               talloc_free(mem_ctx);
+               return NULL;
+       }
+       session_info = NULL;
+
+       result = reg_open_hive(NULL, location, session_info, credentials,
+                              samba_tevent_context_init(NULL),
+                              lp_ctx, &hive_key);
+       talloc_free(mem_ctx);
+       PyErr_WERROR_NOT_OK_RAISE(result);
+
+       return pytalloc_steal(&PyHiveKey, hive_key);
+}
+
 PyTypeObject PyHiveKey = {
        .tp_name = "HiveKey",
        .tp_methods = hive_key_methods,
-       .tp_new = hive_open,
-       .tp_basicsize = sizeof(py_talloc_Object),
-       .tp_dealloc = py_talloc_dealloc,
+       .tp_new = hive_new,
        .tp_flags = Py_TPFLAGS_DEFAULT,
 };
 
 PyTypeObject PyRegistryKey = {
        .tp_name = "RegistryKey",
-       .tp_basicsize = sizeof(py_talloc_Object),
-       .tp_dealloc = py_talloc_dealloc,
        .tp_flags = Py_TPFLAGS_DEFAULT,
 };
 
@@ -266,24 +314,36 @@ static PyObject *py_open_samba(PyObject *self, PyObject *args, PyObject *kwargs)
        struct registry_context *reg_ctx;
        WERROR result;
        struct loadparm_context *lp_ctx;
-       PyObject *py_lp_ctx, *py_session_info, *py_credentials;
+       PyObject *py_lp_ctx = Py_None;
+       PyObject *py_session_info = Py_None;
+       PyObject *py_credentials = Py_None;
        struct auth_session_info *session_info;
        struct cli_credentials *credentials;
+       TALLOC_CTX *mem_ctx;
+
        if (!PyArg_ParseTupleAndKeywords(args, kwargs, "|OOO",
                                         discard_const_p(char *, kwnames),
                                         &py_lp_ctx, &py_session_info,
                                         &py_credentials))
                return NULL;
 
-       lp_ctx = lp_from_py_object(NULL, py_lp_ctx); /* FIXME: leaky */
+       mem_ctx = talloc_new(NULL);
+       if (mem_ctx == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+
+       lp_ctx = lpcfg_from_py_object(mem_ctx, py_lp_ctx);
        if (lp_ctx == NULL) {
                PyErr_SetString(PyExc_TypeError, "Expected loadparm context");
+               talloc_free(mem_ctx);
                return NULL;
        }
 
        credentials = cli_credentials_from_py_object(py_credentials);
        if (credentials == NULL) {
                PyErr_SetString(PyExc_TypeError, "Expected credentials");
+               talloc_free(mem_ctx);
                return NULL;
        }
 
@@ -291,42 +351,13 @@ static PyObject *py_open_samba(PyObject *self, PyObject *args, PyObject *kwargs)
 
        result = reg_open_samba(NULL, &reg_ctx, NULL, 
                                lp_ctx, session_info, credentials);
+       talloc_free(mem_ctx);
        if (!W_ERROR_IS_OK(result)) {
                PyErr_SetWERROR(result);
                return NULL;
        }
-       
-       return py_talloc_steal(&PyRegistry, reg_ctx);
-}
-
-static PyObject *py_open_directory(PyObject *self, PyObject *args)
-{
-       char *location;
-       WERROR result;
-       struct hive_key *key;
-
-       if (!PyArg_ParseTuple(args, "s", &location))
-               return NULL;
-
-       result = reg_open_directory(NULL, location, &key);
-       PyErr_WERROR_IS_ERR_RAISE(result);
-
-       return py_talloc_steal(&PyHiveKey, key);
-}
-
-static PyObject *py_create_directory(PyObject *self, PyObject *args)
-{
-       char *location;
-       WERROR result;
-       struct hive_key *key;
-
-       if (!PyArg_ParseTuple(args, "s", &location))
-               return NULL;
-
-       result = reg_create_directory(NULL, location, &key);
-       PyErr_WERROR_IS_ERR_RAISE(result);
 
-       return py_talloc_steal(&PyHiveKey, key);
+       return pytalloc_steal(&PyRegistry, reg_ctx);
 }
 
 static PyObject *py_open_ldb_file(PyObject *self, PyObject *args, PyObject *kwargs)
@@ -339,6 +370,7 @@ static PyObject *py_open_ldb_file(PyObject *self, PyObject *args, PyObject *kwar
        struct cli_credentials *credentials;
        struct hive_key *key;
        struct auth_session_info *session_info;
+       TALLOC_CTX *mem_ctx;
 
        if (!PyArg_ParseTupleAndKeywords(args, kwargs, "s|OOO", 
                                         discard_const_p(char *, kwnames),
@@ -346,25 +378,34 @@ static PyObject *py_open_ldb_file(PyObject *self, PyObject *args, PyObject *kwar
                                         &py_credentials, &py_lp_ctx))
                return NULL;
 
-       lp_ctx = lp_from_py_object(NULL, py_lp_ctx); /* FIXME: leaky */
+       mem_ctx = talloc_new(NULL);
+       if (mem_ctx == NULL) {
+               PyErr_NoMemory();
+               return NULL;
+       }
+
+       lp_ctx = lpcfg_from_py_object(mem_ctx, py_lp_ctx);
        if (lp_ctx == NULL) {
                PyErr_SetString(PyExc_TypeError, "Expected loadparm context");
+               talloc_free(mem_ctx);
                return NULL;
        }
 
        credentials = cli_credentials_from_py_object(py_credentials);
        if (credentials == NULL) {
                PyErr_SetString(PyExc_TypeError, "Expected credentials");
+               talloc_free(mem_ctx);
                return NULL;
        }
 
        session_info = NULL; /* FIXME */
 
        result = reg_open_ldb_file(NULL, location, session_info, credentials,
-                                  tevent_context_init(NULL), lp_ctx, &key);
-       PyErr_WERROR_IS_ERR_RAISE(result);
+                                  s4_event_context_init(NULL), lp_ctx, &key);
+       talloc_free(mem_ctx);
+       PyErr_WERROR_NOT_OK_RAISE(result);
 
-       return py_talloc_steal(&PyHiveKey, key);
+       return pytalloc_steal(&PyHiveKey, key);
 }
 
 static PyObject *py_str_regtype(PyObject *self, PyObject *args)
@@ -374,7 +415,7 @@ static PyObject *py_str_regtype(PyObject *self, PyObject *args)
        if (!PyArg_ParseTuple(args, "i", &regtype))
                return NULL;
        
-       return PyString_FromString(str_regtype(regtype));
+       return PyStr_FromString(str_regtype(regtype));
 }
 
 static PyObject *py_get_predef_name(PyObject *self, PyObject *args)
@@ -388,35 +429,42 @@ static PyObject *py_get_predef_name(PyObject *self, PyObject *args)
        str = reg_get_predef_name(hkey);
        if (str == NULL)
                Py_RETURN_NONE;
-       return PyString_FromString(str);
+       return PyStr_FromString(str);
 }
 
 static PyMethodDef py_registry_methods[] = {
        { "open_samba", (PyCFunction)py_open_samba, METH_VARARGS|METH_KEYWORDS, "open_samba() -> reg" },
-       { "open_directory", py_open_directory, METH_VARARGS, "open_dir(location) -> key" },
-       { "create_directory", py_create_directory, METH_VARARGS, "create_dir(location) -> key" },
        { "open_ldb", (PyCFunction)py_open_ldb_file, METH_VARARGS|METH_KEYWORDS, "open_ldb(location, session_info=None, credentials=None, loadparm_context=None) -> key" },
+       { "open_hive", (PyCFunction)py_open_hive, METH_VARARGS|METH_KEYWORDS, "open_hive(location, session_info=None, credentials=None, loadparm_context=None) -> key" },
        { "str_regtype", py_str_regtype, METH_VARARGS, "str_regtype(int) -> str" },
        { "get_predef_name", py_get_predef_name, METH_VARARGS, "get_predef_name(hkey) -> str" },
        { NULL }
 };
 
-void initregistry(void)
+static struct PyModuleDef moduledef = {
+    PyModuleDef_HEAD_INIT,
+    .m_name = "registry",
+    .m_doc = "Registry",
+    .m_size = -1,
+    .m_methods = py_registry_methods,
+};
+
+MODULE_INIT_FUNC(registry)
 {
        PyObject *m;
 
-       if (PyType_Ready(&PyHiveKey) < 0)
-               return;
+       if (pytalloc_BaseObject_PyType_Ready(&PyHiveKey) < 0)
+               return NULL;
 
-       if (PyType_Ready(&PyRegistry) < 0)
-               return;
+       if (pytalloc_BaseObject_PyType_Ready(&PyRegistry) < 0)
+               return NULL;
 
-       if (PyType_Ready(&PyRegistryKey) < 0)
-               return;
+       if (pytalloc_BaseObject_PyType_Ready(&PyRegistryKey) < 0)
+               return NULL;
 
-       m = Py_InitModule3("registry", py_registry_methods, "Registry");
+       m = PyModule_Create(&moduledef);
        if (m == NULL)
-               return;
+               return NULL;
 
        PyModule_AddObject(m, "HKEY_CLASSES_ROOT", PyInt_FromLong(HKEY_CLASSES_ROOT));
        PyModule_AddObject(m, "HKEY_CURRENT_USER", PyInt_FromLong(HKEY_CURRENT_USER));
@@ -436,4 +484,6 @@ void initregistry(void)
 
        Py_INCREF(&PyRegistryKey);
        PyModule_AddObject(m, "RegistryKey", (PyObject *)&PyRegistryKey);
+
+       return m;
 }