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
F171592703: D1082.id5640.diff
Fri, Sep 11, 11:51 PM
Unknown Object (File)
Thu, Sep 10, 5:22 AM
Unknown Object (File)
Sat, Sep 5, 3:07 PM
Unknown Object (File)
Sat, Sep 5, 11:26 AM
Unknown Object (File)
Sat, Sep 5, 3:33 AM
Unknown Object (File)
Fri, Sep 4, 10:42 PM
Unknown Object (File)
Fri, Sep 4, 6:46 PM
Unknown Object (File)
Fri, Sep 4, 9:43 AM
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...