Differential D22384

linux(4): return ELOOP instead of EMLINK
ClosedPublic
Actions

Authored by trasz on Nov 15 2019, 4:31 PM.

Details

Reviewers

emaste

Group Reviewers

Linux Emulation

Commits

rS354805: Make linux(4) open(2)/openat(2) return ELOOP instead of EMLINK,

Summary

Make linux(4) open(2)/openat(2) return ELOOP instead of EMLINK
with O_NOFOLLOW. This fixes an LTP testcase openat02:5.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 27538
Build 25762: arc lint + arc unit

Event Timeline

trasz created this revision.Nov 15 2019, 4:31 PM

Herald added a subscriber: imp. · View Herald TranscriptNov 15 2019, 4:31 PM

Harbormaster completed remote builds in B27538: Diff 64377.Nov 15 2019, 4:31 PM

trasz added reviewers: Linux Emulation, emaste.Nov 15 2019, 4:32 PM

trasz retitled this revision from linux(4): return ELOOP instead of EMLINKS to linux(4): return ELOOP instead of EMLINK.

trasz edited the summary of this revision. (Show Details)

Is there a short name/identifier for the LTP testcase that reported this? We could add a reference to it in a comment.

This revision is now accepted and ready to land.Nov 15 2019, 4:34 PM

Sure, it's openat02:5. I'll add it to the commit.

trasz edited the summary of this revision. (Show Details)Nov 15 2019, 6:07 PM

Ok - I think you could add it in the src but either way, fine.

Closed by commit rS354805: Make linux(4) open(2)/openat(2) return ELOOP instead of EMLINK, (authored by trasz). · Explain WhyNov 18 2019, 10:19 AM

This revision was automatically updated to reflect the committed changes.

trasz added a commit: rS354805: Make linux(4) open(2)/openat(2) return ELOOP instead of EMLINK,.

Revision Contents
Changeset List

Path

Size

sys/

compat/

linux/

linux_file.c

5 lines

Diff 64377

View Options

sys/compat/linux/linux_file.c

linux(4): return ELOOP instead of EMLINKClosedPublicActions