pipe_direct_write: Fix mismatched pipelock/unlock
ClosedPublic
Actions

Authored by cem on Jul 13 2015, 4:41 PM.

Tags

None

Referenced Files

	F159971962: D3069.id6892.diff
	Sat, Jun 20, 2:38 AM

	Unknown Object (File)
	May 3 2026, 10:22 PM

	Unknown Object (File)
	Apr 30 2026, 8:19 AM

	Unknown Object (File)
	Apr 29 2026, 10:17 AM

	Unknown Object (File)
	Apr 29 2026, 10:10 AM

	Unknown Object (File)
	Apr 23 2026, 6:29 AM

	Unknown Object (File)
	Apr 7 2026, 10:01 AM

	Unknown Object (File)
	Apr 6 2026, 1:15 PM

View All Files

Subscribers

imp

Details

Reviewers

kib
markj

Commits

rS285483: pipe_direct_write: Fix mismatched pipelock/unlock

Summary

If a signal is caught in pipelock, causing it to fail, pipe_direct_write should
not try to pipeunlock.

Reported by: pho
Sponsored by: EMC / Isilon Storage Division

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Event Timeline

cem updated this revision to Diff 6891.Jul 13 2015, 4:41 PM

cem retitled this revision from to pipe_direct_write: Fix mismatched pipelock/unlock.

cem updated this object.

cem edited the test plan for this revision. (Show Details)

cem added reviewers: markj, kib.

Herald added a subscriber: imp. · View Herald TranscriptJul 13 2015, 4:41 PM

kib accepted this revision.Jul 13 2015, 5:07 PM

kib edited edge metadata.

kib added inline comments.

sys/kern/sys_pipe.c
949	if (error != 0)
951	if ((wpipe->pipe_state & PIPE_EOF) != 0) {

This revision is now accepted and ready to land.Jul 13 2015, 5:07 PM

Fix style issues (thanks kib)

This revision now requires review to proceed.Jul 13 2015, 5:09 PM

Closed by commit rS285483: pipe_direct_write: Fix mismatched pipelock/unlock (authored by cem). · Explain WhyJul 13 2015, 5:45 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

sys/

kern/

sys_pipe.c

5 lines

Diff 6891

View Options

pipe_direct_write: Fix mismatched pipelock/unlockClosedPublicActions