libiscsiutil: preserve errno across warning logs
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F163263391: D56539.id.diff
	Tue, Jul 21, 1:44 PM

	F163190169: D56539.diff
	Mon, Jul 20, 9:54 PM

	Unknown Object (File)
	Mon, Jul 20, 10:59 AM

	Unknown Object (File)
	Thu, Jul 16, 7:34 PM

	Unknown Object (File)
	Thu, Jul 16, 1:45 AM

	Unknown Object (File)
	Wed, Jul 15, 7:18 AM

	Unknown Object (File)
	Wed, Jul 15, 7:18 AM

	Unknown Object (File)
	Tue, Jul 14, 9:42 PM

View All 41 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.Apr 21 2026, 4:41 AM

Herald added a subscriber: imp. · View Herald TranscriptApr 21 2026, 4:41 AM

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

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

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

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

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

Closed by commit rGfe9d81d7c660: libiscsiutil: preserve errno across warning logs (authored by kevans). · Explain WhyApr 28 2026, 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