pidl/python: Work towards supporting more complex types (arrays, etc).
[ira/wip.git] / source / pidl / lib / Parse / Pidl / Samba4 / Python.pm
index 7c379548ed4668a1d8bc7348096981f36a33d230..545d233d08e51f8e47cb62250cb0b9e8a385a26e 100644 (file)
@@ -1,6 +1,6 @@
 ###################################################
 # Python function wrapper generator
-# Copyright jelmer@samba.org 2007
+# Copyright jelmer@samba.org 2007-2008
 # released under the GNU GPL
 
 package Parse::Pidl::Samba4::Python;
@@ -9,8 +9,9 @@ use Exporter;
 @ISA = qw(Exporter);
 
 use strict;
-use Parse::Pidl::Typelist qw(hasType getType mapTypeName);
+use Parse::Pidl::Typelist qw(hasType getType mapTypeName expandAlias);
 use Parse::Pidl::Util qw(has_property ParseExpr);
+use Parse::Pidl::CUtil qw(get_value_of get_pointer_to);
 
 use vars qw($VERSION);
 $VERSION = '0.01';
@@ -64,7 +65,7 @@ sub Import
 sub Const($$)
 {
     my ($self, $const) = @_;
-       $self->register_constant($const->{NAME}, $const->{DATA}->{TYPE}, $const->{VALUE});
+       $self->register_constant($const->{NAME}, $const->{DTYPE}, $const->{VALUE});
 }
 
 sub register_constant($$$$)
@@ -78,6 +79,8 @@ sub EnumAndBitmapConsts($$$)
 {
        my ($self, $name, $d) = @_;
 
+       return unless (defined($d->{ELEMENTS}));
+
        foreach my $e (@{$d->{ELEMENTS}}) {
                $e =~ /^([A-Za-z0-9_]+)=(.*)$/;
                my $cname = $1;
@@ -86,84 +89,142 @@ sub EnumAndBitmapConsts($$$)
        }
 }
 
-sub FromTypeToPythonFunction($$)
+sub FromUnionToPythonFunction($$$)
 {
-       my ($self, $type) = @_;
+       my ($self, $type, $switch, $name) = @_;
 
-       #FIXME
-}
+       $self->pidl("switch ($switch) {");
+       $self->indent;
 
-sub FromPythonToTypeFunction($$)
-{
-       my ($self, $type) = @_;
+       foreach my $e (@{$type->{ELEMENTS}}) {
+               my $conv;
+               
+               if ($e->{NAME}) {
+                       $conv = $self->ConvertObjectToPython($e->{TYPE}, "$name->$e->{NAME}");
+               } else {
+                       $conv = "Py_None";
+               }
+               if (defined($e->{CASE})) {
+                       $self->pidl("$e->{CASE}: return $conv;");
+               } else {
+                       $self->pidl("default: return $conv;");
+               }
+       }
 
-       #FIXME
+       $self->deindent;
+       $self->pidl("}");
+
+       $self->pidl("PyErr_SetString(PyExc_TypeError, \"unknown union level\");");
+       $self->pidl("return NULL;");
 }
 
