diff options
| author | Günther Deschner <gd@samba.org> | 2010-03-05 21:56:50 +0100 |
|---|---|---|
| committer | Günther Deschner <gd@samba.org> | 2010-03-05 22:00:52 +0100 |
| commit | f258e98e177f0f75bab99654b9f32b10bb7ce37f (patch) | |
| tree | 3ba5c0ae0ec671ddd766378314bd73b8d7b1ff11 /librpc/idl | |
| parent | 47d366b5729b21e630760c36ef743dd5374a29fd (diff) | |
| download | samba-f258e98e177f0f75bab99654b9f32b10bb7ce37f.tar.gz samba-f258e98e177f0f75bab99654b9f32b10bb7ce37f.tar.bz2 samba-f258e98e177f0f75bab99654b9f32b10bb7ce37f.zip | |
winreg: fix winreg_QueryValue IDL.
Note that before this change pidl generated code that just dereferenced size_is
and length_is values from unique pointers without checking whether these
pointers were actually NULL.
With this change, pidl now throws a warning like:
warning: Got pointer for `data_size', expected fully derefenced variable
which is not correct, probably because pidl does not evaluate the C expression.
Guenther
Diffstat (limited to 'librpc/idl')
| -rw-r--r-- | librpc/idl/winreg.idl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/librpc/idl/winreg.idl b/librpc/idl/winreg.idl index f1f4dfb719..70206916e0 100644 --- a/librpc/idl/winreg.idl +++ b/librpc/idl/winreg.idl @@ -256,7 +256,7 @@ import "lsa.idl", "security.idl", "misc.idl"; [in,ref] policy_handle *handle, [in,ref] winreg_String *value_name, [in,out,unique] winreg_Type *type, - [in,out,unique,size_is(*data_size),length_is(*data_length)] uint8 *data, + [in,out,unique,size_is(data_size ? *data_size : 0),length_is(data_length ? *data_length : 0),range(0,0x4000000)] uint8 *data, [in,out,unique] uint32 *data_size, [in,out,unique] uint32 *data_length ); |