- djm@cvs.openbsd.org 2013/05/17 00:13:13

     [xmalloc.h cipher.c sftp-glob.c ssh-keyscan.c ssh.c sftp-common.c
     ssh-ecdsa.c auth2-chall.c compat.c readconf.c kexgexs.c monitor.c
     gss-genr.c cipher-3des1.c kex.c monitor_wrap.c ssh-pkcs11-client.c
     auth-options.c rsa.c auth2-pubkey.c sftp.c hostfile.c auth2.c
     servconf.c auth.c authfile.c xmalloc.c uuencode.c sftp-client.c
     auth2-gss.c sftp-server.c bufaux.c mac.c session.c jpake.c kexgexc.c
     sshconnect.c auth-chall.c auth2-passwd.c sshconnect1.c buffer.c
     kexecdhs.c kexdhs.c ssh-rsa.c auth1.c ssh-pkcs11.c auth2-kbdint.c
     kexdhc.c sshd.c umac.c ssh-dss.c auth2-jpake.c bufbn.c clientloop.c
     monitor_mm.c scp.c roaming_client.c serverloop.c key.c auth-rsa.c
     ssh-pkcs11-helper.c ssh-keysign.c ssh-keygen.c match.c channels.c
     sshconnect2.c addrmatch.c mux.c canohost.c kexecdhc.c schnorr.c
     ssh-add.c misc.c auth2-hostbased.c ssh-agent.c bufec.c groupaccess.c
     dns.c packet.c readpass.c authfd.c moduli.c]
     bye, bye xfree(); ok markus@

1 files changed, 12 insertions, 18 deletions

diff --git a/readconf.c b/readconf.c
index 30c1e83bd..45cf910fe 100644
--- a/readconf.c
+++ b/readconf.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: readconf.c,v 1.201 2013/05/16 10:43:34 dtucker Exp $ */
+/* $OpenBSD: readconf.c,v 1.202 2013/05/17 00:13:14 djm Exp $ */
 /*
  * Author: Tatu Ylonen <ylo@cs.hut.fi>
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
@@ -308,22 +308,20 @@ clear_forwardings(Options *options)
         int i;
 
         for (i = 0; i < options->num_local_forwards; i++) {
-                if (options->local_forwards[i].listen_host != NULL)
-                        xfree(options->local_forwards[i].listen_host);
-                xfree(options->local_forwards[i].connect_host);
+                free(options->local_forwards[i].listen_host);
+                free(options->local_forwards[i].connect_host);
         }
         if (options->num_local_forwards > 0) {
-                xfree(options->local_forwards);
+                free(options->local_forwards);
                 options->local_forwards = NULL;
         }
         options->num_local_forwards = 0;
         for (i = 0; i < options->num_remote_forwards; i++) {
-                if (options->remote_forwards[i].listen_host != NULL)
-                        xfree(options->remote_forwards[i].listen_host);
-                xfree(options->remote_forwards[i].connect_host);
+                free(options->remote_forwards[i].listen_host);
+                free(options->remote_forwards[i].connect_host);
         }
         if (options->num_remote_forwards > 0) {
-                xfree(options->remote_forwards);
+                free(options->remote_forwards);
                 options->remote_forwards = NULL;
         }
         options->num_remote_forwards = 0;
@@ -1456,7 +1454,7 @@ parse_forward(Forward *fwd, const char *fwdspec, int dynamicfwd, int remotefwd)
                 i = 0; /* failure */
         }
 
-        xfree(p);
+        free(p);
 
         if (dynamicfwd) {
                 if (!(i == 1 || i == 2))
@@ -1482,13 +1480,9 @@ parse_forward(Forward *fwd, const char *fwdspec, int dynamicfwd, int remotefwd)
         return (i);
 
  fail_free:
-        if (fwd->connect_host != NULL) {
-                xfree(fwd->connect_host);
-                fwd->connect_host = NULL;
-        }
-        if (fwd->listen_host != NULL) {
-                xfree(fwd->listen_host);
-                fwd->listen_host = NULL;
-        }
+        free(fwd->connect_host);
+        fwd->connect_host = NULL;
+        free(fwd->listen_host);
+        fwd->listen_host = NULL;
         return (0);
 }