-sub TypeConstructor($$)
+sub FromPythonToUnionFunction($$$$$)
 {
-       my ($self, $type) = @_;
+       my ($self, $type, $typename, $switch, $mem_ctx, $name) = @_;
 
-       $self->pidl("staticforward PyTypeObject $type->{NAME}_ObjectType;");
-       $self->pidl("typedef struct {");
-       $self->indent;
-       $self->pidl("PyObject_HEAD");
-       $self->pidl("void *object;"); # FIXME: Use real type rather than void
-       $self->deindent;
-       $self->pidl("} $type->{NAME}_Object;");
+       my $has_default = 0;
 
-       $self->pidl("");
+       $self->pidl("$typename *ret = talloc_zero($mem_ctx, $typename);");
 
-       $self->pidl("static PyObject *py_$type->{NAME}_getattr(PyTypeObject *obj, char *name)");
-       $self->pidl("{");
+       $self->pidl("switch ($switch) {");
        $self->indent;
-       $self->pidl("$type->{NAME}_Object *py_object = ($type->{NAME}_Object *)obj;");
-       $self->pidl(mapTypeName($type) . " *object = talloc_get_type(py_object->object, ".mapTypeName($type).");");
-       $self->pidl("return Py_None;");
+
+       foreach my $e (@{$type->{ELEMENTS}}) {
+               if (defined($e->{CASE})) {
+                       $self->pidl("$e->{CASE}:");
+               } else {
+                       $has_default = 1;
+                       $self->pidl("default:");
+               }
+               $self->indent;
+               if ($e->{NAME}) {
+                       $self->ConvertObjectFromPython($mem_ctx, $e->{TYPE}, $name, "ret->$e->{NAME}");
+               }
+               $self->pidl("break;");
+               $self->deindent;
+               $self->pidl("");
+       }
+
+       if (!$has_default) {
+               $self->pidl("default:");
+               $self->indent;
+               $self->pidl("PyErr_SetString(PyExc_TypeError, \"invalid union level value\");");
+               $self->pidl("talloc_free(ret);");
+               $self->pidl("ret = NULL;");
+               $self->deindent;
+       }
+
        $self->deindent;
        $self->pidl("}");
+       $self->pidl("");
+       $self->pidl("return ret;");
+}
+
+sub PythonStruct($$$$)
+{
+       my ($self, $name, $cname, $d) = @_;
+
+       $self->pidl("staticforward PyTypeObject $name\_ObjectType;");
+
        $self->pidl("");
 
-       $self->pidl("static void py_$type->{NAME}_dealloc(PyObject* self)");
+       $self->pidl("static PyObject *py_$name\_getattr(PyObject *obj, char *name)");
        $self->pidl("{");
        $self->indent;
-       $self->pidl("$type->{NAME}_Object *obj = ($type->{NAME}_Object *)self;");
-       $self->pidl("talloc_free(obj->object);");
-       $self->pidl("PyObject_Del(self);");
+       $self->pidl("$cname *object = py_talloc_get_type(obj, $cname);");
+       foreach my $e (@{$d->{ELEMENTS}}) {
+               $self->pidl("if (!strcmp(name, \"$e->{NAME}\")) {");
+               my $varname = "object->$e->{NAME}";
+               $self->indent;
+               $self->pidl("return ".$self->ConvertObjectToPython($e->{TYPE}, $varname) . ";");
+               $self->deindent;
+               $self->pidl("}");
+       }
+       $self->pidl("PyErr_SetString(PyExc_AttributeError, \"no such attribute\");");
+       $self->pidl("return NULL;");
        $self->deindent;
        $self->pidl("}");
        $self->pidl("");
 
-       $self->pidl("static PyObject *py_$type->{NAME}_setattr(PyTypeObject *obj, char *name, PyObject *value)");
+       $self->pidl("static int py_$name\_setattr(PyObject *py_obj, char *name, PyObject *value)");
        $self->pidl("{");
        $self->indent;
-       $self->pidl("$type->{NAME}_Object *py_object = ($type->{NAME}_Object *)obj;");
-       $self->pidl(mapTypeName($type) . " *object = talloc_get_type(py_object->object, ".mapTypeName($type).");");
-       $self->pidl("return Py_None;");
+       $self->pidl("$cname *object = py_talloc_get_type(py_obj, $cname);");
+       foreach my $e (@{$d->{ELEMENTS}}) {
+               $self->pidl("if (!strcmp(name, \"$e->{NAME}\")) {");
+               my $varname = "object->$e->{NAME}";
+               $self->indent;
+               $self->pidl("/* FIXME: talloc_free($varname) if necessary */");
+               $self->ConvertObjectFromPython("mem_ctx", $e->{TYPE}, "value", $varname);
+               $self->pidl("return 0;");
+               $self->deindent;
+               $self->pidl("}");
+       }
+       $self->pidl("PyErr_SetString(PyExc_AttributeError, \"no such attribute\");");
+       $self->pidl("return -1;");
        $self->deindent;
        $self->pidl("}");
        $self->pidl("");
 
-       $self->pidl("static PyTypeObject $type->{NAME}_ObjectType = {");
+       $self->pidl("static PyTypeObject $name\_ObjectType = {");
        $self->indent;
        $self->pidl("PyObject_HEAD_INIT(NULL) 0,");
-       $self->pidl(".tp_name = \"$type->{NAME}\",");
-       $self->pidl(".tp_basicsize = sizeof($type->{NAME}_Object),");
-       $self->pidl(".tp_dealloc = (destructor)py_$type->{NAME}_dealloc,");
-       $self->pidl(".tp_getattr = (getattrfunc)py_$type->{NAME}_getattr,");
-       $self->pidl(".tp_setattr = (setattrfunc)py_$type->{NAME}_setattr,");
+       $self->pidl(".tp_name = \"$name\",");
+       $self->pidl(".tp_basicsize = sizeof(py_talloc_Object),");
+       $self->pidl(".tp_dealloc = py_talloc_dealloc,");
+       $self->pidl(".tp_getattr = py_$name\_getattr,");
+       $self->pidl(".tp_setattr = py_$name\_setattr,");
        $self->deindent;
        $self->pidl("};");
 
        $self->pidl("");
 
-       my $py_fnname = "py_$type->{NAME}";
+       my $py_fnname = "py_$name";
        $self->pidl("static PyObject *$py_fnname(PyObject *self, PyObject *args)");
        $self->pidl("{");
        $self->indent;
-       $self->pidl("$type->{NAME}\_Object *ret;");
-       $self->pidl("ret = PyObject_New($type->{NAME}_Object, &$type->{NAME}_ObjectType);");
-       $self->pidl("return (PyObject *) ret;");
+       $self->pidl("$cname *ret = talloc_zero(NULL, $cname);");
+       $self->pidl("return py_talloc_import(&$name\_ObjectType, ret);");
        $self->deindent;
        $self->pidl("}");
        $self->pidl("");
@@ -180,22 +241,66 @@ sub PythonFunction($$$)
        $self->indent;
        $self->pidl("$iface\_InterfaceObject *iface = ($iface\_InterfaceObject *)self;");
        $self->pidl("NTSTATUS status;");
-       $self->pidl("");
-       # FIXME
-       $self->handle_ntstatus("status", "NULL");
-       $self->pidl("return Py_None;");
+       $self->pidl("TALLOC_CTX *mem_ctx = talloc_new(NULL);");
+       $self->pidl("struct $fn->{NAME} r;");
+       $self->pidl("PyObject *result;");
+       my $result_size = 0;
+
+       foreach my $e (@{$fn->{ELEMENTS}}) {
+               if (grep(/in/,@{$e->{DIRECTION}})) {
+                       $self->pidl("PyObject *py_$e->{NAME};");
+               }
+               if (grep(/out/,@{$e->{DIRECTION}})) {
+                       $result_size++;
+               }
+       }
+       if ($result_size > 0) {
+               $self->pidl("");
+               $self->pidl("ZERO_STRUCT(r.out);");
+       }
+       if ($fn->{RETURN_TYPE}) {
+               $result_size++;
+       }
+
+       foreach my $e (@{$fn->{ELEMENTS}}) {
+               if (grep(/in/,@{$e->{DIRECTION}})) {
+                       $self->ConvertObjectFromPython("mem_ctx", $e->{TYPE}, "py_$e->{NAME}", "r.in.$e->{NAME}");
+               }
+       }
+       $self->pidl("status = dcerpc_$fn->{NAME}(iface->pipe, mem_ctx, &r);");
+       $self->handle_ntstatus("status", "NULL", "mem_ctx");
+
+       $self->pidl("result = PyTuple_New($result_size);");
+
+       my $i = 0;
+
+       foreach my $e (@{$fn->{ELEMENTS}}) {
+               if (grep(/out/,@{$e->{DIRECTION}})) {
+                       $self->pidl("PyTuple_SetItem(result, $i, " . $self->ConvertObjectToPython($e->{TYPE}, "r.out.$e->{NAME}") . ");");
+
+                       $i++;
+               }
+       }
+
+       if (defined($fn->{RETURN_TYPE})) {
+               $self->pidl("PyTuple_SetItem(result, $i, " . $self->ConvertObjectToPython($fn->{RETURN_TYPE}, "r.out.result") . ");");
+       }
+
+       $self->pidl("talloc_free(mem_ctx);");
+       $self->pidl("return result;");
        $self->deindent;
        $self->pidl("}");
        $self->pidl("");
 }
 
