libceph: old_key in process_one_ticket() is redundant
authorIlya Dryomov <idryomov@gmail.com>
Fri, 2 Dec 2016 15:35:06 +0000 (16:35 +0100)
committerIlya Dryomov <idryomov@gmail.com>
Mon, 12 Dec 2016 22:09:19 +0000 (23:09 +0100)
Since commit 0a990e709356 ("ceph: clean up service ticket decoding"),
th->session_key isn't assigned until everything is decoded.

Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Reviewed-by: Sage Weil <sage@redhat.com>
net/ceph/auth_x.c

index 4d14dc8c7746635f95d59478979d64dbdfd75f92..28bde937d58aee303f936226cbbb04bbe1323e99 100644 (file)
@@ -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;