libceph: fix invalid accesses to ceph_connection_v1_info
authorIlya Dryomov <idryomov@gmail.com>
Thu, 3 Jul 2025 10:10:50 +0000 (12:10 +0200)
committerIlya Dryomov <idryomov@gmail.com>
Wed, 10 Sep 2025 19:22:56 +0000 (21:22 +0200)
commitcdbc9836c7afadad68f374791738f118263c5371
tree25488ddc215bd621fc249656a900f77e87d84c06
parent249e0a47cdb46bb9eae65511c569044bd8698d7d
libceph: fix invalid accesses to ceph_connection_v1_info

There is a place where generic code in messenger.c is reading and
another place where it is writing to con->v1 union member without
checking that the union member is active (i.e. msgr1 is in use).

On 64-bit systems, con->v1.auth_retry overlaps with con->v2.out_iter,
so such a read is almost guaranteed to return a bogus value instead of
0 when msgr2 is in use.  This ends up being fairly benign because the
side effect is just the invalidation of the authorizer and successive
fetching of new tickets.

con->v1.connect_seq overlaps with con->v2.conn_bufs and the fact that
it's being written to can cause more serious consequences, but luckily
it's not something that happens often.

Cc: stable@vger.kernel.org
Fixes: cd1a677cad99 ("libceph, ceph: implement msgr2.1 protocol (crc and secure modes)")
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Reviewed-by: Viacheslav Dubeyko <Slava.Dubeyko@ibm.com>
net/ceph/messenger.c