Page MenuHomeFreeBSD
Differential D4547

EAGAIN handling for imsg_read.
ClosedPublic
Actions

Authored by araujo on Dec 14 2015, 7:27 AM.
Tags
None
Referenced Files
F136967651: D4547.id11219.diff
Thu, Nov 20, 9:00 PM
F136967033: D4547.id.diff
Thu, Nov 20, 8:57 PM
F136966762: D4547.id11322.diff
Thu, Nov 20, 8:56 PM
F136966747: D4547.id11223.diff
Thu, Nov 20, 8:56 PM
F136966046: D4547.id11220.diff
Thu, Nov 20, 8:53 PM
F136964768: D4547.diff
Thu, Nov 20, 8:48 PM
Unknown Object (File)
Wed, Nov 19, 7:24 PM
Unknown Object (File)
Wed, Nov 19, 5:47 AM
View All Files
Subscribers
imp

Details

Reviewers
• rodrigc
bapt
Commits
rS292270: EAGAIN handling for imsg_read.
Summary

sync code with OpenBSD: http://cvsweb.openbsd.org/cgi-bin/cvsweb/src/usr.sbin/ypldap/ypldap_dns.c?rev=1.10&content-type=text/x-cvsweb-markup&sortby=date

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 1625
Build 1631: arc lint + arc unit

Event Timeline

araujo updated this revision to Diff 11219.Dec 14 2015, 7:27 AM
araujo retitled this revision from to EAGAIN handling for imsg_read..
araujo updated this object.
araujo edited the test plan for this revision. (Show Details)
araujo added reviewers: • rodrigc, bapt.
Herald added a subscriber: imp. · View Herald TranscriptDec 14 2015, 7:27 AM
araujo updated this revision to Diff 11220.Dec 14 2015, 7:30 AM

Forgotten to add other two files.

araujo added a comment.Dec 14 2015, 9:18 AM

Note that on usr.sbin/ypldap/ldapclient.c there are some garbage that will be removed, it was used to debug the ypldap core dump.

I have fixed it already, I will open a new REVIEW.

araujo updated this revision to Diff 11223.Dec 14 2015, 9:20 AM

Remove the garbage mentioned on my another msg.

bapt accepted this revision.Dec 15 2015, 7:58 AM
bapt edited edge metadata.
This revision is now accepted and ready to land.Dec 15 2015, 7:58 AM
Closed by commit rS292270: EAGAIN handling for imsg_read. (authored by araujo). · Explain WhyDec 15 2015, 3:38 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
usr.sbin/
ypldap/
14 lines
8 lines
2 lines

Diff 11220

View Options

usr.sbin/ypldap/ldapclient.c

Loading...
View Options

usr.sbin/ypldap/ypldap.c

Loading...
View Options

usr.sbin/ypldap/ypldap_dns.c

Loading...