On Wed, 2025-08-20 at 14:39 -0400, Benjamin Marzinski wrote: > print_foreign_topology() called get_paths() to get a vector of > (struct gen_path *) items and then called get_multipath_layout__(), > which expects a vector of (struct gen_multipath *) items, with the > path > vector. This can easily end badly. Fix it to correctly call > get_path_layout__(), and rename width to p_width in the functions > that > end up calling snprint_multipath_topology__(), which is expecting to > get > passed the path field widths.
Thanks for catching this! I don't like the p_width rename so much, as we generally don't have this kind of variable naming convention, but that's not a problem. Reviewed-by: Martin Wilck <mwi...@suse.com>
