Remove redundant check for NULL in sshfwd_close(). The thing we're
authorsimon <simon@cda61777-01e9-0310-a592-d414129be87e>
Thu, 9 Sep 2010 14:35:16 +0000 (14:35 +0000)
committersimon <simon@cda61777-01e9-0310-a592-d414129be87e>
Thu, 9 Sep 2010 14:35:16 +0000 (14:35 +0000)
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

ssh.c

diff --git a/ssh.c b/ssh.c
index 43d642d..401706c 100644 (file)
--- 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