diff options
author | Volker Lendecke <vlendec@samba.org> | 2006-08-25 18:24:43 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:38:52 -0500 |
commit | 9230c659f66df3da9fda5e7cae1717f19c9c08c7 (patch) | |
tree | f238a3aa901ba6fd7cfcfc7fee87077d10966f58 /source3/libaddns | |
parent | 6365dcef4d753b11a33ae4311a09a4c475f566aa (diff) | |
download | samba-9230c659f66df3da9fda5e7cae1717f19c9c08c7.tar.gz samba-9230c659f66df3da9fda5e7cae1717f19c9c08c7.tar.bz2 samba-9230c659f66df3da9fda5e7cae1717f19c9c08c7.zip |
r17835: Fix Coverity bugs 306, 309, 310.
Jeremy, you might want to look at the trans2 one.
Volker
(This used to be commit d727fc681c073a1b09accd31a07341b58e10eebb)
Diffstat (limited to 'source3/libaddns')
-rw-r--r-- | source3/libaddns/dnssock.c | 2 | ||||
-rw-r--r-- | source3/libaddns/dnsupdate.c | 5 |
2 files changed, 1 insertions, 6 deletions
diff --git a/source3/libaddns/dnssock.c b/source3/libaddns/dnssock.c index 3d0991a1aa..d3223b9bd8 100644 --- a/source3/libaddns/dnssock.c +++ b/source3/libaddns/dnssock.c @@ -732,7 +732,7 @@ void DNSFreeSendBufferContext( HANDLE hSendBuffer ) pSendBufferContext = ( DNS_SENDBUFFER_CONTEXT * ) hSendBuffer; - if ( pSendBufferContext->pSendBuffer ) { + if ( pSendBufferContext && pSendBufferContext->pSendBuffer ) { DNSFreeMemory( pSendBufferContext->pSendBuffer ); } if ( pSendBufferContext ) { diff --git a/source3/libaddns/dnsupdate.c b/source3/libaddns/dnsupdate.c index 36c5cbb13a..503316d305 100644 --- a/source3/libaddns/dnsupdate.c +++ b/source3/libaddns/dnsupdate.c @@ -431,7 +431,6 @@ int32 DNSUpdateGenerateSignature( gss_ctx_id_t * pGSSContext, { int32 dwError = 0; int32 dwMinorStatus = 0; - HANDLE hSendBuffer = ( HANDLE ) NULL; uint8 *pMessageBuffer = NULL; int32 dwMessageSize = 0; int32 dwMaxSignatureSize = 0; @@ -477,10 +476,6 @@ int32 DNSUpdateGenerateSignature( gss_ctx_id_t * pGSSContext, error: - if ( hSendBuffer ) { - DNSFreeSendBufferContext( hSendBuffer ); - } - if ( pMessageBuffer ) { DNSFreeMemory( pMessageBuffer ); } |