diff options
| author | sara hartse <[email protected]> | 2017-09-07 13:10:53 -0700 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2018-07-24 17:12:06 -0700 |
| commit | 473c976a0c4da84d8b49edaaf3d88a716d8dde2c (patch) | |
| tree | 6e4f211c26508f8ca50f1a2e1843e990e34d8a8b /module/zfs/dmu_traverse.c | |
| parent | 802b1a7b3b413486d4ff249f84c5e6cfd005ed55 (diff) | |
OpenZFS 9457 - libzfs_import.c:add_config() has a memory leak
A memory leak occurs on lines 209 and 213 because the config is not
freed in the error case. The interface to add_config() seems less than
ideal - it would be better if it copied any data necessary from the
config and the caller freed it.
Porting notes:
* This issue had already been resolved on Linux by adding the missing
calls to nvlist_free(). But we'll adopt the upstream fix to keep
the behavior of the code consistent.
Authored by: Sara Hartse <[email protected]>
Reviewed by: Matt Ahrens <[email protected]>
Reviewed by: Serapheim Dimitropoulos <[email protected]>
Reviewed by: Giuseppe Di Natale <[email protected]>
Reviewed by: George Melikov <[email protected]>
Approved by: Robert Mustacchi <[email protected]>
Ported-by: Brian Behlendorf <[email protected]>
OpenZFS-issue: https://illumos.org/issues/9457
OpenZFS-commit: https://github.com/openzfs/openzfs/commit/be86bb8a
Closes #7713
Diffstat (limited to 'module/zfs/dmu_traverse.c')
0 files changed, 0 insertions, 0 deletions