Bug 17872

Summary: arpintr() fix followup
Product: Base System Reporter: C. Stephen Gunn <csg>
Component: kernAssignee: freebsd-bugs (Nobody) <bugs>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: 4.0-STABLE   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.diff none

Description C. Stephen Gunn 2000-04-09 04:30:01 UTC 
My previous patch to arpintr() does too much work.  I incorrectly
assumed that since it was wrong to check m->m_len for the length
of the entire mbuf chain, (completely forgetting m->mh_len), that
I had to do all the work myself.  I even made a comment about
how silly it was to do all the work with "wanna implement m_size?"

Fix: Apply this patch to remove the for-loop calculating the length
of the mbuf chain, and just trust m->mh_len.

How-To-Repeat: 
There's no crash, just a code cleanup.
Comment 1 Jeroen Ruigrok van der Werven freebsd_committer freebsd_triage 2000-04-30 16:29:02 UTC 
State Changed
From-To: open->closed

Closed by Wes Peters on the 11th of April to 5-CURRENT and 4-STABLE.