-sub handle_ntstatus($$$)
+sub handle_ntstatus($$$$)
 {
-       my ($self, $var, $retval) = @_;
+       my ($self, $var, $retval, $mem_ctx) = @_;
 
        $self->pidl("if (NT_STATUS_IS_ERR($var)) {");
        $self->indent;
        $self->pidl("PyErr_SetString(PyExc_RuntimeError, nt_errstr($var));");
+       $self->pidl("talloc_free($mem_ctx);") if ($mem_ctx);
        $self->pidl("return $retval;");
        $self->deindent;
        $self->pidl("}");
@@ -206,18 +311,25 @@ sub PythonType($$$)
 {
        my ($self, $d, $interface, $basename) = @_;
 
-       if ($d->{TYPE} eq "STRUCT" or $d->{TYPE} eq "TYPEDEF" and 
-               $d->{DATA}->{TYPE} eq "STRUCT") {
-               $self->FromTypeToPythonFunction($d);
-               $self->FromPythonToTypeFunction($d);
-               my $py_fnname = $self->TypeConstructor($d);
+       my $actual_ctype = $d;
+       if ($actual_ctype->{TYPE} eq "TYPEDEF") {
+               $actual_ctype = $actual_ctype->{DATA};
+       }
+
+       if ($actual_ctype->{TYPE} eq "STRUCT") {
+               my $py_fnname;
+               if ($d->{TYPE} eq "STRUCT") {
+                       $py_fnname = $self->PythonStruct($d->{NAME}, mapTypeName($d), $d);
+               } else {
+                       $py_fnname = $self->PythonStruct($d->{NAME}, mapTypeName($d), $d->{DATA});
+               }
 
                my $fn_name = $d->{NAME};
 
                $fn_name =~ s/^$interface->{NAME}_//;
                $fn_name =~ s/^$basename\_//;
 
-               $self->register_module_method($fn_name, $py_fnname, "METH_VARARGS|METH_KEYWORDS", "NULL");
+               $self->register_module_method($fn_name, $py_fnname, "METH_NOARGS", "NULL");
        }
 
        if ($d->{TYPE} eq "ENUM" or $d->{TYPE} eq "BITMAP") {
@@ -227,6 +339,24 @@ sub PythonType($$$)
        if ($d->{TYPE} eq "TYPEDEF" and ($d->{DATA}->{TYPE} eq "ENUM" or $d->{DATA}->{TYPE} eq "BITMAP")) {
                $self->EnumAndBitmapConsts($d->{NAME}, $d->{DATA});
        }
+
+       if ($actual_ctype->{TYPE} eq "UNION") {
+               $self->pidl("PyObject *py_import_$d->{NAME}(int level, " .mapTypeName($d) . " *in)");
+               $self->pidl("{");
+               $self->indent;
+               $self->FromUnionToPythonFunction($actual_ctype, "level", "in") if ($actual_ctype->{TYPE} eq "UNION");
+               $self->deindent;
+               $self->pidl("}");
+               $self->pidl("");
+
+               $self->pidl(mapTypeName($d) . " *py_export_$d->{NAME}(TALLOC_CTX *mem_ctx, int level, PyObject *in)");
+               $self->pidl("{");
+               $self->indent;
+               $self->FromPythonToUnionFunction($actual_ctype, mapTypeName($d), "level", "mem_ctx", "in") if ($actual_ctype->{TYPE} eq "UNION");
+               $self->deindent;
+               $self->pidl("}");
+               $self->pidl("");
+       }
 }
 
 sub Interface($$$)
@@ -290,10 +420,10 @@ sub Interface($$$)
        $self->pidl("}");
        $self->pidl("");
 
-       $self->pidl("static PyObject *interface_$interface->{NAME}_getattr(PyTypeObject *obj, char *name)");
+       $self->pidl("static PyObject *interface_$interface->{NAME}_getattr(PyObject *obj, char *name)");
        $self->pidl("{");
        $self->indent;
-       $self->pidl("return Py_FindMethod(interface_$interface->{NAME}\_methods, (PyObject *)obj, name);");
+       $self->pidl("return Py_FindMethod(interface_$interface->{NAME}\_methods, obj, name);");
        $self->deindent;
        $self->pidl("}");
 
@@ -304,8 +434,8 @@ sub Interface($$$)
        $self->pidl("PyObject_HEAD_INIT(NULL) 0,");
        $self->pidl(".tp_name = \"$interface->{NAME}\",");
        $self->pidl(".tp_basicsize = sizeof($interface->{NAME}_InterfaceObject),");
-       $self->pidl(".tp_dealloc = interface_$interface->{NAME}_dealloc,");
-       $self->pidl(".tp_getattr = interface_$interface->{NAME}_getattr,");
+       $self->pidl(".tp_dealloc = (destructor)interface_$interface->{NAME}_dealloc,");
+       $self->pidl(".tp_getattr = (getattrfunc)interface_$interface->{NAME}_getattr,");
        $self->deindent;
        $self->pidl("};");
 
@@ -319,6 +449,7 @@ sub Interface($$$)
        $self->pidl("const char *binding_string;");
        $self->pidl("struct cli_credentials *credentials;");
        $self->pidl("struct loadparm_context *lp_ctx;");
+       $self->pidl("TALLOC_CTX *mem_ctx = NULL;");
        $self->pidl("NTSTATUS status;");
        $self->pidl("");
 
@@ -328,7 +459,7 @@ sub Interface($$$)
 
        $self->pidl("status = dcerpc_pipe_connect(NULL, &ret->pipe, binding_string, ");
        $self->pidl("             &ndr_table_$interface->{NAME}, credentials, NULL, lp_ctx);");
-       $self->handle_ntstatus("status", "NULL");
+       $self->handle_ntstatus("status", "NULL", "mem_ctx");
 
        $self->pidl("return (PyObject *)ret;");
        $self->deindent;
@@ -347,9 +478,162 @@ sub register_module_method($$$$$)
        push (@{$self->{module_methods}}, [$fn_name, $pyfn_name, $flags, $doc])
 }
 
-sub Parse($$$$)
+sub ConvertObjectFromPython($$$$$)
+{
+       my ($self, $mem_ctx, $ctype, $cvar, $target) = @_;
+
+       die("undef type for $cvar") unless(defined($ctype));
+
+       if (ref($ctype) ne "HASH") {
+               $ctype = getType($ctype);
+       }
+
+       if (ref($ctype) ne "HASH") {
+               $self->pidl("$target = FIXME($cvar);");
+               return;
+       }
+
+       my $actual_ctype = $ctype;
+       if ($ctype->{TYPE} eq "TYPEDEF") {
+               $actual_ctype = $ctype->{DATA};
+       }
+
+       if ($actual_ctype->{TYPE} eq "ENUM" or $actual_ctype->{TYPE} eq "BITMAP" or 
+               $actual_ctype->{TYPE} eq "SCALAR" and (
+               expandAlias($actual_ctype->{NAME}) =~ /^(u?int[0-9]+|hyper|NTTIME|time_t|NTTIME_hyper|NTTIME_1sec|dlong|udlong|udlongr)$/)) {
+               $self->pidl("$target = PyInt_AsLong($cvar);");
+               return;
+       }
+
+       if ($actual_ctype->{TYPE} eq "STRUCT") {
+               $self->pidl("$target = py_talloc_get_type($cvar, " . mapTypeName($ctype) . ");");
+               return;
+       }
+
+       if ($actual_ctype->{TYPE} eq "UNION") {
+               $self->pidl("$target = py_export_$ctype->{NAME}($cvar);");
+               return;
+       }
+
+       if ($actual_ctype->{TYPE} eq "SCALAR" and $actual_ctype->{NAME} eq "DATA_BLOB") {
+               $self->pidl("$target = data_blob_talloc($mem_ctx, PyString_AsString($cvar), PyString_Size($cvar));");
+               return;
+       }
+
+       if ($actual_ctype->{TYPE} eq "SCALAR" and 
+               ($actual_ctype->{NAME} eq "string" or $actual_ctype->{NAME} eq "nbt_string" or $actual_ctype->{NAME} eq "nbt_name" or $actual_ctype->{NAME} eq "wrepl_nbt_name")) {
+               $self->pidl("$target = talloc_strdup($mem_ctx, PyString_AsString($cvar));");
+               return;
+       }
+
+       if ($actual_ctype->{TYPE} eq "SCALAR" and $actual_ctype->{NAME} eq "ipv4address") {
+               $self->pidl("$target = FIXME($cvar);");
+               return;
+               }
+
+
+       if ($actual_ctype->{TYPE} eq "SCALAR" and $actual_ctype->{NAME} eq "NTSTATUS") {
+               $self->pidl("$target = PyInt_AsLong($cvar);");
+               return;
+       }
+
+       if ($actual_ctype->{TYPE} eq "SCALAR" and $actual_ctype->{NAME} eq "WERROR") {
+               $self->pidl("$target = PyInt_AsLong($cvar);");
+               return;
+       }
+
+       if ($actual_ctype->{TYPE} eq "SCALAR" and $actual_ctype->{NAME} eq "string_array") {
+               $self->pidl("$target = FIXME($cvar);");
+               return;
+       }
+
+       if ($actual_ctype->{TYPE} eq "SCALAR" and $actual_ctype->{NAME} eq "pointer") {
+               $self->pidl("$target = PyCObject_AsVoidPtr($cvar);");
+               return;
+       }
+
+       die("unknown type ".mapTypeName($ctype) . ": $cvar");
+}
+
+sub ConvertScalarToPython($$$)
+{
+       my ($self, $ctypename, $cvar) = @_;
+
+       $ctypename = expandAlias($ctypename);
+
+       if ($ctypename =~ /^(int|long|char|u?int[0-9]+|hyper|dlong|udlong|udlongr|time_t|NTTIME_hyper|NTTIME|NTTIME_1sec)$/) {
+               return "PyInt_FromLong($cvar)";
+       }
+
+       if ($ctypename eq "DATA_BLOB") {
+               return "PyString_FromStringAndSize($cvar->data, $cvar->length)";
+       }
+
+       if ($ctypename eq "NTSTATUS") {
+               return "PyInt_FromLong(NT_STATUS_V($cvar))";
+       }
+
+       if ($ctypename eq "WERROR") {
+               return "PyInt_FromLong(W_ERROR_V($cvar))";
+       }
+
+       if (($ctypename eq "string" or $ctypename eq "nbt_string" or $ctypename eq "nbt_name" or $ctypename eq "wrepl_nbt_name")) {
+               return "PyString_FromString($cvar)";
+       }
+
+       if ($ctypename eq "string_array") { return "FIXME($cvar)"; }
+
+       if ($$ctypename eq "ipv4address") { return "FIXME($cvar)"; }
+       if ($ctypename eq "pointer") {
+               return "PyCObject_FromVoidPtr($cvar, talloc_free)";
+       }
+
+       die("Unknown scalar type $ctypename");
+}
+
+sub ConvertObjectToPython($$$)
+{
+       my ($self, $ctype, $cvar) = @_;
+
+       die("undef type for $cvar") unless(defined($ctype));
+
+       if (ref($ctype) ne "HASH") {
+               if (not hasType($ctype)) {
+                       if (ref($ctype) eq "HASH") {
+                               return "py_import_$ctype->{TYPE}_$ctype->{NAME}($cvar)";
+                       } else {
+                               return "py_import_$ctype($cvar)"; # best bet
+                       }
+               }
+
+               $ctype = getType($ctype);
+       }
+
+       my $actual_ctype = $ctype;
+       if ($ctype->{TYPE} eq "TYPEDEF") {
+               $actual_ctype = $ctype->{DATA};
+       }
+
+       if ($actual_ctype->{TYPE} eq "ENUM" or $actual_ctype->{TYPE} eq "BITMAP" or 
+               ($actual_ctype->{TYPE} eq "SCALAR") {
+               return $self->ConvertScalarToPython($actual_ctype->{NAME}, $cvar);
+       }
+
+       if ($actual_ctype->{TYPE} eq "UNION") {
+               return "py_import_$ctype->{NAME}($cvar)";
+       }
+
+       if ($actual_ctype->{TYPE} eq "STRUCT") {
+               # FIXME: if $cvar is not a pointer, do a talloc_dup()
+               return "py_talloc_import(&$ctype->{NAME}_ObjectType, $cvar)";
+       }
+
+       die("unknown type ".mapTypeName($ctype) . ": $cvar");
+}
+
+sub Parse($$$$$)
 {
-    my($self,$basename,$ndr,$hdr) = @_;
+    my($self,$basename,$ndr,$ndr_hdr,$hdr) = @_;
     
     my $py_hdr = $hdr;
     $py_hdr =~ s/ndr_([^\/]+)$/py_$1/g;
@@ -361,7 +645,9 @@ sub Parse($$$$)
 #include \"includes.h\"
 #include <Python.h>
 #include \"librpc/rpc/dcerpc.h\"
+#include \"scripting/python/pytalloc.h\"
 #include \"$hdr\"
+#include \"$ndr_hdr\"
 #include \"$py_hdr\"
 
 ");
@@ -390,8 +676,17 @@ sub Parse($$$$)
        $self->pidl("PyObject *m;");
        $self->pidl("m = Py_InitModule(\"$basename\", $basename\_methods);");
        foreach (keys %{$self->{constants}}) {
-               # FIXME: Handle non-string constants
-               $self->pidl("PyModule_AddObject(m, \"$_\", PyString_FromString(" . $self->{constants}->{$_}->[1] . "));");
+               my $py_obj;
+               my ($ctype, $cvar) = @{$self->{constants}->{$_}};
+               if ($cvar =~ /^[0-9]+$/ or $cvar =~ /^0x[0-9a-fA-F]+$/) {
+                       $py_obj = "PyInt_FromLong($cvar)";
+               } elsif ($cvar =~ /^".*"$/) {
+                       $py_obj = "PyString_FromString($cvar)";
+               } else {
+                       $py_obj = $self->ConvertScalarToPython($ctype, $cvar);
+               }
+
+               $self->pidl("PyModule_AddObject(m, \"$_\", $py_obj);");
        }
        $self->deindent;
        $self->pidl("}");