Solved TODO

eclipse-ecal · Mar 19, 2024 · b8e7616 · b8e7616
1 parent 0657dbb
commit b8e7616
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/ecaludp/src/protocol/datagram_builder_v5.cpp b/ecaludp/src/protocol/datagram_builder_v5.cpp
@@ -35,7 +35,11 @@ namespace ecaludp
 
     DatagramList create_datagram_list(const std::vector<asio::const_buffer>& buffer_sequence, size_t max_datagram_size, std::array<char, 4> magic_header_bytes)
     {
-      // TODO: Complain when the max_udp_datagram_size is too small (the header doesn't even fit)
+      // Complain when the max_udp_datagram_size is too small (the header doesn't even fit)
+      if (max_datagram_size <= sizeof(ecaludp::v5::Header))
+      {
+        throw std::invalid_argument("max_datagram_size is too small");
+      }
 
       constexpr size_t header_size = sizeof(ecaludp::v5::Header);