rcolebaugh / rpms / openssh

Forked from rpms/openssh 2 years ago
Clone

Blame SOURCES/pam_ssh_agent_auth-0.9.3-no-xfree.patch

f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/authfd.c	2013-10-30 17:14:26.013615342 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/authfd.c	2013-10-30 17:15:07.353327799 +0100
f09e2e
@@ -260,7 +260,7 @@ 
f09e2e
 {
f09e2e
 	buffer_free(&auth->identities);
f09e2e
 	close(auth->fd);
f09e2e
-	xfree(auth);
f09e2e
+	free(auth);
f09e2e
 }
f09e2e
 
f09e2e
 /* Lock/unlock agent */
f09e2e
@@ -379,7 +379,7 @@ 
f09e2e
 		blob = buffer_get_string(&auth->identities, &blen);
f09e2e
 		*comment = buffer_get_string(&auth->identities, NULL);
f09e2e
 		key = key_from_blob(blob, blen);
f09e2e
-		xfree(blob);
f09e2e
+		free(blob);
f09e2e
 		break;
f09e2e
 	default:
f09e2e
 		return NULL;
f09e2e
@@ -472,7 +472,7 @@ 
f09e2e
 	buffer_put_string(&msg, blob, blen);
f09e2e
 	buffer_put_string(&msg, data, datalen);
f09e2e
 	buffer_put_int(&msg, flags);
f09e2e
-	xfree(blob);
f09e2e
+	free(blob);
f09e2e
 
f09e2e
 	if (ssh_request_reply(auth, &msg, &msg) == 0) {
f09e2e
 		buffer_free(&msg;;
f09e2e
@@ -612,7 +612,7 @@ 
f09e2e
 		key_to_blob(key, &blob, &blen);
f09e2e
 		buffer_put_char(&msg, SSH2_AGENTC_REMOVE_IDENTITY);
f09e2e
 		buffer_put_string(&msg, blob, blen);
f09e2e
-		xfree(blob);
f09e2e
+		free(blob);
f09e2e
 	} else {
f09e2e
 		buffer_free(&msg;;
f09e2e
 		return 0;
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/bufaux.c	2013-10-30 17:14:26.014615310 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/bufaux.c	2013-10-30 17:15:07.354327768 +0100
f09e2e
@@ -176,7 +176,7 @@ 
f09e2e
 	/* Get the string. */
f09e2e
 	if (buffer_get_ret(buffer, value, len) == -1) {
f09e2e
 		logerror("buffer_get_string_ret: buffer_get failed");
f09e2e
-		xfree(value);
f09e2e
+		free(value);
f09e2e
 		return (NULL);
f09e2e
 	}
f09e2e
 	/* Append a null character to make processing easier. */
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/bufbn.c	2013-10-30 17:14:26.014615310 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/bufbn.c	2013-10-30 17:15:07.354327768 +0100
f09e2e
@@ -69,7 +69,7 @@ 
f09e2e
 	if (oi != bin_size) {
f09e2e
 		logerror("buffer_put_bignum_ret: BN_bn2bin() failed: oi %d != bin_size %d",
f09e2e
 		    oi, bin_size);
f09e2e
-		xfree(buf);
f09e2e
+		free(buf);
f09e2e
 		return (-1);
f09e2e
 	}
f09e2e
 
f09e2e
@@ -80,7 +80,7 @@ 
f09e2e
 	buffer_append(buffer, buf, oi);
f09e2e
 
f09e2e
 	memset(buf, 0, bin_size);
f09e2e
-	xfree(buf);
f09e2e
+	free(buf);
f09e2e
 
f09e2e
 	return (0);
f09e2e
 }
f09e2e
@@ -167,13 +167,13 @@ 
f09e2e
 	if (oi < 0 || (u_int)oi != bytes - 1) {
f09e2e
 		logerror("buffer_put_bignum2_ret: BN_bn2bin() failed: "
f09e2e
 		    "oi %d != bin_size %d", oi, bytes);
f09e2e
-		xfree(buf);
f09e2e
+		free(buf);
f09e2e
 		return (-1);
f09e2e
 	}
f09e2e
 	hasnohigh = (buf[1] & 0x80) ? 0 : 1;
f09e2e
 	buffer_put_string(buffer, buf+hasnohigh, bytes-hasnohigh);
f09e2e
 	memset(buf, 0, bytes);
f09e2e
-	xfree(buf);
f09e2e
+	free(buf);
f09e2e
 	return (0);
f09e2e
 }
f09e2e
 
f09e2e
@@ -197,21 +197,21 @@ 
f09e2e
 
f09e2e
 	if (len > 0 && (bin[0] & 0x80)) {
f09e2e
 		logerror("buffer_get_bignum2_ret: negative numbers not supported");
f09e2e
-		xfree(bin);
f09e2e
+		free(bin);
f09e2e
 		return (-1);
f09e2e
 	}
f09e2e
 	if (len > 8 * 1024) {
f09e2e
 		logerror("buffer_get_bignum2_ret: cannot handle BN of size %d",
f09e2e
 		    len);
f09e2e
-		xfree(bin);
f09e2e
+		free(bin);
f09e2e
 		return (-1);
f09e2e
 	}
f09e2e
 	if (BN_bin2bn(bin, len, value) == NULL) {
f09e2e
 		logerror("buffer_get_bignum2_ret: BN_bin2bn failed");
f09e2e
-		xfree(bin);
f09e2e
+		free(bin);
f09e2e
 		return (-1);
f09e2e
 	}
f09e2e
-	xfree(bin);
f09e2e
+	free(bin);
f09e2e
 	return (0);
f09e2e
 }
