From 38c005e7c07b5ccaab3345dc8c66cbc27b95692a Mon Sep 17 00:00:00 2001 From: Garrett D'Amore Date: Mon, 20 Jan 2020 23:31:55 -0800 Subject: fixes #1169 survey and xsurvey could use message cloning fixes #1160 Consider limiting maximum hop count to 15 fixes #1098 Maximum maxTTL should be compile time defined This doesn't expose the max-MaxTTL in the CMakeList.txt -- there is really no reason anyone should be changing it. This does not yet inline the message header into the nni_msg_t, but it is my intention to do so soon, and eliminate most of the conditional cases for failure on inserting into the header. --- src/protocol/reqrep0/req_test.c | 3 +++ 1 file changed, 3 insertions(+) (limited to 'src/protocol/reqrep0/req_test.c') diff --git a/src/protocol/reqrep0/req_test.c b/src/protocol/reqrep0/req_test.c index 2524dfd2..6cb06f62 100644 --- a/src/protocol/reqrep0/req_test.c +++ b/src/protocol/reqrep0/req_test.c @@ -52,6 +52,9 @@ test_req_ttl_option(void) TEST_NNG_PASS(nng_setopt_int(req, opt, 1)); TEST_NNG_FAIL(nng_setopt_int(req, opt, 0), NNG_EINVAL); TEST_NNG_FAIL(nng_setopt_int(req, opt, -1), NNG_EINVAL); + // This test will fail if the NNI_MAX_MAX_TTL is changed from the + // builtin default of 15. + TEST_NNG_FAIL(nng_setopt_int(req, opt, 16), NNG_EINVAL); TEST_NNG_FAIL(nng_setopt_int(req, opt, 256), NNG_EINVAL); TEST_NNG_PASS(nng_setopt_int(req, opt, 3)); TEST_NNG_PASS(nng_getopt_int(req, opt, &v)); -- cgit v1.2.3-70-g09d2