<div dir="auto">Hindi language please </div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Jun 24, 2022, 12:40 AM Andrew Clayton <<a href="mailto:andrew@digital-domain.net">andrew@digital-domain.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">In src/nxt_http_route_addr.c::nxt_http_route_addr_pattern_parse() there<br>
was potentially undefined behaviour when shifting a 32 bit value by 32<br>
bits, this could happen if cidr_prefix was 0.<br>
<br>
Promote the shiftee to unsigned long long to avoid this issue.<br>
---<br>
V2 Changes<br>
<br>
  * Promote the shiftee to ULL rather than just UL<br>
<br>
 src/nxt_http_route_addr.c | 2 +-<br>
 1 file changed, 1 insertion(+), 1 deletion(-)<br>
<br>
diff --git a/src/nxt_http_route_addr.c b/src/nxt_http_route_addr.c<br>
index 2907a90..985fb73 100644<br>
--- a/src/nxt_http_route_addr.c<br>
+++ b/src/nxt_http_route_addr.c<br>
@@ -233,7 +233,7 @@ nxt_http_route_addr_pattern_parse(nxt_mp_t *mp,<br>
         }<br>
<br>
         addr.length = delim - addr.start;<br>
-        inet->end = htonl(0xFFFFFFFF & (0xFFFFFFFF << (32 - cidr_prefix)));<br>
+        inet->end = htonl(0xFFFFFFFF & (0xFFFFFFFFULL << (32 - cidr_prefix)));<br>
<br>
         inet->start = nxt_inet_addr(addr.start, addr.length) & inet->end;<br>
         if (nxt_slow_path(inet->start == INADDR_NONE)) {<br>
-- <br>
2.36.1<br>
<br>
_______________________________________________<br>
unit mailing list -- <a href="mailto:unit@nginx.org" target="_blank" rel="noreferrer">unit@nginx.org</a><br>
To unsubscribe send an email to <a href="mailto:unit-leave@nginx.org" target="_blank" rel="noreferrer">unit-leave@nginx.org</a><br>
</blockquote></div>