From 3fb8eebfe304f715bb63aeefa60b3b76e30288a8 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Tue, 17 Aug 2021 16:52:24 +0200 Subject: [PATCH] rpc_client: Slightly simplify rpc_transport_np_init_pipe_open() Avoid an unnecessary else, use tevent_req_nterror() in if-clause Signed-off-by: Volker Lendecke Reviewed-by: Jeremy Allison --- source3/rpc_client/rpc_transport_np.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/source3/rpc_client/rpc_transport_np.c b/source3/rpc_client/rpc_transport_np.c index b24023661ff..27e38235ca0 100644 --- a/source3/rpc_client/rpc_transport_np.c +++ b/source3/rpc_client/rpc_transport_np.c @@ -146,16 +146,16 @@ static void rpc_transport_np_init_pipe_open(struct tevent_req *subreq) "tevent_timer")); } return; - } else if (!NT_STATUS_IS_OK(status)) { - tevent_req_nterror(req, status); + } + + if (tevent_req_nterror(req, status)) { return; } status = rpc_transport_tstream_init(state, &stream, &state->transport); - if (!NT_STATUS_IS_OK(status)) { - tevent_req_nterror(req, status); + if (tevent_req_nterror(req, status)) { return; }