diff options
| author | Andrew Clayton <andrew@digital-domain.net> | 2022-06-15 16:20:01 +0100 |
|---|---|---|
| committer | Andrew Clayton <andrew@digital-domain.net> | 2022-06-15 21:15:32 +0100 |
| commit | 0d6417673bbeddf43c7c2d298a3edca4bf6ee54c (patch) | |
| tree | 57431db7d6aa31960eea3d57b9478b0ba56e5f46 | |
| parent | 6a8081d71e805b12d0f7fd32ce72d60babadfc85 (diff) | |
| download | unit-0d6417673bbeddf43c7c2d298a3edca4bf6ee54c.tar.gz unit-0d6417673bbeddf43c7c2d298a3edca4bf6ee54c.tar.bz2 | |
Route: avoided undefined behaviour.archive/shiftTooManyBits
In src/nxt_http_route_addr.c::nxt_http_route_addr_pattern_parse() there
was potentially undefined behaviour when shifting a 32 bit value by 32
bits, this could happen if cidr_prefix was 0.
Promote the shiftee to unsigned long to avoid this issue.
| -rw-r--r-- | src/nxt_http_route_addr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/nxt_http_route_addr.c b/src/nxt_http_route_addr.c index 2907a902..67c781af 100644 --- a/src/nxt_http_route_addr.c +++ b/src/nxt_http_route_addr.c @@ -233,7 +233,7 @@ nxt_http_route_addr_pattern_parse(nxt_mp_t *mp, } addr.length = delim - addr.start; - inet->end = htonl(0xFFFFFFFF & (0xFFFFFFFF << (32 - cidr_prefix))); + inet->end = htonl(0xFFFFFFFF & (0xFFFFFFFFUL << (32 - cidr_prefix))); inet->start = nxt_inet_addr(addr.start, addr.length) & inet->end; if (nxt_slow_path(inet->start == INADDR_NONE)) { |