Age | Commit message (Collapse) | Author | Files | Lines |
|
relative_short is like relative but instead of having the offset coded on 4 bytes
it's coded on 2 bytes. Such things happen in GET_DFS_REFERAL messages.
Signed-off-by: Stefan Metzmacher <metze@samba.org>
|
|
a20e095 used (uint8_t) instead of (uint8_t *).
Guenther
|
|
foo / 5 * sizeof(bar)' isn't the same as
'(foo / 2) * sizeof(bar)'.
metze
|
|
foo / 5 * sizeof(bar)' isn't the same as
'(foo / 2) * sizeof(bar)'.
metze
|
|
This reverts commit c2cdb4ad5c9398ef0d3310613107999f8d33c7ce.
It's not needed anymore.
metze
|
|
relative pointers
metze
|
|
This reverts commit daa964013bc5d036f4da571ce22c0052ef40943a.
|
|
This reverts commit 98fb71782e05ae72cd6abeb38b6e0b96a50c1761.
|
|
This reverts commit 494b2aff8826947e3bd556aecb175746163da485.
|
|
|
|
This involves storing the interface table in the pipes_struct
|
|
|
|
12 insertions(+), 10651 deletions(-)
I think that says it all :-)
|
|
When we have an enum that is used as a union discriminator, what
matters is that the scalar mappings are the same, not if the types are
the same (otherwise we get warnings about uint1632).
Thanks to gd for noticing this.
|
|
TDB doesn't have NDR64, but it needs to know how to map the new types
from pidl
|
|
The default enum in NDR63 is 32 bits, not 16 bits. We need a uint1632
type to get the alignment right.
|
|
|
|
List,
Please review this patch to pidl.
Basically, we need to process the wireshark conformance file BEFORE
we process the idl file since this file may define external types
and set the alignment for them (using the TYPE directive).
Otherwise pidl will default all external types to use 4byte alignment
which breaks (much more often) on NDR64
regards
ronnie sahlberg
From 8f86903fc353d0906bd82e72ce19c5af09beb001 Mon Sep 17 00:00:00 2001
From: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Date: Mon, 5 Oct 2009 15:22:43 +1100
Subject: [PATCH] In the PIDL wireshark backend, we define external types in the conformance
file using the TYPE directive.
If we declare external types here, we must parse this file before we process
the IDL file, or else these external types will all default to 4byte padding
(pidl assumes all unknown types are 4byte aligned).
Make sure we read the conformance file and create these new types before
we parse the idl file.
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
|
|
This is the type used for a variable that is 32 bits for NDR32 and 64
bits for NDR64
|
|
NDR64 has a 'trailing gap' alignment, which aligns the end of a
structure on the overall structure alignment.
This explains the discrepancy we had with the RPC-SAMR test and NDR64
|
|
This fixes the problem with samr UserInfo16 when NDR64 is enabled
|
|
macro.
|
|
Caught by Guenther.
|
|
This allows for easier implementation of the NDR32/NDR64 split
|
|
Added support for NDR64 to the samba4 pidl generator
|
|
Jeremy.
|
|
to the C standard an enum is guarenteed to be an (int), which means
for 4 byte ints specifying a type of 0x80000000 is an invalid value.
The Solaris compiler complains about this. Fix by adding an (int)
cast in front of the value generation.
Jeremy.
|
|
|
|
metze
|
|
metze
|
|
metze
|
|
functions
metze
|
|
ParseFunction as glue
This prepares the following ParseFunctionAsync* commits.
metze
|
|
ParseOutputArgument()
metze
|
|
metze
|
|
This tries to fix commit 8eff9f9a3167eb0c2a4c00edf5a4cdbbc06c4dfd.
The problem was:
Undefined subroutine &Parse::Pidl::Samba4::Python::mapType called at /home/metze/master4/pidl/lib/Parse/Pidl/Samba4/Python.pm line 848.
Jelmer: please check this.
metze
|
|
This fixes some problems in the samr Python bindings that pidl was
(correctly) warning about.
|
|
Use py_talloc_reference in DCE/RPC code, fixes
access to SAMR pipe.
|
|
Original commit message in wireshark SVN (rev 28961):
====
From Kovarththanan Rajaratnam:
More "Cleanup header_field_info definitions"
====
Signed-off-by: Ronnie Sahlberg <ronniesahlberg@gmail.com>
Signed-off-by: Stefan Metzmacher <metze@samba.org>
|
|
metze
|
|
midl also supports this:
struct {
long l1;
[string] wchar_t str[16];
long l2;
};
Where the wire size of str is encoded like a length_is() header:
4-byte offset == 0;
4-byte array length;
The strings are zero terminated.
metze
|
|
'HASH(0x....)'.
|
|
than spewing pages of perl warnings.
|
|
Signed-off-by: Jelmer Vernooij <jelmer@samba.org>
|
|
It was creating new objects with bogus data.
This reverts commit 210830392999691469a55f4f5e2032abcdb20b8a.
|
|
For samba4 you need one of this:
samba/source4# make full_idl
samba/source4# make clean
samba# git clean -x -f
metze
|
|
metze
|
|
|
|
Code isn't generated to iterate over arrays of length 0, but the
variable declarations still are. The result is 'unused variable'
warnings. This only seems to be happening in one place right now, so I
targeted the fix to this case, but refactoring could be done to make
all variable declarations do this zero length check. Making it the
default would be a much more invasive fix.
Jelmer, please check!
|
|
(bug #6140)
|