Project

General

Profile

Actions

Bug #3645

closed

dmu_send_impl: possibilty of pool hold leak

Added by Andriy Gapon about 8 years ago. Updated about 8 years ago.

Status:
Closed
Priority:
Normal
Category:
zfs - Zettabyte File System
Start date:
2013-03-23
Due date:
% Done:

100%

Estimated time:
Difficulty:
Bite-size
Tags:
needs-triage
Gerrit CR:

Description

This can happen if dump_bytes fails.
Additionally, dsl_dataset_long_rele would be called without matching
dsl_dataset_long_hold.


Files

dmu_send.c.patch (686 Bytes) dmu_send.c.patch Martin Matuška, 2013-04-03 10:40 AM
Actions #1

Updated by Andriy Gapon about 8 years ago

Andriy Gapon wrote:

This can happen if dump_bytes fails.

The first call to dump_bytes, to be specific:
https://github.com/illumos/illumos-gate/blob/master/usr/src/uts/common/fs/zfs/dmu_send.c#L484

Actions #2

Updated by Martin Matuška about 8 years ago

Attaching patch that fixes the problem.

Actions #3

Updated by Matthew Ahrens about 8 years ago

See also bug 3695, which is a dup of this.

Actions #4

Updated by Matthew Ahrens about 8 years ago

  • Status changed from New to In Progress
  • Assignee set to Matthew Ahrens
Actions #5

Updated by Christopher Siden about 8 years ago

  • Status changed from In Progress to Closed
  • % Done changed from 0 to 100
commit de8d9cf
Author: Matthew Ahrens <mahrens@delphix.com>
Date:   Wed Apr 10 14:54:56 2013

    3645 dmu_send_impl: possibilty of pool hold leak
    3692 Panic on zfs receive of a recursive deduplicated stream
    Reviewed by: Adam Leventhal <ahl@delphix.com>
    Reviewed by: Christopher Siden <christopher.siden@delphix.com>
    Reviewed by: Dan McDonald <danmcd@nexenta.com>
    Approved by: Richard Lowe <richlowe@richlowe.net>
Actions

Also available in: Atom PDF