f09e2e
 
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/buffer.c	2013-10-30 17:14:26.014615310 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/buffer.c	2013-10-30 17:15:07.355327737 +0100
f09e2e
@@ -50,7 +50,7 @@ 
f09e2e
 	if (buffer->alloc > 0) {
f09e2e
 		memset(buffer->buf, 0, buffer->alloc);
f09e2e
 		buffer->alloc = 0;
f09e2e
-		xfree(buffer->buf);
f09e2e
+		free(buffer->buf);
f09e2e
 	}
f09e2e
 }
f09e2e
 
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/iterate_ssh_agent_keys.c	2013-10-30 17:14:26.031614782 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/iterate_ssh_agent_keys.c	2013-10-30 17:15:07.357327674 +0100
f09e2e
@@ -197,9 +197,9 @@ 
f09e2e
                 if(userauth_pubkey_from_id(id)) {
f09e2e
                     retval = 1;
f09e2e
                 }
f09e2e
-                xfree(id->filename);
f09e2e
+                free(id->filename);
f09e2e
                 key_free(id->key);
f09e2e
-                xfree(id);
f09e2e
+                free(id);
f09e2e
                 if(retval == 1)
f09e2e
                     break;
f09e2e
             }
f09e2e
@@ -209,7 +209,7 @@ 
f09e2e
     else {
f09e2e
         verbose("No ssh-agent could be contacted");
f09e2e
     }
f09e2e
-    xfree(session_id2);
f09e2e
+    free(session_id2);
f09e2e
     EVP_cleanup();
f09e2e
     return retval;
f09e2e
 }
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/key.c	2013-10-30 17:14:26.017615218 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/key.c	2013-10-30 17:15:07.358327643 +0100
f09e2e
@@ -154,7 +154,7 @@ 
f09e2e
 		fatal("key_free: bad key type %d", k->type);
f09e2e
 		break;
f09e2e
 	}
f09e2e
-	xfree(k);
f09e2e
+	free(k);
f09e2e
 }
f09e2e
 
f09e2e
 int
f09e2e
@@ -229,7 +229,7 @@ 
f09e2e
 		EVP_DigestUpdate(&ctx, blob, len);
f09e2e
 		EVP_DigestFinal(&ctx, retval, dgst_raw_length);
f09e2e
 		memset(blob, 0, len);
f09e2e
-		xfree(blob);
f09e2e
+		free(blob);
f09e2e
 	} else {
f09e2e
 		fatal("key_fingerprint_raw: blob is null");
f09e2e
 	}
f09e2e
@@ -324,7 +324,7 @@ 
f09e2e
 		break;
f09e2e
 	}
f09e2e
 	memset(dgst_raw, 0, dgst_raw_len);
f09e2e
-	xfree(dgst_raw);
f09e2e
+	free(dgst_raw);
f09e2e
 	return retval;
f09e2e
 }
f09e2e
 
f09e2e
@@ -447,11 +447,11 @@ 
f09e2e
 		n = uudecode(cp, blob, len);
