Commit d289478c authored May 12, 2026 by Raphael Zimmer Committed by Ilya Dryomov May 12, 2026

libceph: handle rbtree insertion error in decode_choose_args()



A message of type CEPH_MSG_OSD_MAP contains an OSD map that itself
contains a CRUSH map. The received CRUSH map may optionally contain
choose_args that get decoded in decode_choose_args(). In this function,
num_choose_arg_maps is read from the message, and a corresponding number
of crush_choose_arg_maps gets decoded afterwards. Each
crush_choose_arg_map has a choose_args_index, which serves as the key
when inserting it into the choose_args rbtree of the decoded crush_map.
If a (potentially corrupted) message contains two crush_choose_arg_maps
with the same index, the assertion in insert_choose_arg_map() triggers a
kernel BUG when trying to insert the second crush_choose_arg_map.

This patch fixes the issue by switching to the non-asserting rbtree
insertion function and rejecting the message if the insertion fails.

[ idryomov: changelog ]

Cc: stable@vger.kernel.org
Signed-off-by: Raphael Zimmer <raphael.zimmer@tu-ilmenau.de>
Reviewed-by: Ilya Dryomov <idryomov@gmail.com>
Signed-off-by: Ilya Dryomov <idryomov@gmail.com>

parent 35d0ed82

net/ceph/osdmap.c

+4 −1

Original line number	Diff line number	Diff line
		@@ -392,7 +392,10 @@ static int decode_choose_args(void *p, void end, struct crush_map *c)
		goto e_inval;
		}

		insert_choose_arg_map(&c->choose_args, arg_map);
		if (!__insert_choose_arg_map(&c->choose_args, arg_map)) {
		ret = -EEXIST;
		goto fail;
		}
		}

		return 0;