pidl: fix printing of server side ndr request debug messages
[amitay/samba.git] / pidl / lib / Parse / Pidl / Samba3 / ServerNDR.pm
index 6034fb6f160ecdef84152cc2612f3282638cd1b5..1cbf1876015bb96eb4e05880c6e540cd10ff2a53 100644 (file)
@@ -11,7 +11,7 @@ use Exporter;
 @EXPORT_OK = qw(DeclLevel);
 
 use strict;
-use Parse::Pidl qw(warning fatal);
+use Parse::Pidl qw(warning error fatal);
 use Parse::Pidl::Typelist qw(mapTypeName scalar_is_reference);
 use Parse::Pidl::Util qw(ParseExpr has_property is_constant);
 use Parse::Pidl::NDR qw(GetNextLevel);
@@ -48,9 +48,9 @@ sub DeclLevel($$)
        return $res;
 }
 
-sub AllocOutVar($$$$)
+sub AllocOutVar($$$$$)
 {
-       my ($e, $mem_ctx, $name, $env) = @_;
+       my ($e, $mem_ctx, $name, $env, $fail) = @_;
 
        my $l = $e->{LEVELS}[0];
 
@@ -58,44 +58,93 @@ sub AllocOutVar($$$$)
        if ($l->{TYPE} eq "POINTER") {
                my $nl = GetNextLevel($e, $l);
                $l = $nl if ($nl->{TYPE} eq "ARRAY");
-       }
+       } elsif
 
        # we don't support multi-dimentional arrays yet
-       if ($l->{TYPE} eq "ARRAY") {
+       ($l->{TYPE} eq "ARRAY") {
                my $nl = GetNextLevel($e, $l);
                if ($nl->{TYPE} eq "ARRAY") {
                        fatal($e->{ORIGINAL},"multi-dimentional [out] arrays are not supported!");
                }
+       } else {
+               # neither pointer nor array, no need to alloc something.
+               return;
        }
 
        if ($l->{TYPE} eq "ARRAY") {
-               my $size = ParseExpr($l->{SIZE_IS}, $env, $e);
-               pidl "$name = talloc_zero_array($mem_ctx, " . DeclLevel($e, 1) . ", $size);";
+               unless(defined($l->{SIZE_IS})) {
+                       error($e->{ORIGINAL}, "No size known for array `$e->{NAME}'");
+                       pidl "#error No size known for array `$e->{NAME}'";
+               } else {
+                       my $size = ParseExpr($l->{SIZE_IS}, $env, $e);
+                       pidl "$name = talloc_zero_array($mem_ctx, " . DeclLevel($e, 1) . ", $size);";
+               }
        } else {
                pidl "$name = talloc_zero($mem_ctx, " . DeclLevel($e, 1) . ");";
        }
 
        pidl "if ($name == NULL) {";
-       pidl "\ttalloc_free($mem_ctx);";
-       pidl "\treturn false;";
+       $fail->();
        pidl "}";
        pidl "";
 }
 
