mirror of
https://git.openldap.org/openldap/openldap.git
synced 2025-01-06 10:46:21 +08:00
Cleanup NULL derefs
This commit is contained in:
parent
aa87b9ad2e
commit
2d4da0b23c
@ -65,8 +65,9 @@ ldap_create_page_control_value(
|
|||||||
if ( ld == NULL || value == NULL ||
|
if ( ld == NULL || value == NULL ||
|
||||||
pagesize < 1 || pagesize > LDAP_MAXINT )
|
pagesize < 1 || pagesize > LDAP_MAXINT )
|
||||||
{
|
{
|
||||||
ld->ld_errno = LDAP_PARAM_ERROR;
|
if ( ld )
|
||||||
return ld->ld_errno;
|
ld->ld_errno = LDAP_PARAM_ERROR;
|
||||||
|
return LDAP_PARAM_ERROR;
|
||||||
}
|
}
|
||||||
|
|
||||||
assert( LDAP_VALID( ld ) );
|
assert( LDAP_VALID( ld ) );
|
||||||
@ -195,8 +196,9 @@ ldap_parse_pageresponse_control(
|
|||||||
ber_int_t count;
|
ber_int_t count;
|
||||||
|
|
||||||
if ( ld == NULL || ctrl == NULL || cookie == NULL ) {
|
if ( ld == NULL || ctrl == NULL || cookie == NULL ) {
|
||||||
ld->ld_errno = LDAP_PARAM_ERROR;
|
if ( ld )
|
||||||
return ld->ld_errno;
|
ld->ld_errno = LDAP_PARAM_ERROR;
|
||||||
|
return LDAP_PARAM_ERROR;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* Create a BerElement from the berval returned in the control. */
|
/* Create a BerElement from the berval returned in the control. */
|
||||||
|
@ -297,8 +297,9 @@ ldap_create_sort_control_value(
|
|||||||
ber_tag_t tag;
|
ber_tag_t tag;
|
||||||
|
|
||||||
if ( ld == NULL || keyList == NULL || value == NULL ) {
|
if ( ld == NULL || keyList == NULL || value == NULL ) {
|
||||||
ld->ld_errno = LDAP_PARAM_ERROR;
|
if ( ld )
|
||||||
return ld->ld_errno;
|
ld->ld_errno = LDAP_PARAM_ERROR;
|
||||||
|
return LDAP_PARAM_ERROR;
|
||||||
}
|
}
|
||||||
|
|
||||||
assert( LDAP_VALID( ld ) );
|
assert( LDAP_VALID( ld ) );
|
||||||
@ -496,8 +497,7 @@ ldap_parse_sortresponse_control(
|
|||||||
ber_len_t berLen;
|
ber_len_t berLen;
|
||||||
|
|
||||||
if (ld == NULL) {
|
if (ld == NULL) {
|
||||||
ld->ld_errno = LDAP_PARAM_ERROR;
|
return LDAP_PARAM_ERROR;
|
||||||
return(ld->ld_errno);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
if (ctrl == NULL) {
|
if (ctrl == NULL) {
|
||||||
|
@ -95,8 +95,9 @@ ldap_create_vlv_control_value(
|
|||||||
BerElement *ber;
|
BerElement *ber;
|
||||||
|
|
||||||
if ( ld == NULL || vlvinfop == NULL || value == NULL ) {
|
if ( ld == NULL || vlvinfop == NULL || value == NULL ) {
|
||||||
ld->ld_errno = LDAP_PARAM_ERROR;
|
if ( ld )
|
||||||
return ld->ld_errno;
|
ld->ld_errno = LDAP_PARAM_ERROR;
|
||||||
|
return LDAP_PARAM_ERROR;
|
||||||
}
|
}
|
||||||
|
|
||||||
assert( LDAP_VALID( ld ) );
|
assert( LDAP_VALID( ld ) );
|
||||||
|
Loading…
Reference in New Issue
Block a user