Date: Mon, 30 Dec 2024 15:04:49 GMT From: Michael Tuexen <tuexen@FreeBSD.org> To: src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org Subject: git: b5739c8b1222 - main - TCP RACK, BBR: ensure return value is always ininitialized Message-ID: <202412301504.4BUF4nM3084987@gitrepo.freebsd.org>
next in thread | raw e-mail | index | archive | help
The branch main has been updated by tuexen: URL: https://cgit.FreeBSD.org/src/commit/?id=b5739c8b12226c791bc8e7a4ac6e48965333509a commit b5739c8b12226c791bc8e7a4ac6e48965333509a Author: Michael Tuexen <tuexen@FreeBSD.org> AuthorDate: 2024-12-30 15:02:43 +0000 Commit: Michael Tuexen <tuexen@FreeBSD.org> CommitDate: 2024-12-30 15:02:43 +0000 TCP RACK, BBR: ensure return value is always ininitialized Do not return an uninitialized value from ctf_do_queued_segments() in case no packets are actually processed (all are skipped). Reviewed by: rrs CID: 1523774 MFC after: 1 week Sponsored by: Netflix, Inc. Differential Revision: https://reviews.freebsd.org/D48217 --- sys/netinet/tcp_stacks/rack_bbr_common.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/sys/netinet/tcp_stacks/rack_bbr_common.c b/sys/netinet/tcp_stacks/rack_bbr_common.c index 150298f8413f..156091feeb30 100644 --- a/sys/netinet/tcp_stacks/rack_bbr_common.c +++ b/sys/netinet/tcp_stacks/rack_bbr_common.c @@ -478,7 +478,7 @@ skipped_pkt: if (no_vn == 0) { CURVNET_RESTORE(); } - return(retval); + return (0); } int
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202412301504.4BUF4nM3084987>