Page MenuHomeFreeBSD
Differential D2828

ARP request proper locking handling
ClosedPublic
Actions

Authored by eri on Jun 15 2015, 4:59 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 11, 3:22 PM
Unknown Object (File)
Mon, Apr 1, 12:33 AM
Unknown Object (File)
Feb 23 2024, 5:20 AM
Unknown Object (File)
Jan 3 2024, 7:52 AM
Unknown Object (File)
Dec 20 2023, 1:20 AM
Unknown Object (File)
Nov 26 2023, 11:56 AM
Unknown Object (File)
Nov 25 2023, 9:27 PM
Unknown Object (File)
Nov 23 2023, 4:30 AM
View All 95 Files
Subscribers
ae
franco_opnsense.org
imp
jim_netgate.com
koobs
lattera-gmail.com

Details

Reviewers
ae
gnn
Group Reviewers
network
Commits
rS284512: If there is a system with a bpf consumer running and a packet is wanted
Summary

If there is a system with a bpf consumer running and with direct dispatch policy on an expired arp entry can crash the system if the bpf code wants to sleep when called.

Also described here https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=200323

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

eri updated this revision to Diff 6211.Jun 15 2015, 4:59 PM
eri retitled this revision from to ARP request proper locking handling.
eri updated this object.
eri edited the test plan for this revision. (Show Details)
eri added reviewers: network, gnn.
eri set the repository for this revision to rS FreeBSD src repository - subversion.
Herald added a subscriber: imp. · View Herald TranscriptJun 15 2015, 4:59 PM
koobs added a subscriber: koobs.Jun 15 2015, 5:05 PM
lattera-gmail.com added a subscriber: lattera-gmail.com.Jun 15 2015, 5:38 PM
jim_netgate.com added a subscriber: jim_netgate.com.Jun 15 2015, 6:06 PM
franco_opnsense.org added a subscriber: franco_opnsense.org.Jun 15 2015, 8:52 PM
franco_opnsense.org added a comment.Jun 15 2015, 9:02 PM

It's highly likely that LLE_EXCLUSIVE is always set since la_preempt is being modified. The other code branch of arpresolve() calling arprequest() is modifying/unlocking la in the same way. Cheers for tracking this down.

ae accepted this revision.Jun 15 2015, 10:10 PM
ae added a reviewer: ae.
ae added a subscriber: ae.

Looks correct to me.

This revision is now accepted and ready to land.Jun 15 2015, 10:10 PM
gnn accepted this revision.Jun 16 2015, 2:29 AM
gnn edited edge metadata.

Accepted by mentor as well. Please commit.

Closed by commit rS284512: If there is a system with a bpf consumer running and a packet is wanted (authored by eri). · Explain WhyJun 17 2015, 12:23 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
netinet/
15 lines

Diff 6264

View Options

head/sys/netinet/if_ether.c

Loading...