libiscsiutil: preserve errno across warning logs
ClosedPublic
Actions

Authored by kevans on Tue, Apr 21, 4:41 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, May 14, 3:40 PM

	Unknown Object (File)
	Thu, May 14, 9:48 AM

	Unknown Object (File)
	Thu, May 14, 4:35 AM

	Unknown Object (File)
	Mon, May 11, 2:05 PM

	Unknown Object (File)
	Sun, May 10, 6:36 PM

	Unknown Object (File)
	Sun, May 10, 12:15 PM

	Unknown Object (File)
	Sun, May 10, 12:10 PM

	Unknown Object (File)
	Sun, May 10, 8:03 AM

View All 23 Files

Subscribers

imp

Details

Reviewers

asomers
jhb

Commits

rGfe9d81d7c660: libiscsiutil: preserve errno across warning logs

Summary

Various calls in the implementation can clobber errno; preserve it for
the caller since none of these will bubble up pass/fail to simplify
some future error handling in ctld.

Diff Detail

Repository

rG FreeBSD src repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kevans created this revision.Tue, Apr 21, 4:41 AM

Herald added a subscriber: imp. · View Herald TranscriptTue, Apr 21, 4:41 AM

kevans requested review of this revision.Tue, Apr 21, 4:41 AM

Harbormaster completed remote builds in B72388: Diff 175947.Tue, Apr 21, 4:41 AM

kevans added a child revision: D56540: ctl: add a new error type to distinguish EINVAL vs. EEXIST.Tue, Apr 21, 4:41 AM

jhb accepted this revision.Thu, Apr 23, 4:30 PM

This revision is now accepted and ready to land.Thu, Apr 23, 4:30 PM

Closed by commit rGfe9d81d7c660: libiscsiutil: preserve errno across warning logs (authored by kevans). · Explain WhyTue, Apr 28, 8:52 PM

This revision was automatically updated to reflect the committed changes.

kevans added a commit: rGfe9d81d7c660: libiscsiutil: preserve errno across warning logs.

Revision Contents
Changeset List

Path

Size

lib/

libiscsiutil/

log.c

4 lines

Diff 176763

View Options

libiscsiutil: preserve errno across warning logsClosedPublicActions