HomeFreeBSD

MFV r316930: 5814 bpobj_iterate_impl(): Close a refcount leak iterating on a…

Description

MFV r316930: 5814 bpobj_iterate_impl(): Close a refcount leak iterating on a sublist.

illumos/illumos-gate@b67dde11a73a9455d641403cbbb65ec2add41b41
https://github.com/illumos/illumos-gate/commit/b67dde11a73a9455d641403cbbb65ec2add41b41

https://www.illumos.org/issues/5814

Lets pull in this patch from freebsd:
http://svnweb.freebsd.org/base?view=revision&revision=271781
bpobj_iterate_impl(): Close a refcount leak iterating on a sublist.
If bpobj_space() returned non-zero here, the sublist would have been
left open, along with the bonus buffer hold it requires. This call
does not invoke any calls to bpobj_close() itself.
This bug doesn't have any known vector, but was found on inspection.
MFC after: 1 week
Sponsored by: Spectra Logic
Affects: All ZFS versions starting 21 May 2010 (illumos cde58dbc)
MFSpectraBSD: r1050998 on 2014/03/26
Fix bpobj_iterate_impl() to properly call bpobj_close() if bpobj_space()
returns an error.

Reviewed by: Prakash Surya <prakash.surya@delphix.com>
Reviewed by: Matthew Ahrens <mahrens@delphix.com>
Reviewed by: Paul Dagnelie <paul.dagnelie@delphix.com>
Reviewed by: Simon Klinkert <simon.klinkert@gmail.com>
Approved by: Gordon Ross <gwr@nexenta.com>
Author: Will Andrews <will@freebsd.org>

FreeBSD note: this is a "record-only" commit as the actual change was
directly committed to FreeBSD by (or on behalf of) the author.

Details

Provenance
avgAuthored on
Reviewer
matthew
Parents
rS318833: MFV r316925: 6101 attempt to lzc_create() a filesystem under a volume results…
Branches
Unknown
Tags
Unknown