From f47c15e7bac217f0b0282a6e535f03f26e1e9d9d Mon Sep 17 00:00:00 2001 From: Nick Mathewson Date: Fri, 27 May 2011 23:28:17 -0400 Subject: [PATCH] Make the new http_connect_fail_test use a multicast addr, not broadcast Turns out that FreeBSD does _not_ give a ENETUNREACH error when told to make a TCP socket to 255.255.255.255, but it is quite happy to do so for 239.10.20.30. So that's what we'll do. Found by Robert Ransom and Dave Hart. --- test/regress_http.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/test/regress_http.c b/test/regress_http.c index cf0fb971..0aef4d74 100644 --- a/test/regress_http.c +++ b/test/regress_http.c @@ -3040,9 +3040,9 @@ http_connection_fail_test(void *arg) evhttp_free(http); http = NULL; - /* Pick an unroutable address. The limited broadcast address should do - * when working with TCP. */ - evcon = evhttp_connection_base_new(data->base, NULL, "255.255.255.255", 80); + /* Pick an unroutable address. This administratively scoped multicast + * address should do when working with TCP. */ + evcon = evhttp_connection_base_new(data->base, NULL, "239.10.20.30", 80); tt_assert(evcon); /*