From 8486f0fa0495e51d204dca970bf864041123b631 Mon Sep 17 00:00:00 2001 From: Volker Lendecke Date: Wed, 30 Mar 2011 06:57:05 +0200 Subject: s3: Fix Coverity ID 2236: REVERSE_INULL We have already dereferenced "b", and the other calls in this loop can only have worked with a valid handle --- source3/smbd/lanman.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/source3/smbd/lanman.c b/source3/smbd/lanman.c index bf2987f907..a04f062274 100644 --- a/source3/smbd/lanman.c +++ b/source3/smbd/lanman.c @@ -1148,9 +1148,7 @@ static bool api_DosPrintQEnum(struct smbd_server_connection *sconn, subcntarr[i] = num_jobs; subcnt += subcntarr[i]; - if (b && is_valid_policy_hnd(&handle)) { - dcerpc_spoolss_ClosePrinter(b, mem_ctx, &handle, &werr); - } + dcerpc_spoolss_ClosePrinter(b, mem_ctx, &handle, &werr); } if (init_package(&desc,queuecnt,subcnt)) { -- cgit