Commit 462e6504 authored by Ilya Dryomov's avatar Ilya Dryomov
Browse files

libceph: old_key in process_one_ticket() is redundant 



Since commit 0a990e70 ("ceph: clean up service ticket decoding"),
th->session_key isn't assigned until everything is decoded.

Signed-off-by: default avatarIlya Dryomov <idryomov@gmail.com>
Reviewed-by: default avatarSage Weil <sage@redhat.com>
parent 36721ece

net/ceph/auth_x.c

+1 −3
Original line number Diff line number Diff line
@@ -148,7 +148,6 @@ static int process_one_ticket(struct ceph_auth_client *ac, 
	int dlen;

	char is_enc;

	struct timespec validity;

	struct ceph_crypto_key old_key;

	void *ticket_buf = NULL;

	void *tp, *tpend;

	void **ptp;
@@ -187,7 +186,6 @@ static int process_one_ticket(struct ceph_auth_client *ac, 
	if (tkt_struct_v != 1)

		goto bad;



	memcpy(&old_key, &th->session_key, sizeof(old_key));

	ret = ceph_crypto_key_decode(&new_session_key, &dp, dend);

	if (ret)

		goto out;
@@ -204,7 +202,7 @@ static int process_one_ticket(struct ceph_auth_client *ac, 
	if (is_enc) {

		/* encrypted */

		dout(" encrypted ticket\n");

		dlen = ceph_x_decrypt(&old_key, p, end, &ticket_buf, 0);

		dlen = ceph_x_decrypt(&th->session_key, p, end, &ticket_buf, 0);

		if (dlen < 0) {

			ret = dlen;

			goto out;