+sub CallWithStruct($$$$)
+{
+       my ($pipes_struct, $mem_ctx, $fn, $fail) = @_;
+       my $env = GenerateFunctionOutEnv($fn);
+       my $hasout = 0;
+       foreach (@{$fn->{ELEMENTS}}) {
+               if (grep(/out/, @{$_->{DIRECTION}})) { $hasout = 1; }
+       }
+
+       pidl "ZERO_STRUCT(r->out);" if ($hasout);
+
+       my $proto = "_$fn->{NAME}(struct pipes_struct *p, struct $fn->{NAME} *r";
+       my $ret = "_$fn->{NAME}($pipes_struct, r";
+       foreach (@{$fn->{ELEMENTS}}) {
+               my @dir = @{$_->{DIRECTION}};
+               if (grep(/in/, @dir) and grep(/out/, @dir)) {
+                       pidl "r->out.$_->{NAME} = r->in.$_->{NAME};";
+               }
+       }
+
+       foreach (@{$fn->{ELEMENTS}}) {
+               my @dir = @{$_->{DIRECTION}};
+               if (grep(/in/, @dir) and grep(/out/, @dir)) {
+                       # noop
+               } elsif (grep(/out/, @dir) and not
+                                has_property($_, "represent_as")) {
+                       AllocOutVar($_, $mem_ctx, "r->out.$_->{NAME}", $env, $fail);
+               }
+       }
+       $ret .= ")";
+       $proto .= ");";
+
+       if ($fn->{RETURN_TYPE}) {
+               $ret = "r->out.result = $ret";
+               $proto = "$fn->{RETURN_TYPE} $proto";
+       } else {
+               $proto = "void $proto";
+       }
+
+       pidl_hdr "$proto";
+       pidl "$ret;";
+}
+
 sub ParseFunction($$)
 {
        my ($if,$fn) = @_;
 
        my $op = "NDR_".uc($fn->{NAME});
 
-       pidl "static bool api_$fn->{NAME}(pipes_struct *p)";
+       pidl "static bool api_$fn->{NAME}(struct pipes_struct *p)";
        pidl "{";
        indent;
        pidl "const struct ndr_interface_call *call;";
        pidl "struct ndr_pull *pull;";
        pidl "struct ndr_push *push;";
        pidl "enum ndr_err_code ndr_err;";
-       pidl "DATA_BLOB blob;";
        pidl "struct $fn->{NAME} *r;";
        pidl "";
        pidl "call = &ndr_table_$if->{NAME}.calls[$op];";
@@ -105,18 +154,16 @@ sub ParseFunction($$)
        pidl "\treturn false;";
        pidl "}";
        pidl "";
-       pidl "if (!prs_data_blob(&p->in_data.data, &blob, r)) {";
-       pidl "\ttalloc_free(r);";
-       pidl "\treturn false;";
-       pidl "}";
-       pidl "";
-       pidl "pull = ndr_pull_init_blob(&blob, r, NULL);";
+       pidl "pull = ndr_pull_init_blob(&p->in_data.data, r);";
        pidl "if (pull == NULL) {";
        pidl "\ttalloc_free(r);";
        pidl "\treturn false;";
        pidl "}";
        pidl "";
        pidl "pull->flags |= LIBNDR_FLAG_REF_ALLOC;";
+       pidl "if (p->endian) {";
+       pidl "\tpull->flags |= LIBNDR_FLAG_BIGENDIAN;";
+       pidl "}";
        pidl "ndr_err = call->ndr_pull(pull, NDR_IN, r);";
        pidl "if (!NDR_ERR_CODE_IS_SUCCESS(ndr_err)) {";
        pidl "\ttalloc_free(r);";
@@ -124,41 +171,16 @@ sub ParseFunction($$)
        pidl "}";
        pidl "";
        pidl "if (DEBUGLEVEL >= 10) {";
-       pidl "\tNDR_PRINT_IN_DEBUG($fn->{NAME}, r);";
+       pidl "\tNDR_PRINT_FUNCTION_DEBUG($fn->{NAME}, NDR_IN, r);";
        pidl "}";
        pidl "";
 
-       my $env = GenerateFunctionOutEnv($fn);
-       my $hasout = 0;
-       foreach (@{$fn->{ELEMENTS}}) {
-               if (grep(/out/, @{$_->{DIRECTION}})) { $hasout = 1; }
-       }
-
-       pidl "ZERO_STRUCT(r->out);" if ($hasout);
-
-       my $proto = "_$fn->{NAME}(pipes_struct *p, struct $fn->{NAME} *r";
-       my $ret = "_$fn->{NAME}(p, r";
-       foreach (@{$fn->{ELEMENTS}}) {
-               my @dir = @{$_->{DIRECTION}};
-               if (grep(/in/, @dir) and grep(/out/, @dir)) {
-                       pidl "r->out.$_->{NAME} = r->in.$_->{NAME};";
-               } elsif (grep(/out/, @dir) and not
-                                has_property($_, "represent_as")) {
-                       AllocOutVar($_, "r", "r->out.$_->{NAME}", $env);
+       CallWithStruct("p", "r", $fn, 
+       sub { 
+                       pidl "\ttalloc_free(r);";
+                       pidl "\treturn false;";
                }
-       }
-       $ret .= ")";
-       $proto .= ");";
-
-       if ($fn->{RETURN_TYPE}) {
-               $ret = "r->out.result = $ret";
-               $proto = "$fn->{RETURN_TYPE} $proto";
-       } else {
-               $proto = "void $proto";
-       }
-
-       pidl_hdr "$proto";
-       pidl "$ret;";
+       );
 
        pidl "";
        pidl "if (p->rng_fault_state) {";
@@ -168,10 +190,10 @@ sub ParseFunction($$)
        pidl "}";
        pidl "";
        pidl "if (DEBUGLEVEL >= 10) {";
-       pidl "\tNDR_PRINT_OUT_DEBUG($fn->{NAME}, r);";
+       pidl "\nNDR_PRINT_FUNCTION_DEBUG($fn->{NAME}, NDR_OUT | NDR_SET_VALUES, r);";
        pidl "}";
        pidl "";
-       pidl "push = ndr_push_init_ctx(r, NULL);";
+       pidl "push = ndr_push_init_ctx(r);";
        pidl "if (push == NULL) {";
        pidl "\ttalloc_free(r);";
        pidl "\treturn false;";
@@ -183,11 +205,8 @@ sub ParseFunction($$)
        pidl "\treturn false;";
        pidl "}";
        pidl "";
-       pidl "blob = ndr_push_blob(push);";
-       pidl "if (!prs_copy_data_in(&p->out_data.rdata, (const char *)blob.data, (uint32_t)blob.length)) {";
-       pidl "\ttalloc_free(r);";
-       pidl "\treturn false;";
-       pidl "}";
+       pidl "p->out_data.rdata = ndr_push_blob(push);";
+       pidl "talloc_steal(p->mem_ctx, p->out_data.rdata.data);";
        pidl "";
        pidl "talloc_free(r);";
        pidl "";
@@ -205,7 +224,11 @@ sub ParseInterface($)
 
        pidl_hdr "#ifndef __SRV_$uif\__";
        pidl_hdr "#define __SRV_$uif\__";
-       ParseFunction($if, $_) foreach (@{$if->{FUNCTIONS}});
+
+       foreach (@{$if->{FUNCTIONS}}) {
+               next if ($_->{PROPERTIES}{noopnum});
+               ParseFunction($if, $_);
+       }
 
        pidl "";
        pidl "/* Tables */";
@@ -214,6 +237,7 @@ sub ParseInterface($)
        indent;
 
        foreach (@{$if->{FUNCTIONS}}) {
+               next if ($_->{PROPERTIES}{noopnum});
                pidl "{\"" . uc($_->{NAME}) . "\", NDR_" . uc($_->{NAME}) . ", api_$_->{NAME}},";
        }
 
@@ -232,12 +256,22 @@ sub ParseInterface($)
        pidl "}";
        pidl "";
 
-       pidl_hdr "NTSTATUS rpc_$if->{NAME}_init(void);";
-       pidl "NTSTATUS rpc_$if->{NAME}_init(void)";
-       pidl "{";
-       pidl "\treturn rpc_pipe_register_commands(SMB_RPC_INTERFACE_VERSION, \"$if->{NAME}\", \"$if->{NAME}\", \&ndr_table_$if->{NAME}.syntax_id, api_$if->{NAME}_cmds, sizeof(api_$if->{NAME}_cmds) / sizeof(struct api_struct));";
-       pidl "}";
-
+       if (not has_property($if, "no_srv_register")) {
+           pidl_hdr "struct rpc_srv_callbacks;";
+           pidl_hdr "NTSTATUS rpc_$if->{NAME}_init(const struct rpc_srv_callbacks *rpc_srv_cb);";
+           pidl "NTSTATUS rpc_$if->{NAME}_init(const struct rpc_srv_callbacks *rpc_srv_cb)";
+           pidl "{";
+           pidl "\treturn rpc_srv_register(SMB_RPC_INTERFACE_VERSION, \"$if->{NAME}\", \"$if->{NAME}\", \&ndr_table_$if->{NAME}, api_$if->{NAME}_cmds, sizeof(api_$if->{NAME}_cmds) / sizeof(struct api_struct), rpc_srv_cb);";
+           pidl "}";
+
+           pidl "";
+
+           pidl_hdr "NTSTATUS rpc_$if->{NAME}_shutdown(void);";
+           pidl "NTSTATUS rpc_$if->{NAME}_shutdown(void)";
+           pidl "{";
+           pidl "\treturn rpc_srv_unregister(\&ndr_table_$if->{NAME});";
+           pidl "}";
+       }
        pidl_hdr "#endif /* __SRV_$uif\__ */";
 }