Project

General

Profile

Bug #4900

zfs filesystem/snapshot limit leaks

Added by Jerry Jelinek almost 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Start date:
2014-05-30
Due date:
% Done:

0%

Estimated time:
Difficulty:
Medium
Tags:
needs-triage

Description

I see another [leak] in dsl_dir_rename_check() on the error path in the "if (dsl_dir_is_zapified(dd))" block (near line 1734). It returns without dropping the dsl dir refs for newparent and dd that were aquired at the begining of the function.

The fs_limits is clearly writing its metadata (zap entries) while it's only in the enabled state.

dsl_dir_init_fs_ss_count() is the tricky one. It looks like it could be called from dsl_dir_rename_check() it the feature is merely enabled. This looks like a bug to me -- it should only be called if the feature is active. Furthermore, dsl_dir_init_fs_ss_count() should assert that the feature is active.

The case I observed was during the dsl_dir_rename_check(). Renaming a dataset ended up in dsl_dir_init_fs_ss_count().

History

#1

Updated by Jerry Jelinek almost 6 years ago

  • Status changed from New to Closed

This is a dup of 4901

Also available in: Atom PDF