Reviewed by: Matt Ahrens <m...@delphix.com> Reviewed by: Serapheim Dimitropoulos <serapheim.dimi...@delphix.com>
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. Upstream bug: DLPX-49583 You can view, comment on, or merge this pull request online at: https://github.com/openzfs/openzfs/pull/614 -- Commit Summary -- * 9457 libzfs_import.c:add_config() has a memory leak -- File Changes -- M usr/src/lib/libzfs/common/libzfs_import.c (14) -- Patch Links -- https://github.com/openzfs/openzfs/pull/614.patch https://github.com/openzfs/openzfs/pull/614.diff -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/openzfs/openzfs/pull/614 ------------------------------------------ openzfs: openzfs-developer Permalink: https://openzfs.topicbox.com/groups/developer/discussions/T78dc7a649f386c31-Me92fa3859451672316095fc1 Delivery options: https://openzfs.topicbox.com/groups