Actions
Bug #3743
closedzfs needs a refcount audit
Start date:
2013-04-23
Due date:
% Done:
100%
Estimated time:
Difficulty:
Medium
Tags:
needs-triage
Gerrit CR:
External Bug:
Description
A number of areas are vulnerable to refcount leaks, particularly in error paths. Existing refcount leaks need to be plugged.
Ideally, we should develop a mechanism to check for these types of mistakes, perhaps as a static analyzer extension.
Updated by Will Andrews over 10 years ago
Specific areas that currently need to be fixed:
- zap_cursor_fini() is not always called in error cases.
- zap_table_load(): calls dbuf_rele() unconditionally after dbuf_hold().
- dsl_dataset_hold_obj(): dbuf_rele() not called in error case.
Updated by Christopher Siden over 10 years ago
- Category set to zfs - Zettabyte File System
- Status changed from New to Closed
- Assignee set to Christopher Siden
- % Done changed from 0 to 100
commit b287be1 Author: Will Andrews <will@firepipe.net> Date: Tue Jun 11 10:13:38 2013 3743 zfs needs a refcount audit Reviewed by: Matthew Ahrens <mahrens@delphix.com> Reviewed by: Eric Schrock <eric.schrock@delphix.com> Reviewed by: George Wilson <george.wilson@delphix.com> Approved by: Christopher Siden <christopher.siden@delphix.com>
Actions