f09e2e
 		if (n < 0) {
f09e2e
 			logerror("key_read: uudecode %s failed", cp);
f09e2e
-			xfree(blob);
f09e2e
+			free(blob);
f09e2e
 			return -1;
f09e2e
 		}
f09e2e
 		k = key_from_blob(blob, (u_int)n);
f09e2e
-		xfree(blob);
f09e2e
+		free(blob);
f09e2e
 		if (k == NULL) {
f09e2e
 			logerror("key_read: key_from_blob %s failed", cp);
f09e2e
 			return -1;
f09e2e
@@ -526,8 +526,8 @@ 
f09e2e
 			fprintf(f, "%s %s", key_ssh_name(key), uu);
f09e2e
 			success = 1;
f09e2e
 		}
f09e2e
-		xfree(blob);
f09e2e
-		xfree(uu);
f09e2e
+		free(blob);
f09e2e
+		free(uu);
f09e2e
 	}
f09e2e
 	return success;
f09e2e
 }
f09e2e
@@ -673,12 +673,12 @@ 
f09e2e
 		switch (key_type_from_name(p)) {
f09e2e
 		case KEY_RSA1:
f09e2e
 		case KEY_UNSPEC:
f09e2e
-			xfree(s);
f09e2e
+			free(s);
f09e2e
 			return 0;
f09e2e
 		}
f09e2e
 	}
f09e2e
 	verbose("key names ok: [%s]", names);
f09e2e
-	xfree(s);
f09e2e
+	free(s);
f09e2e
 	return 1;
f09e2e
 }
f09e2e
 
f09e2e
@@ -743,7 +743,7 @@ 
f09e2e
 		logerror("key_from_blob: remaining bytes in key blob %d", rlen);
f09e2e
  out:
f09e2e
 	if (ktype != NULL)
f09e2e
-		xfree(ktype);
f09e2e
+		free(ktype);
f09e2e
 	buffer_free(&b);
f09e2e
 	return key;
f09e2e
 }
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/misc.c	2013-10-30 17:14:26.017615218 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/misc.c	2013-10-30 17:15:07.360327581 +0100
f09e2e
@@ -251,13 +251,13 @@ 
f09e2e
 		*remote = SSH_TUNID_ANY;
f09e2e
 		sp = xstrdup(s);
f09e2e
 		if ((ep = strchr(sp, ':')) == NULL) {
f09e2e
-			xfree(sp);
f09e2e
+			free(sp);
f09e2e
 			return (a2tun(s, NULL));
f09e2e
 		}
f09e2e
 		ep[0] = '\0'; ep++;
f09e2e
 		*remote = a2tun(ep, NULL);
f09e2e
 		tun = a2tun(sp, NULL);
f09e2e
-		xfree(sp);
f09e2e
+		free(sp);
f09e2e
 		return (*remote == SSH_TUNID_ERR ? *remote : tun);
f09e2e
 	}
f09e2e
 
f09e2e
@@ -490,7 +490,7 @@ 
f09e2e
 	if (which >= args->num)
f09e2e
 		fatal("replacearg: tried to replace invalid arg %d >= %d",
f09e2e
 		    which, args->num);
f09e2e
-	xfree(args->list[which]);
f09e2e
+	free(args->list[which]);
f09e2e
 	args->list[which] = cp;
f09e2e
 }
f09e2e
 
f09e2e
@@ -501,8 +501,8 @@ 
f09e2e
 
f09e2e
 	if (args->list != NULL) {
f09e2e
 		for (i = 0; i < args->num; i++)
f09e2e
-			xfree(args->list[i]);
f09e2e
-		xfree(args->list);
f09e2e
+			free(args->list[i]);
f09e2e
+		free(args->list);
f09e2e
 		args->nalloc = args->num = 0;
f09e2e
 		args->list = NULL;
f09e2e
 	}
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/pam_user_authorized_keys.c	2013-10-30 17:14:26.017615218 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/pam_user_authorized_keys.c	2013-10-30 17:15:07.361327550 +0100
f09e2e
@@ -121,7 +121,7 @@ 
f09e2e
         }
f09e2e
         authorized_keys_file = tilde_expand_filename(auth_keys_file_buf, authorized_keys_file_allowed_owner_uid);
