It's an internal error if get_impl returns null. Add an assert to be sure.

[simgrid.git] / src / s4u / s4u_Link.cpp

diff --git a/src/s4u/s4u_Link.cpp b/src/s4u/s4u_Link.cpp
index 65d544b48d012996e7aec118b5b39302e3c4f993..0f5c75101388a1df59e75d591f43c71226c5b4e6 100644 (file)
--- a/src/s4u/s4u_Link.cpp
+++ b/src/s4u/s4u_Link.cpp
@@ -40,7 +40,9 @@ kernel::resource::LinkImpl* Link::get_impl() const
   xbt_assert(
       get_sharing_policy() != SharingPolicy::SPLITDUPLEX,
       "Impossible to get a LinkImpl* from a Split-Duplex link. You should call this method to each UP/DOWN member");
-  return dynamic_cast<kernel::resource::LinkImpl*>(pimpl_);
+  auto* link_impl = dynamic_cast<kernel::resource::LinkImpl*>(pimpl_);
+  xbt_assert(link_impl != nullptr);
+  return link_impl;
 }
 
 Link* Link::by_name_or_null(const std::string& name)
@@ -195,12 +197,16 @@ Link* Link::set_properties(const std::unordered_map<std::string, std::string>& p
 
 Link* SplitDuplexLink::get_link_up() const
 {
-  return dynamic_cast<kernel::resource::SplitDuplexLinkImpl*>(pimpl_)->get_link_up();
+  const auto* pimpl = dynamic_cast<kernel::resource::SplitDuplexLinkImpl*>(pimpl_);
+  xbt_assert(pimpl, "Requesting link_up from a non split-duplex link: %s", get_cname());
+  return pimpl->get_link_up();
 }
 
 Link* SplitDuplexLink::get_link_down() const
 {
-  return dynamic_cast<kernel::resource::SplitDuplexLinkImpl*>(pimpl_)->get_link_down();
+  const auto* pimpl = dynamic_cast<kernel::resource::SplitDuplexLinkImpl*>(pimpl_);
+  xbt_assert(pimpl, "Requesting link_down from a non split-duplex link: %s", get_cname());
+  return pimpl->get_link_down();
 }
 
 SplitDuplexLink* SplitDuplexLink::by_name(const std::string& name)