diff options
author | Günther Deschner <gd@samba.org> | 2010-03-05 21:56:50 +0100 |
---|---|---|
committer | Karolin Seeger <kseeger@samba.org> | 2010-03-29 09:41:04 +0200 |
commit | 2dd35c3f47a8cbe01495646c4fac4b1726974a5d (patch) | |
tree | 3e0163a337f011edd93626e455a450ce0b310cb3 /librpc | |
parent | dd8b43413dcc12f53940760e7b176997c065dabc (diff) | |
download | samba-2dd35c3f47a8cbe01495646c4fac4b1726974a5d.tar.gz |
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
(cherry picked from commit f258e98e177f0f75bab99654b9f32b10bb7ce37f)
(cherry picked from commit f9a52643a5665efd5b11733179e574e3d0282e8b)
Diffstat (limited to 'librpc')
-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 f1f4dfb719e..70206916e02 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 ); |