diff options
| author | Jim McDonough <jmcd@samba.org> | 2006-03-15 02:31:11 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:15:29 -0500 |
| commit | cb370cc28ce361628df137c9aef02739aca062db (patch) | |
| tree | 3f9129c09480eece599ff09c5d830f477be3a6e5 /source/libmsrpc | |
| parent | d241f74e06eac7b61e5b7e09c2b9a955ec560fec (diff) | |
| download | samba-cb370cc28ce361628df137c9aef02739aca062db.tar.gz samba-cb370cc28ce361628df137c9aef02739aca062db.tar.xz samba-cb370cc28ce361628df137c9aef02739aca062db.zip | |
r14408: More on fix for coverity #36. The previous fix would cause us to
marshall a buffer based on an unknown size. Zero out the sec_desc
buffer to prevent this. This is still not getting proper results for
a registry security descriptor (everything gets ACCESS DENIED), but
at least we aren't blowing out memory now...
Diffstat (limited to 'source/libmsrpc')
| -rw-r--r-- | source/libmsrpc/cac_winreg.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/source/libmsrpc/cac_winreg.c b/source/libmsrpc/cac_winreg.c index aa847602cc3..acd83f40760 100644 --- a/source/libmsrpc/cac_winreg.c +++ b/source/libmsrpc/cac_winreg.c @@ -831,6 +831,8 @@ int cac_RegGetKeySecurity(CacServerHandle *hnd, TALLOC_CTX *mem_ctx, struct RegG uint32 buf_size; SEC_DESC_BUF buf; + ZERO_STRUCT(buf); + if(!hnd) return CAC_FAILURE; |