summaryrefslogtreecommitdiffstats
path: root/librpc
diff options
context:
space:
mode:
authorGünther Deschner <gd@samba.org>2010-03-05 21:56:50 +0100
committerGünther Deschner <gd@samba.org>2010-03-05 22:00:52 +0100
commitf258e98e177f0f75bab99654b9f32b10bb7ce37f (patch)
tree3ba5c0ae0ec671ddd766378314bd73b8d7b1ff11 /librpc
parent47d366b5729b21e630760c36ef743dd5374a29fd (diff)
downloadsamba-f258e98e177f0f75bab99654b9f32b10bb7ce37f.tar.gz
samba-f258e98e177f0f75bab99654b9f32b10bb7ce37f.tar.xz
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')
-rw-r--r--librpc/idl/winreg.idl2
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
);