Paths

Table of Contentst

Differential D20118

Fix erroneous assert in libzfs diff
ClosedPublic
Actions

Authored by freqlabs on May 1 2019, 1:05 AM.

Tags

None

Referenced Files

	F132765903: D20118.id.diff
	Sun, Oct 19, 7:04 PM

	F132765893: D20118.id61413.diff
	Sun, Oct 19, 7:03 PM

	F132765892: D20118.id56893.diff
	Sun, Oct 19, 7:03 PM

	F132704766: D20118.diff
	Sun, Oct 19, 4:59 AM

	Unknown Object (File)
	Mon, Oct 13, 1:50 AM

	Unknown Object (File)
	Sun, Oct 12, 7:30 AM

	Unknown Object (File)
	Tue, Sep 30, 2:58 AM

	Unknown Object (File)
	Tue, Sep 23, 8:04 AM

View All 86 Files

Subscribers

delphij

imp

Details

Reviewers

mav

Commits

rS351574: MFV/ZoL: Fix wrong assertion in libzfs diff error handling

Summary

All of the error cases in differ() set di->zerr to EPIPE so it doesn't make sense to assert that the error is EINVAL. If an error is set, assert it is EPIPE.

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

freqlabs created this revision.May 1 2019, 1:05 AM

Herald added subscribers: delphij, imp. · View Herald TranscriptMay 1 2019, 1:05 AM

ZoL commit: https://github.com/zfsonlinux/zfs/commit/9dc41a769df164875d974c2431b2453e70e16c41

freqlabs retitled this revision from Fix erronious assert in libzfs diff to Fix erroneous assert in libzfs diff.Aug 28 2019, 1:35 PM

mav accepted this revision.Aug 28 2019, 4:57 PM

This revision is now accepted and ready to land.Aug 28 2019, 4:57 PM

Closed by commit rS351574: MFV/ZoL: Fix wrong assertion in libzfs diff error handling (authored by mav). · Explain WhyAug 28 2019, 5:39 PM

This revision was automatically updated to reflect the committed changes.

mav added a commit: rS351574: MFV/ZoL: Fix wrong assertion in libzfs diff error handling.

Revision Contents
Changeset List

Path

Size

cddl/

contrib/

opensolaris/

lib/

libzfs/

common/

libzfs_diff.c

2 lines

Diff 56893

View Options

cddl/contrib/opensolaris/lib/libzfs/common/libzfs_diff.c

Fix erroneous assert in libzfs diffClosedPublicActions