From f8e7894e15123334f676b584d38c96f8f437e414 Mon Sep 17 00:00:00 2001
From: Ben Harris <bjh21@bjh21.me.uk>
Date: Wed, 3 Oct 2007 21:06:00 +0000
Subject: [PATCH] snew() always returns non-NULL, so checking if its return
 value is NULL is pointless.

[originally from svn r7758]
---
 ssh.c | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/ssh.c b/ssh.c
index 92d96f1b..63aecbc2 100644
--- a/ssh.c
+++ b/ssh.c
@@ -9228,15 +9228,13 @@ void *new_sock_channel(void *handle, Socket s)
     Ssh ssh = (Ssh) handle;
     struct ssh_channel *c;
     c = snew(struct ssh_channel);
-    c->ssh = ssh;
 
-    if (c) {
-	ssh2_channel_init(c);
-	c->halfopen = TRUE;
-	c->type = CHAN_SOCKDATA_DORMANT;/* identify channel type */
-	c->u.pfd.s = s;
-	add234(ssh->channels, c);
-    }
+    c->ssh = ssh;
+    ssh2_channel_init(c);
+    c->halfopen = TRUE;
+    c->type = CHAN_SOCKDATA_DORMANT;/* identify channel type */
+    c->u.pfd.s = s;
+    add234(ssh->channels, c);
     return c;
 }