Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 12 Nov 2023 04:13:32 +0000
From:      bugzilla-noreply@freebsd.org
To:        net@FreeBSD.org
Subject:   [Bug 275002] if_wg: Missing failure check for m_copyback()
Message-ID:  <bug-275002-7501-W1cB0uonpu@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-275002-7501@https.bugs.freebsd.org/bugzilla/>
References:  <bug-275002-7501@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D275002

--- Comment #3 from Aaron LI <aly@aaronly.me> ---
(In reply to Kristof Provost from comment #1)

> I don't think that's an issue here because the mbuf is known to be of
> length 'len' before it's passed to m_copyout(). The allocation is done
> just before the m_copyback().

Oh, that's true. I forget that implication set by m_get2() just before
m_copyback().

As kevans suggested, a KASSERT() after m_copyback() would make it clear
and remove this confusion.

--=20
You are receiving this mail because:
You are the assignee for the bug.=



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?bug-275002-7501-W1cB0uonpu>