s3: Fix Coverity ID 2232, REVERSE_INULL
authorVolker Lendecke <vl@samba.org>
Wed, 16 Mar 2011 20:50:59 +0000 (21:50 +0100)
committerVolker Lendecke <vl@samba.org>
Wed, 16 Mar 2011 21:07:24 +0000 (22:07 +0100)
No point checking for !cli after dereferencing it

source3/rpc_client/cli_pipe.c

index ebbe849ac7bcf1414de602cbc6e3f2b4caa02f1a..0109bdae725432a426f5746fb19742a0e7b1ce55 100644 (file)
@@ -3256,7 +3256,7 @@ NTSTATUS cli_get_session_key(TALLOC_CTX *mem_ctx,
                             struct rpc_pipe_client *cli,
                             DATA_BLOB *session_key)
 {
-       struct pipe_auth_data *a = cli->auth;
+       struct pipe_auth_data *a;
        struct schannel_state *schannel_auth;
        struct auth_ntlmssp_state *ntlmssp_ctx;
        struct spnego_context *spnego_ctx;
@@ -3268,7 +3268,9 @@ NTSTATUS cli_get_session_key(TALLOC_CTX *mem_ctx,
                return NT_STATUS_INVALID_PARAMETER;
        }
 
-       if (!cli->auth) {
+       a = cli->auth;
+
+       if (a == NULL) {
                return NT_STATUS_INVALID_PARAMETER;
        }