diff options
author | Volker Lendecke <vl@samba.org> | 2021-01-22 20:01:27 +0100 |
---|---|---|
committer | Volker Lendecke <vl@samba.org> | 2021-01-28 16:58:35 +0000 |
commit | fcf60f155a24bea97395b7ee0200543440782a74 (patch) | |
tree | 7c1a7b5ad86aea78b118dae27107e2c3a658bf65 /librpc | |
parent | eed1257268e96123d9655527071c00d0ab38f803 (diff) | |
download | samba-fcf60f155a24bea97395b7ee0200543440782a74.tar.gz |
librpc: Simplify dcerpc_binding_set_abstract_syntax()
It might be a question of style, but I find it simpler this way.
Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Samuel Cabrero <scabrero@samba.org>
Diffstat (limited to 'librpc')
-rw-r--r-- | librpc/rpc/binding.c | 12 |
1 files changed, 2 insertions, 10 deletions
diff --git a/librpc/rpc/binding.c b/librpc/rpc/binding.c index 57ceed012f4..d4ca43e8066 100644 --- a/librpc/rpc/binding.c +++ b/librpc/rpc/binding.c @@ -652,20 +652,12 @@ _PUBLIC_ NTSTATUS dcerpc_binding_set_abstract_syntax(struct dcerpc_binding *b, if (syntax == NULL) { status = dcerpc_binding_set_string_option(b, "abstract_syntax", NULL); - if (!NT_STATUS_IS_OK(status)) { - return status; - } - - return NT_STATUS_OK; + return status; } if (ndr_syntax_id_equal(&ndr_syntax_id_null, syntax)) { status = dcerpc_binding_set_string_option(b, "abstract_syntax", NULL); - if (!NT_STATUS_IS_OK(status)) { - return status; - } - - return NT_STATUS_OK; + return status; } status = dcerpc_binding_set_string_option( |