From: simon <simon@cda61777-01e9-0310-a592-d414129be87e>
Date: Thu, 9 Sep 2010 14:35:16 +0000 (+0000)
Subject: Remove redundant check for NULL in sshfwd_close(). The thing we're
X-Git-Url: https://git.distorted.org.uk/~mdw/sgt/putty/commitdiff_plain/75778ef1d5ac3469168fe6dbadf1d9faba5985d0?ds=sidebyside

Remove redundant check for NULL in sshfwd_close(). The thing we're
testing against NULL has already been dereferenced by the time we
bother to test it, so it's a bit pointless - and in any case, no
null pointer can come to this function from any existing call site.


git-svn-id: svn://svn.tartarus.org/sgt/putty@8990 cda61777-01e9-0310-a592-d414129be87e
---

diff --git a/ssh.c b/ssh.c
index 43d642dd..401706c4 100644
--- a/ssh.c
+++ b/ssh.c
@@ -4164,7 +4164,7 @@ void sshfwd_close(struct ssh_channel *c)
     if (ssh->state == SSH_STATE_CLOSED)
 	return;
 
-    if (c && !c->closes) {
+    if (!c->closes) {
 	/*
 	 * If halfopen is true, we have sent
 	 * CHANNEL_OPEN for this channel, but it hasn't even been