Commit a9c62419 added a switch() in a for() loop, which changes the semantics of "break;".
Signed-off-by: Chuck Lever <[email protected]> --- src/libnsdb/fileserver.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/libnsdb/fileserver.c b/src/libnsdb/fileserver.c index d3f7560..9c80521 100644 --- a/src/libnsdb/fileserver.c +++ b/src/libnsdb/fileserver.c @@ -1757,17 +1757,18 @@ nsdb_ping_contexts_s(nsdb_t host, char **contexts, unsigned int *ldap_err) switch (retval) { case FEDFS_OK: free(dn); - break; + goto out; case FEDFS_ERR_NSDB_LDAP_VAL: if (ldap_result == LDAP_CONFIDENTIALITY_REQUIRED) retval = FEDFS_ERR_NSDB_AUTH; else *ldap_err = ldap_result; - break; + goto out; default: retval = FEDFS_ERR_NSDB_NONCE; } } +out: return retval; } _______________________________________________ fedfs-utils-devel mailing list [email protected] https://oss.oracle.com/mailman/listinfo/fedfs-utils-devel
