- 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, 38 insertions, 54 deletions

diff --git a/session.c b/session.c
index 4c4461ded..f47cc1788 100644
--- a/session.c
+++ b/session.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: session.c,v 1.264 2013/04/19 01:03:01 djm Exp $ */
+/* $OpenBSD: session.c,v 1.265 2013/05/17 00:13:14 djm Exp $ */
 /*
  * Copyright (c) 1995 Tatu Ylonen <ylo@cs.hut.fi>, Espoo, Finland
  *                    All rights reserved
@@ -199,7 +199,7 @@ auth_input_request_forwarding(struct passwd * pw)
                 packet_send_debug("Agent forwarding disabled: "
                     "mkdtemp() failed: %.100s", strerror(errno));
                 restore_uid();
-                xfree(auth_sock_dir);
+                free(auth_sock_dir);
                 auth_sock_dir = NULL;
                 goto authsock_err;
         }
@@ -244,11 +244,10 @@ auth_input_request_forwarding(struct passwd * pw)
         return 1;
 
  authsock_err:
-        if (auth_sock_name != NULL)
-                xfree(auth_sock_name);
+        free(auth_sock_name);
         if (auth_sock_dir != NULL) {
                 rmdir(auth_sock_dir);
-                xfree(auth_sock_dir);
+                free(auth_sock_dir);
         }
         if (sock != -1)
                 close(sock);
@@ -364,8 +363,8 @@ do_authenticated1(Authctxt *authctxt)
                         packet_check_eom();
                         success = session_setup_x11fwd(s);
                         if (!success) {
-                                xfree(s->auth_proto);
-                                xfree(s->auth_data);
+                                free(s->auth_proto);
+                                free(s->auth_data);
                                 s->auth_proto = NULL;
                                 s->auth_data = NULL;
                         }
@@ -412,7 +411,7 @@ do_authenticated1(Authctxt *authctxt)
                                 if (do_exec(s, command) != 0)
                                         packet_disconnect(
                                             "command execution failed");
-                                xfree(command);
+                                free(command);
                         } else {
                                 if (do_exec(s, NULL) != 0)
                                         packet_disconnect(
@@ -977,7 +976,7 @@ child_set_env(char ***envp, u_int *envsizep, const char *name,
                         break;
         if (env[i]) {
                 /* Reuse the slot. */
-                xfree(env[i]);
+                free(env[i]);
         } else {
                 /* New variable.  Expand if necessary. */
                 envsize = *envsizep;
@@ -1219,8 +1218,8 @@ do_setup_env(Session *s, const char *shell)
                                 child_set_env(&env, &envsize, str, str + i + 1);
                         }
                         custom_environment = ce->next;
-                        xfree(ce->s);
-                        xfree(ce);
+                        free(ce->s);
+                        free(ce);
                 }
         }
 
@@ -1232,7 +1231,7 @@ do_setup_env(Session *s, const char *shell)
         laddr = get_local_ipaddr(packet_get_connection_in());
         snprintf(buf, sizeof buf, "%.50s %d %.50s %d",
             get_remote_ipaddr(), get_remote_port(), laddr, get_local_port());
-        xfree(laddr);
+        free(laddr);
         child_set_env(&env, &envsize, "SSH_CONNECTION", buf);
 
         if (s->ttyfd != -1)
@@ -1403,7 +1402,7 @@ do_nologin(struct passwd *pw)
 #endif
         if (stat(nl, &sb) == -1) {
                 if (nl != def_nl)
-                        xfree(nl);
+                        free(nl);
                 return;
         }
 
@@ -2054,7 +2053,7 @@ session_pty_req(Session *s)
         s->ypixel = packet_get_int();
 
         if (strcmp(s->term, "") == 0) {
-                xfree(s->term);
+                free(s->term);
                 s->term = NULL;
         }
 
@@ -2062,8 +2061,7 @@ session_pty_req(Session *s)
         debug("Allocating pty.");
         if (!PRIVSEP(pty_allocate(&s->ptyfd, &s->ttyfd, s->tty,
             sizeof(s->tty)))) {
-                if (s->term)
-                        xfree(s->term);
+                free(s->term);
                 s->term = NULL;
                 s->ptyfd = -1;
                 s->ttyfd = -1;
@@ -2124,7 +2122,7 @@ session_subsystem_req(Session *s)
                 logit("subsystem request for %.100s failed, subsystem not found",
                     subsys);
 
-        xfree(subsys);
+        free(subsys);
         return success;
 }
 
@@ -2146,8 +2144,8 @@ session_x11_req(Session *s)
 
         success = session_setup_x11fwd(s);
         if (!success) {
-                xfree(s->auth_proto);
-                xfree(s->auth_data);
+                free(s->auth_proto);
+                free(s->auth_data);
                 s->auth_proto = NULL;
                 s->auth_data = NULL;
         }
@@ -2169,7 +2167,7 @@ session_exec_req(Session *s)
         char *command = packet_get_string(&len);
         packet_check_eom();
         success = do_exec(s, command) == 0;
-        xfree(command);
+        free(command);
         return success;
 }
 
@@ -2215,8 +2213,8 @@ session_env_req(Session *s)
         debug2("Ignoring env request %s: disallowed name", name);
 
  fail:
-        xfree(name);
-        xfree(val);
+        free(name);
+        free(val);
         return (0);
 }
 
@@ -2398,24 +2396,16 @@ session_close_single_x11(int id, void *arg)
                 if (s->x11_chanids[i] != id)
                         session_close_x11(s->x11_chanids[i]);
         }
-        xfree(s->x11_chanids);
+        free(s->x11_chanids);
         s->x11_chanids = NULL;
-        if (s->display) {
-                xfree(s->display);
-                s->display = NULL;
-        }
-        if (s->auth_proto) {
-                xfree(s->auth_proto);
-                s->auth_proto = NULL;
-        }
-        if (s->auth_data) {
-                xfree(s->auth_data);
-                s->auth_data = NULL;
-        }
-        if (s->auth_display) {
-                xfree(s->auth_display);
-                s->auth_display = NULL;
-        }
+        free(s->display);
+        s->display = NULL;
+        free(s->auth_proto);
+        s->auth_proto = NULL;
+        free(s->auth_data);
+        s->auth_data = NULL;
+        free(s->auth_display);
+        s->auth_display = NULL;
 }
 
 static void
@@ -2477,24 +2467,18 @@ session_close(Session *s)
         debug("session_close: session %d pid %ld", s->self, (long)s->pid);
         if (s->ttyfd != -1)
                 session_pty_cleanup(s);
-        if (s->term)
-                xfree(s->term);
-        if (s->display)
-                xfree(s->display);
-        if (s->x11_chanids)
-                xfree(s->x11_chanids);
-        if (s->auth_display)
-                xfree(s->auth_display);
-        if (s->auth_data)
-                xfree(s->auth_data);
-        if (s->auth_proto)
-                xfree(s->auth_proto);
+        free(s->term);
+        free(s->display);
+        free(s->x11_chanids);
+        free(s->auth_display);
+        free(s->auth_data);
+        free(s->auth_proto);
         if (s->env != NULL) {
                 for (i = 0; i < s->num_env; i++) {
-                        xfree(s->env[i].name);
-                        xfree(s->env[i].val);
+                        free(s->env[i].name);
+                        free(s->env[i].val);
                 }
-                xfree(s->env);
+                free(s->env);
         }
         session_proctitle(s);
         session_unused(s->self);