From: Ture Pålsson <ture@turepalsson.se>
Date: Wed, 1 Aug 2018 15:34:09 +0000 (+0000)
Subject: Fixed build on FreeBSD v11.2 (#266)
X-Git-Tag: M-staged-PR266
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=7f7b4fd3f9af404d5bc528f7a73320f3ed1cc7d4;p=thirdparty%2Fsquid.git

Fixed build on FreeBSD v11.2 (#266)

... and other platforms where htonl() is a macro that uses its argument
multiple times (GCC -Werror=sequence-point).
---

diff --git a/src/wccp2.cc b/src/wccp2.cc
index f17f728feb..eb36fc0082 100644
--- a/src/wccp2.cc
+++ b/src/wccp2.cc
@@ -1743,7 +1743,8 @@ wccp2AssignBuckets(void *)
 
         assignment_key = (struct assignment_key_t *) &wccp_packet[offset];
 
-        assignment_key->master_number = htonl(++service_list_ptr->change_num);
+        ++service_list_ptr->change_num;
+        assignment_key->master_number = htonl(service_list_ptr->change_num);
 
         offset += sizeof(struct assignment_key_t);