Page MenuHomeFreeBSD
Differential D1082

Rewrite linux_recvfrom. To avoid double conversion of sockaddr use kern_recvit() directly. And check fromlen parameter before sockaddr copyin and conversion.
ClosedPublic
Actions

Authored by dchagin on Nov 2 2014, 10:17 PM.
Tags
None
Referenced Files
F169664812: D1082.id5640.diff
Wed, Sep 2, 5:37 AM
Unknown Object (File)
Tue, Sep 1, 12:44 AM
Unknown Object (File)
Mon, Aug 31, 7:06 PM
Unknown Object (File)
Mon, Aug 31, 5:31 AM
Unknown Object (File)
Sun, Aug 30, 12:35 AM
Unknown Object (File)
Sat, Aug 29, 6:56 PM
Unknown Object (File)
Sat, Aug 29, 6:50 PM
Unknown Object (File)
Mon, Aug 24, 7:51 PM
View All Files
Subscribers
imp
trasz

Details

Reviewers
None
Group Reviewers
Linux Emulation
Commits
rS283433: Rewrite linux_recvfrom. To avoid double conversion of sockaddr use

Diff Detail

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

Event Timeline

dchagin updated this revision to Diff 2228.Nov 2 2014, 10:17 PM
dchagin retitled this revision from to Rewrite linux_recvfrom. To avoid double conversion of sockaddr use kern_recvit() directly. And check fromlen parameter before sockaddr copyin and conversion..
dchagin added a reviewer: Linux Emulation.Nov 2 2014, 10:19 PM
trasz added a subscriber: trasz.Mar 27 2015, 11:19 AM
trasz added inline comments.
sys/compat/linux/linux_socket.c
1035 ↗(On Diff #2228)

!= 0; few other occurences below.

1053 ↗(On Diff #2228)

sys_recvfrom() calls recvit() here, which also copies out the msg_namelen. It seems to be missing here.

1057 ↗(On Diff #2228)

!= NULL

dchagin updated this revision to Diff 4474.Mar 27 2015, 8:58 PM

whoops, you are right. fixed. style(9) also fixed.

Herald added a subscriber: imp. · View Herald TranscriptMar 27 2015, 8:58 PM
Closed by commit rS283433: Rewrite linux_recvfrom. To avoid double conversion of sockaddr use (authored by dchagin). · Explain WhyMay 24 2015, 4:26 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
head/
sys/
compat/
linux/
66 lines

Diff 5640

View Options

head/sys/compat/linux/linux_socket.c

Loading...