- 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, 14 deletions

diff --git a/auth2-pubkey.c b/auth2-pubkey.c
index c28bef7a2..4c326df7a 100644
--- a/auth2-pubkey.c
+++ b/auth2-pubkey.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: auth2-pubkey.c,v 1.35 2013/03/07 00:19:59 djm Exp $ */
+/* $OpenBSD: auth2-pubkey.c,v 1.36 2013/05/17 00:13:13 djm Exp $ */
 /*
  * Copyright (c) 2000 Markus Friedl.  All rights reserved.
  *
@@ -154,7 +154,7 @@ userauth_pubkey(Authctxt *authctxt)
                     buffer_len(&b))) == 1)
                         authenticated = 1;
                 buffer_free(&b);
-                xfree(sig);
+                free(sig);
         } else {
                 debug("test whether pkalg/pkblob are acceptable");
                 packet_check_eom();
@@ -182,8 +182,8 @@ done:
         debug2("userauth_pubkey: authenticated %d pkalg %s", authenticated, pkalg);
         if (key != NULL)
                 key_free(key);
-        xfree(pkalg);
-        xfree(pkblob);
+        free(pkalg);
+        free(pkblob);
         return authenticated;
 }
 
@@ -200,7 +200,7 @@ match_principals_option(const char *principal_list, struct KeyCert *cert)
                     principal_list, NULL)) != NULL) {
                         debug3("matched principal from key options \"%.100s\"",
                             result);
-                        xfree(result);
+                        free(result);
                         return 1;
                 }
         }
@@ -336,7 +336,7 @@ check_authkeys_file(FILE *f, char *file, Key* key, struct passwd *pw)
                                 reason = "Certificate does not contain an "
                                     "authorized principal";
  fail_reason:
-                                xfree(fp);
+                                free(fp);
                                 error("%s", reason);
                                 auth_debug_add("%s", reason);
                                 continue;
@@ -346,13 +346,13 @@ check_authkeys_file(FILE *f, char *file, Key* key, struct passwd *pw)
                             &reason) != 0)
                                 goto fail_reason;
                         if (auth_cert_options(key, pw) != 0) {
-                                xfree(fp);
+                                free(fp);
                                 continue;
                         }
                         verbose("Accepted certificate ID \"%s\" "
                             "signed by %s CA %s via %s", key->cert->key_id,
                             key_type(found), fp, file);
-                        xfree(fp);
+                        free(fp);
                         found_key = 1;
                         break;
                 } else if (key_equal(found, key)) {
@@ -367,7 +367,7 @@ check_authkeys_file(FILE *f, char *file, Key* key, struct passwd *pw)
                         fp = key_fingerprint(found, SSH_FP_MD5, SSH_FP_HEX);
                         verbose("Found matching %s key: %s",
                             key_type(found), fp);
-                        xfree(fp);
+                        free(fp);
                         break;
                 }
         }
@@ -425,10 +425,8 @@ user_cert_trusted_ca(struct passwd *pw, Key *key)
         ret = 1;
 
  out:
-        if (principals_file != NULL)
-                xfree(principals_file);
-        if (ca_fp != NULL)
-                xfree(ca_fp);
+        free(principals_file);
+        free(ca_fp);
         return ret;
 }
 
@@ -633,7 +631,7 @@ user_key_allowed(struct passwd *pw, Key *key)
                     options.authorized_keys_files[i], pw);
 
                 success = user_key_allowed2(pw, key, file);
-                xfree(file);
+                free(file);
         }
 
         return success;