summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeremy Allison <jra@samba.org>2006-03-09 23:20:52 +0000
committerGerald (Jerry) Carter <jerry@samba.org>2007-10-10 11:11:13 -0500
commitdc93156fe96c1861f3ad072496ad223513e915e0 (patch)
treefbf3261d53e931fab5bcf9f8ba72a00c83111158
parentacf0c6fb66b4182a0e5e7610d5063649ca237311 (diff)
downloadsamba-dc93156fe96c1861f3ad072496ad223513e915e0.tar.gz
samba-dc93156fe96c1861f3ad072496ad223513e915e0.tar.bz2
samba-dc93156fe96c1861f3ad072496ad223513e915e0.zip
r14121: We never pass NULL to the rpc_api_pipe fn so don't
trigger coverity checks by testing for NULL. Jeremy. (This used to be commit 6b4484159293d725613249adbfa01472dea1c722)
-rw-r--r--source3/rpc_client/cli_pipe.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/rpc_client/cli_pipe.c b/source3/rpc_client/cli_pipe.c
index d3db77b3cf..45b2e96bac 100644
--- a/source3/rpc_client/cli_pipe.c
+++ b/source3/rpc_client/cli_pipe.c
@@ -744,8 +744,8 @@ static NTSTATUS rpc_api_pipe(struct rpc_pipe_client *cli,
char *rparam = NULL;
uint32 rparam_len = 0;
uint16 setup[2];
- char *pdata = data ? prs_data_p(data) : NULL;
- uint32 data_len = data ? prs_offset(data) : 0;
+ char *pdata = prs_data_p(data);
+ uint32 data_len = prs_offset(data);
char *prdata = NULL;
uint32 rdata_len = 0;
uint32 max_data = cli->max_xmit_frag ? cli->max_xmit_frag : RPC_MAX_PDU_FRAG_LEN;