libiscsiutil: preserve errno across warning logs
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F167859649: D56539.id.diff
	Tue, Aug 25, 1:37 AM

	F167852383: D56539.diff
	Tue, Aug 25, 12:32 AM

	Unknown Object (File)
	Sun, Aug 23, 7:38 AM

	Unknown Object (File)
	Sun, Aug 23, 1:09 AM

	Unknown Object (File)
	Sun, Aug 23, 12:35 AM

	Unknown Object (File)
	Wed, Aug 12, 7:39 PM

	Unknown Object (File)
	Sun, Aug 9, 6:07 PM

	Unknown Object (File)
	Sun, Aug 9, 6:06 PM

View All 58 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