Problem: msg.cpp:set_routing_id accepts a zero routing ID

Solution: check/reject a zero routing ID with EINVAL.

Problem: msg.cpp:set_routing_id accepts a zero routing ID
Solution: check/reject a zero routing ID with EINVAL.
4db73fc1 · Pieter Hintjens · 6b00f40f · 4db73fc1
Commit 4db73fc1 authored Sep 11, 2015 by Pieter Hintjens
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 2 deletions

msg.cpp src/msg.cpp +6 -2

No files found.
--- a/src/msg.cpp
+++ b/src/msg.cpp
@@ -504,8 +504,12 @@ uint32_t zmq::msg_t::get_routing_id ()
 int zmq::msg_t::set_routing_id (uint32_t routing_id_)
 {
-    u.base.routing_id = routing_id_;
+    if (routing_id_) {
-    return 0;
+        u.base.routing_id = routing_id_;
+        return 0;
+    }
+    errno = EINVAL;
+    return -1;
 }
 zmq::atomic_counter_t *zmq::msg_t::refcnt()