summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthias Dieter Wallnöfer <mwallnoefer@yahoo.de>2010-03-21 15:12:37 +0100
committerMatthias Dieter Wallnöfer <mwallnoefer@yahoo.de>2010-03-21 15:12:37 +0100
commit2bbdb2cd34233b0836f1aa7365e02c85494af4fa (patch)
tree5963fe02642c1c40444b0b1dc0185493a9f82e77
parent0f2cf82e5e52da6fc71742df7b13c9f372bcf113 (diff)
downloadsamba-2bbdb2cd34233b0836f1aa7365e02c85494af4fa.tar.gz
samba-2bbdb2cd34233b0836f1aa7365e02c85494af4fa.tar.bz2
samba-2bbdb2cd34233b0836f1aa7365e02c85494af4fa.zip
s4:WINREG RPC server - don't check for the "name" size in "EnumValue"
This isn't needed at all since: 1.) a new name object is created and sent back to the client 2.) the "size" seems to be the size of the "name" pointer. On my test with the "regedt32" program this has always been "4".
-rw-r--r--source4/rpc_server/winreg/rpc_winreg.c5
1 files changed, 0 insertions, 5 deletions
diff --git a/source4/rpc_server/winreg/rpc_winreg.c b/source4/rpc_server/winreg/rpc_winreg.c
index 5bfc655277..506c72df44 100644
--- a/source4/rpc_server/winreg/rpc_winreg.c
+++ b/source4/rpc_server/winreg/rpc_winreg.c
@@ -300,11 +300,6 @@ static WERROR dcesrv_winreg_EnumValue(struct dcesrv_call_state *dce_call,
data.length = *r->in.length;
}
- /* check if there is enough room for the name */
- if (r->in.name->size < 2*strlen_m_term(data_name)) {
- return WERR_MORE_DATA;
- }
-
/* "data_name" is NULL when we query the default attribute */
if (data_name != NULL) {
r->out.name->name = data_name;