diff options
| author | Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de> | 2010-03-21 15:12:37 +0100 |
|---|---|---|
| committer | Matthias Dieter Wallnöfer <mwallnoefer@yahoo.de> | 2010-03-21 15:12:37 +0100 |
| commit | 2bbdb2cd34233b0836f1aa7365e02c85494af4fa (patch) | |
| tree | 5963fe02642c1c40444b0b1dc0185493a9f82e77 | |
| parent | 0f2cf82e5e52da6fc71742df7b13c9f372bcf113 (diff) | |
| download | samba-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.c | 5 |
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; |