Skip to content

Conversation

@KanjiMonster
Copy link
Contributor

Link info ops that use link info also need to implement rtnl_link_info_ops::io_clone(), else cloning the link will result in a broken object, and trying to use any rtnl_link_bridge_*() accessors will result in a null pointer access due to rtnl_link::l_info not being taken over.

So implement the missing rtnl_link_info_ops::io_clone() callback.

Fixes: 7391a38 ("bridge: Add support for link_info of a bridge")

Link info ops that use link info also need to implement
rtnl_link_info_ops::io_clone(), else cloning the link will result in a
broken object, and trying to use any rtnl_link_bridge_*() accessors will
result in a null pointer access due to rtnl_link::l_info not being taken
over.

So implement the missing rtnl_link_info_ops::io_clone() callback.

Fixes: 7391a38 ("bridge: Add support for link_info of a bridge")
Signed-off-by: Jonas Gorski <[email protected]>
thom311 pushed a commit that referenced this pull request Jun 3, 2025
Link info ops that use link info also need to implement
rtnl_link_info_ops::io_clone(), else cloning the link will result in a
broken object, and trying to use any rtnl_link_bridge_*() accessors will
result in a null pointer access due to rtnl_link::l_info not being taken
over.

So implement the missing rtnl_link_info_ops::io_clone() callback.

Fixes: 7391a38 ("bridge: Add support for link_info of a bridge")
Signed-off-by: Jonas Gorski <[email protected]>

#426
@thom311
Copy link
Owner

thom311 commented Jun 3, 2025

merged. Thank you!!

@thom311 thom311 closed this Jun 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants