mirror of
git://source.winehq.org/git/wine.git
synced 2024-09-14 21:55:28 +00:00
wldap32: Set error code in error cases (Coverity).
Signed-off-by: Fabian Maurer <dark.shadow4@web.de>
This commit is contained in:
parent
0408d3a800
commit
89f1d626dc
|
@ -218,13 +218,19 @@ WCHAR * CDECL ldap_first_attributeW( LDAP *ld, WLDAP32_LDAPMessage *entry, WLDAP
|
|||
if (ld && entry) retU = ldap_first_attribute( CTX(ld), MSG(entry), &berU );
|
||||
else return NULL;
|
||||
|
||||
if (retU && (ber = malloc( sizeof(*ber) )))
|
||||
if (!retU)
|
||||
return NULL;
|
||||
|
||||
if (!(ber = malloc( sizeof(*ber) )))
|
||||
{
|
||||
ld->ld_errno = WLDAP32_LDAP_NO_MEMORY;
|
||||
ldap_memfree( retU );
|
||||
return NULL;
|
||||
}
|
||||
|
||||
ber->opaque = (char *)berU;
|
||||
*ptr = ber;
|
||||
ret = strUtoW( retU );
|
||||
}
|
||||
|
||||
ldap_memfree( retU );
|
||||
return ret;
|
||||
}
|
||||
|
@ -358,12 +364,18 @@ WLDAP32_LDAPMessage * CDECL WLDAP32_ldap_next_entry( LDAP *ld, WLDAP32_LDAPMessa
|
|||
if (entry->lm_next) return entry->lm_next;
|
||||
|
||||
msgU = ldap_next_entry( CTX(ld), MSG(entry) );
|
||||
if (msgU && (msg = calloc( 1, sizeof(*msg) )))
|
||||
|
||||
if (!msgU)
|
||||
return NULL;
|
||||
|
||||
if (!(msg = calloc( 1, sizeof(*msg) )))
|
||||
{
|
||||
MSG(msg) = msgU;
|
||||
entry->lm_next = msg;
|
||||
ld->ld_errno = WLDAP32_LDAP_NO_MEMORY;
|
||||
return NULL;
|
||||
}
|
||||
|
||||
MSG(msg) = msgU;
|
||||
entry->lm_next = msg;
|
||||
return msg;
|
||||
}
|
||||
|
||||
|
@ -382,12 +394,18 @@ WLDAP32_LDAPMessage * CDECL WLDAP32_ldap_next_reference( LDAP *ld, WLDAP32_LDAPM
|
|||
if (entry->lm_next) return entry->lm_next;
|
||||
|
||||
msgU = ldap_next_reference( CTX(ld), MSG(entry) );
|
||||
if (msgU && (msg = calloc( 1, sizeof(*msg) )))
|
||||
|
||||
if (!msgU)
|
||||
return NULL;
|
||||
|
||||
if (!(msg = calloc( 1, sizeof(*msg) )))
|
||||
{
|
||||
MSG(msg) = msgU;
|
||||
entry->lm_next = msg;
|
||||
ld->ld_errno = WLDAP32_LDAP_NO_MEMORY;
|
||||
return NULL;
|
||||
}
|
||||
|
||||
MSG(msg) = msgU;
|
||||
entry->lm_next = msg;
|
||||
return msg;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue