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

diff --git a/ssh-pkcs11.c b/ssh-pkcs11.c
index 1f4c1c8e4..a17326baa 100644
--- a/ssh-pkcs11.c
+++ b/ssh-pkcs11.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: ssh-pkcs11.c,v 1.6 2010/06/08 21:32:19 markus Exp $ */
+/* $OpenBSD: ssh-pkcs11.c,v 1.7 2013/05/17 00:13:14 djm Exp $ */
 /*
  * Copyright (c) 2010 Markus Friedl.  All rights reserved.
  *
@@ -120,9 +120,9 @@ pkcs11_provider_unref(struct pkcs11_provider *p)
         if (--p->refcount <= 0) {
                 if (p->valid)
                         error("pkcs11_provider_unref: %p still valid", p);
-                xfree(p->slotlist);
-                xfree(p->slotinfo);
-                xfree(p);
+                free(p->slotlist);
+                free(p->slotinfo);
+                free(p);
         }
 }
 
@@ -180,9 +180,8 @@ pkcs11_rsa_finish(RSA *rsa)
                         rv = k11->orig_finish(rsa);
                 if (k11->provider)
                         pkcs11_provider_unref(k11->provider);
-                if (k11->keyid)
-                        xfree(k11->keyid);
-                xfree(k11);
+                free(k11->keyid);
+                free(k11);
         }
         return (rv);
 }
@@ -266,11 +265,11 @@ pkcs11_rsa_private_encrypt(int flen, const u_char *from, u_char *to, RSA *rsa,
                         return (-1);    /* bail out */
                 if ((rv = f->C_Login(si->session, CKU_USER, pin, strlen(pin)))
                     != CKR_OK) {
-                        xfree(pin);
+                        free(pin);
                         error("C_Login failed: %lu", rv);
                         return (-1);
                 }
-                xfree(pin);
+                free(pin);
                 si->logged_in = 1;
         }
         key_filter[1].pValue = k11->keyid;
@@ -470,7 +469,7 @@ pkcs11_fetch_keys(struct pkcs11_provider *p, CK_ULONG slotidx, Key ***keysp,
                         }
                 }
                 for (i = 0; i < 3; i++)
-                        xfree(attribs[i].pValue);
+                        free(attribs[i].pValue);
         }
         if ((rv = f->C_FindObjectsFinal(session)) != CKR_OK)
                 error("C_FindObjectsFinal failed: %lu", rv);
@@ -579,11 +578,9 @@ fail:
         if (need_finalize && (rv = f->C_Finalize(NULL)) != CKR_OK)
                 error("C_Finalize failed: %lu", rv);
         if (p) {
-                if (p->slotlist)
-                        xfree(p->slotlist);
-                if (p->slotinfo)
-                        xfree(p->slotinfo);
-                xfree(p);
+                free(p->slotlist);
+                free(p->slotinfo);
+                free(p);
         }
         if (handle)
                 dlclose(handle);