The branch main has been updated by kp: URL: https://cgit.FreeBSD.org/src/commit/?id=b0ba95a9ce57db8b6eef6ab2e6c9c2482c7d30ed
commit b0ba95a9ce57db8b6eef6ab2e6c9c2482c7d30ed Author: Kristof Provost <k...@freebsd.org> AuthorDate: 2025-02-12 16:28:30 +0000 Commit: Kristof Provost <k...@freebsd.org> CommitDate: 2025-02-19 10:41:09 +0000 pf: remove now unnecessary casts from hash update calls Obtained from: OpenBSD, tedu <t...@openbsd.org>, e04cbca0bb Sponsored by: Rubicon Communications, LLC ("Netgate") --- sys/netpfil/pf/pf.c | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/sys/netpfil/pf/pf.c b/sys/netpfil/pf/pf.c index 0e816f1205c5..9b1a2f6a1bac 100644 --- a/sys/netpfil/pf/pf.c +++ b/sys/netpfil/pf/pf.c @@ -5067,16 +5067,16 @@ pf_tcp_iss(struct pf_pdesc *pd) ctx = V_pf_tcp_secret_ctx; - SHA512_Update(&ctx, (char *)&pd->hdr.tcp.th_sport, sizeof(u_short)); - SHA512_Update(&ctx, (char *)&pd->hdr.tcp.th_dport, sizeof(u_short)); + SHA512_Update(&ctx, &pd->hdr.tcp.th_sport, sizeof(u_short)); + SHA512_Update(&ctx, &pd->hdr.tcp.th_dport, sizeof(u_short)); switch (pd->af) { case AF_INET6: - SHA512_Update(&ctx, (char *)&pd->src->v6, sizeof(struct in6_addr)); - SHA512_Update(&ctx, (char *)&pd->dst->v6, sizeof(struct in6_addr)); + SHA512_Update(&ctx, &pd->src->v6, sizeof(struct in6_addr)); + SHA512_Update(&ctx, &pd->dst->v6, sizeof(struct in6_addr)); break; case AF_INET: - SHA512_Update(&ctx, (char *)&pd->src->v4, sizeof(struct in_addr)); - SHA512_Update(&ctx, (char *)&pd->dst->v4, sizeof(struct in_addr)); + SHA512_Update(&ctx, &pd->src->v4, sizeof(struct in_addr)); + SHA512_Update(&ctx, &pd->dst->v4, sizeof(struct in_addr)); break; } SHA512_Final(digest.bytes, &ctx);