diff options
| author | Andrew Tridgell <tridge@samba.org> | 2003-12-12 03:59:09 +0000 |
|---|---|---|
| committer | Andrew Tridgell <tridge@samba.org> | 2003-12-12 03:59:09 +0000 |
| commit | 16309de71d6c8de96e869aeaab0b879185991d87 (patch) | |
| tree | d122fe6bd25cd87df5422b0af661c9e93db31ef8 /source4/build/pidl/parser.pm | |
| parent | fcc4efd1ea637c810eed8444080b87d7f92c837a (diff) | |
| download | samba-16309de71d6c8de96e869aeaab0b879185991d87.tar.gz samba-16309de71d6c8de96e869aeaab0b879185991d87.tar.bz2 samba-16309de71d6c8de96e869aeaab0b879185991d87.zip | |
* the RPC-ECHO pipe now works in smbd, as long as the data sizes
don't cause fragmented pdus (I'll add fragments shortly)
* change data_blob_talloc() to not zero memory when the 2nd argument
is NULL. The zeroing just masks bugs, and can't even allow a DOS
attack
* modified pidl to ensure that [ref] arguments to the out side of
functions are allocated when parsing the in side. This allows rpc
backends to assume that [ref] variables are all setup. Doesn't work
correctly for [ref] arrays yet
* changed DLIST_ADD_END() to take the type instead of a tmp
variable. This means you don't need to declare a silly tmp variable in
the caller
(This used to be commit 46e0a358198eeb9af1907ee2a29025d3ab23b6d1)
Diffstat (limited to 'source4/build/pidl/parser.pm')
| -rw-r--r-- | source4/build/pidl/parser.pm | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/source4/build/pidl/parser.pm b/source4/build/pidl/parser.pm index ee24e9d5ef..88ba348486 100644 --- a/source4/build/pidl/parser.pm +++ b/source4/build/pidl/parser.pm @@ -1256,6 +1256,11 @@ sub ParseFunctionPull($) if (util::has_property($e, "in")) { ParseFunctionElementPull($e, "in"); } + # we need to allocate any reference output variables, so that + # a dcerpc backend can be sure they are non-null + if (util::has_property($e, "out") && util::has_property($e, "ref")) { + pidl "\tNDR_ALLOC(ndr, r->out.$e->{NAME});\n"; + } } pidl "\nndr_out:\n"; |