f09e2e
         strncpy(auth_keys_file_buf, authorized_keys_file, sizeof(auth_keys_file_buf) - 1 );
f09e2e
-        xfree(authorized_keys_file) /* when we percent_expand later, we'd step on this, so free it immediately */;
f09e2e
+        free(authorized_keys_file) /* when we percent_expand later, we'd step on this, so free it immediately */;
f09e2e
     }
f09e2e
 
f09e2e
     if(strstr(auth_keys_file_buf, "%h")) {
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/pam_user_key_allowed2.c	2013-10-30 17:14:26.018615187 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/pam_user_key_allowed2.c	2013-10-30 17:15:07.361327550 +0100
f09e2e
@@ -121,7 +121,7 @@ 
f09e2e
 			fp = key_fingerprint(found, SSH_FP_MD5, SSH_FP_HEX);
f09e2e
 			logit("Found matching %s key: %s",
f09e2e
 			    key_type(found), fp);
f09e2e
-			xfree(fp);
f09e2e
+			free(fp);
f09e2e
 			break;
f09e2e
 		}
f09e2e
 	}
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/ssh-dss.c	2013-10-30 17:14:26.014615310 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/ssh-dss.c	2013-10-30 17:15:07.361327550 +0100
f09e2e
@@ -135,17 +135,17 @@ 
f09e2e
 		if (strcmp("ssh-dss", ktype) != 0) {
f09e2e
 			logerror("ssh_dss_verify: cannot handle type %s", ktype);
f09e2e
 			buffer_free(&b);
f09e2e
-			xfree(ktype);
f09e2e
+			free(ktype);
f09e2e
 			return -1;
f09e2e
 		}
f09e2e
-		xfree(ktype);
f09e2e
+		free(ktype);
f09e2e
 		sigblob = buffer_get_string(&b, &len;;
f09e2e
 		rlen = buffer_len(&b);
f09e2e
 		buffer_free(&b);
f09e2e
 		if (rlen != 0) {
f09e2e
 			logerror("ssh_dss_verify: "
f09e2e
 			    "remaining bytes in signature %d", rlen);
f09e2e
-			xfree(sigblob);
f09e2e
+			free(sigblob);
f09e2e
 			return -1;
f09e2e
 		}
f09e2e
 	}
f09e2e
@@ -167,7 +167,7 @@ 
f09e2e
 
f09e2e
 	/* clean up */
f09e2e
 	memset(sigblob, 0, len);
f09e2e
-	xfree(sigblob);
f09e2e
+	free(sigblob);
f09e2e
 
f09e2e
 	/* sha1 the data */
f09e2e
 	EVP_DigestInit(&md, evp_md);
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/ssh-rsa.c	2013-10-30 17:14:26.015615278 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/ssh-rsa.c	2013-10-30 17:15:07.362327518 +0100
f09e2e
@@ -70,7 +70,7 @@ 
f09e2e
 
f09e2e
 		logerror("ssh_rsa_sign: RSA_sign failed: %s",
f09e2e
 		    ERR_error_string(ecode, NULL));
f09e2e
-		xfree(sig);
f09e2e
+		free(sig);
f09e2e
 		return -1;
f09e2e
 	}
f09e2e
 	if (len < slen) {
f09e2e
@@ -80,7 +80,7 @@ 
f09e2e
 		memset(sig, 0, diff);
f09e2e
 	} else if (len > slen) {
f09e2e
 		logerror("ssh_rsa_sign: slen %u slen2 %u", slen, len);
f09e2e
-		xfree(sig);
f09e2e
+		free(sig);
f09e2e
 		return -1;
f09e2e
 	}
f09e2e
 	/* encode signature */
f09e2e
@@ -96,7 +96,7 @@ 
f09e2e
 	}
f09e2e
 	buffer_free(&b);
f09e2e
 	memset(sig, 's', slen);
f09e2e
-	xfree(sig);
f09e2e
+	free(sig);
f09e2e
 
f09e2e
 	return 0;
f09e2e
 }
f09e2e
@@ -128,23 +128,23 @@ 
f09e2e
 	if (strcmp("ssh-rsa", ktype) != 0) {
f09e2e
 		logerror("ssh_rsa_verify: cannot handle type %s", ktype);
f09e2e
 		buffer_free(&b);
f09e2e
-		xfree(ktype);
f09e2e
+		free(ktype);
f09e2e
 		return -1;
f09e2e
 	}
