Following the current coding guidelines, it is considered bad practice to return from
within a macro and change control flow as they look like normal function calls.
Change-Id: I27e7597f4078f0a6c96b5dadd12454fffe33e6b2
Signed-off-by: Garming Sam <garming@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Andreas Schneider <asn@samba.org>
* send and we need to return directly
*/
status = extended_ops[i].fn(call, reply, &error_str);
- NT_STATUS_IS_OK_RETURN(status);
+ if (NT_STATUS_IS_OK(status)) {
+ return status;
+ }
if (NT_STATUS_IS_LDAP(status)) {
result = NT_STATUS_LDAP_CODE(status);