f09e2e
-	xfree(ktype);
f09e2e
+	free(ktype);
f09e2e
 	sigblob = buffer_get_string(&b, &len;;
f09e2e
 	rlen = buffer_len(&b);
f09e2e
 	buffer_free(&b);
f09e2e
 	if (rlen != 0) {
f09e2e
 		logerror("ssh_rsa_verify: remaining bytes in signature %d", rlen);
f09e2e
-		xfree(sigblob);
f09e2e
+		free(sigblob);
f09e2e
 		return -1;
f09e2e
 	}
f09e2e
 	/* RSA_verify expects a signature of RSA_size */
f09e2e
 	modlen = RSA_size(key->rsa);
f09e2e
 	if (len > modlen) {
f09e2e
 		logerror("ssh_rsa_verify: len %u > modlen %u", len, modlen);
f09e2e
-		xfree(sigblob);
f09e2e
+		free(sigblob);
f09e2e
 		return -1;
f09e2e
 	} else if (len < modlen) {
f09e2e
 		u_int diff = modlen - len;
f09e2e
@@ -158,7 +158,7 @@ 
f09e2e
 	nid = (datafellows & SSH_BUG_RSASIGMD5) ? NID_md5 : NID_sha1;
f09e2e
 	if ((evp_md = EVP_get_digestbynid(nid)) == NULL) {
f09e2e
 		logerror("ssh_rsa_verify: EVP_get_digestbynid %d failed", nid);
f09e2e
-		xfree(sigblob);
f09e2e
+		free(sigblob);
f09e2e
 		return -1;
f09e2e
 	}
f09e2e
 	EVP_DigestInit(&md, evp_md);
f09e2e
@@ -168,7 +168,7 @@ 
f09e2e
 	ret = openssh_RSA_verify(nid, digest, dlen, sigblob, len, key->rsa);
f09e2e
 	memset(digest, 'd', sizeof(digest));
f09e2e
 	memset(sigblob, 's', len);
f09e2e
-	xfree(sigblob);
f09e2e
+	free(sigblob);
f09e2e
 	verbose("ssh_rsa_verify: signature %scorrect", (ret==0) ? "in" : "");
f09e2e
 	return ret;
f09e2e
 }
f09e2e
@@ -258,6 +258,6 @@ 
f09e2e
 	ret = 1;
f09e2e
 done:
f09e2e
 	if (decrypted)
f09e2e
-		xfree(decrypted);
f09e2e
+		free(decrypted);
f09e2e
 	return ret;
f09e2e
 }
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/userauth_pubkey_from_id.c	2013-10-30 17:14:26.014615310 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/userauth_pubkey_from_id.c	2013-10-30 17:15:07.362327518 +0100
f09e2e
@@ -92,9 +92,9 @@ 
f09e2e
     if(&b != NULL)
f09e2e
         buffer_free(&b);
f09e2e
     if(sig != NULL)
f09e2e
-        xfree(sig);
f09e2e
+        free(sig);
f09e2e
     if(pkblob != NULL)
f09e2e
-        xfree(pkblob);
f09e2e
+        free(pkblob);
f09e2e
     CRYPTO_cleanup_all_ex_data();
f09e2e
     return authenticated;
f09e2e
 }
f09e2e
--- pam_ssh_agent_auth-0.9.3.orig/uuencode.c	2013-10-30 17:14:26.015615278 +0100
f09e2e
+++ pam_ssh_agent_auth-0.9.3.orig/uuencode.c	2013-10-30 17:15:07.362327518 +0100
f09e2e
@@ -56,7 +56,7 @@ 
f09e2e
 	/* and remove trailing whitespace because __b64_pton needs this */
f09e2e
 	*p = '\0';
f09e2e
 	len = __b64_pton(encoded, target, targsize);
f09e2e
-	xfree(encoded);
f09e2e
+	free(encoded);
f09e2e
 	return len;
f09e2e
 }
f09e2e
 
f09e2e
@@ -79,5 +79,5 @@ 
f09e2e
 	}
f09e2e
 	if (i % 70 != 69)
f09e2e
 		fprintf(fp, "\n");
f09e2e
-	xfree(buf);
f09e2e
+	free(buf);
f09e2e
 }