diff options
author | Joe Thornber <ejt@redhat.com> | 2018-06-08 13:40:53 +0100 |
---|---|---|
committer | Joe Thornber <ejt@redhat.com> | 2018-06-08 13:40:53 +0100 |
commit | d5da55ed85248adb066d293c2a1b863ce17d2779 (patch) | |
tree | e526dd2f773e3016a3acd3cb8e490ed11ba7c439 /lib/filters | |
parent | c78239d8605f333915543c0e0c3ccf4f4ef5ee8f (diff) | |
download | lvm2-d5da55ed85248adb066d293c2a1b863ce17d2779.tar.gz |
device_mapper: remove dbg_malloc.
I wrote dbg_malloc before we had valgrind. These days there's just
no need.
Diffstat (limited to 'lib/filters')
-rw-r--r-- | lib/filters/filter-composite.c | 11 | ||||
-rw-r--r-- | lib/filters/filter-fwraid.c | 5 | ||||
-rw-r--r-- | lib/filters/filter-internal.c | 5 | ||||
-rw-r--r-- | lib/filters/filter-md.c | 5 | ||||
-rw-r--r-- | lib/filters/filter-mpath.c | 5 | ||||
-rw-r--r-- | lib/filters/filter-partitioned.c | 5 | ||||
-rw-r--r-- | lib/filters/filter-persistent.c | 19 | ||||
-rw-r--r-- | lib/filters/filter-signature.c | 5 | ||||
-rw-r--r-- | lib/filters/filter-type.c | 5 | ||||
-rw-r--r-- | lib/filters/filter-usable.c | 11 |
10 files changed, 43 insertions, 33 deletions
diff --git a/lib/filters/filter-composite.c b/lib/filters/filter-composite.c index 8d9e06147..8691aebf3 100644 --- a/lib/filters/filter-composite.c +++ b/lib/filters/filter-composite.c @@ -13,6 +13,7 @@ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "base/memory/zalloc.h" #include "lib/misc/lib.h" #include "lib/filters/filter.h" #include "lib/device/device.h" @@ -53,8 +54,8 @@ static void _composite_destroy(struct dev_filter *f) for (filters = (struct dev_filter **) f->private; *filters; ++filters) (*filters)->destroy(*filters); - dm_free(f->private); - dm_free(f); + free(f->private); + free(f); } static int _dump(struct dev_filter *f, int merge_existing) @@ -85,7 +86,7 @@ struct dev_filter *composite_filter_create(int n, int use_dev_ext_info, struct d if (!filters) return_NULL; - if (!(filters_copy = dm_malloc(sizeof(*filters) * (n + 1)))) { + if (!(filters_copy = malloc(sizeof(*filters) * (n + 1)))) { log_error("Composite filters allocation failed."); return NULL; } @@ -93,9 +94,9 @@ struct dev_filter *composite_filter_create(int n, int use_dev_ext_info, struct d memcpy(filters_copy, filters, sizeof(*filters) * n); filters_copy[n] = NULL; - if (!(cft = dm_zalloc(sizeof(*cft)))) { + if (!(cft = zalloc(sizeof(*cft)))) { log_error("Composite filters allocation failed."); - dm_free(filters_copy); + free(filters_copy); return NULL; } diff --git a/lib/filters/filter-fwraid.c b/lib/filters/filter-fwraid.c index 62022b3d0..03c63b31d 100644 --- a/lib/filters/filter-fwraid.c +++ b/lib/filters/filter-fwraid.c @@ -12,6 +12,7 @@ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "base/memory/zalloc.h" #include "lib/misc/lib.h" #include "lib/filters/filter.h" @@ -96,14 +97,14 @@ static void _destroy(struct dev_filter *f) if (f->use_count) log_error(INTERNAL_ERROR "Destroying firmware RAID filter while in use %u times.", f->use_count); - dm_free(f); + free(f); } struct dev_filter *fwraid_filter_create(struct dev_types *dt __attribute__((unused))) { struct dev_filter *f; - if (!(f = dm_zalloc(sizeof(*f)))) { + if (!(f = zalloc(sizeof(*f)))) { log_error("Firmware RAID filter allocation failed"); return NULL; } diff --git a/lib/filters/filter-internal.c b/lib/filters/filter-internal.c index ebbee7ec0..85e04d49a 100644 --- a/lib/filters/filter-internal.c +++ b/lib/filters/filter-internal.c @@ -12,6 +12,7 @@ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "base/memory/zalloc.h" #include "lib/misc/lib.h" #include "lib/filters/filter.h" @@ -58,14 +59,14 @@ static void _destroy(struct dev_filter *f) if (f->use_count) log_error(INTERNAL_ERROR "Destroying internal filter while in use %u times.", f->use_count); - dm_free(f); + free(f); } struct dev_filter *internal_filter_create(void) { struct dev_filter *f; - if (!(f = dm_zalloc(sizeof(*f)))) { + if (!(f = zalloc(sizeof(*f)))) { log_error("md filter allocation failed"); return NULL; } diff --git a/lib/filters/filter-md.c b/lib/filters/filter-md.c index b9d3bc876..4fcbca939 100644 --- a/lib/filters/filter-md.c +++ b/lib/filters/filter-md.c @@ -13,6 +13,7 @@ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "base/memory/zalloc.h" #include "lib/misc/lib.h" #include "lib/filters/filter.h" @@ -138,14 +139,14 @@ static void _destroy(struct dev_filter *f) if (f->use_count) log_error(INTERNAL_ERROR "Destroying md filter while in use %u times.", f->use_count); - dm_free(f); + free(f); } struct dev_filter *md_filter_create(struct cmd_context *cmd, struct dev_types *dt) { struct dev_filter *f; - if (!(f = dm_zalloc(sizeof(*f)))) { + if (!(f = zalloc(sizeof(*f)))) { log_error("md filter allocation failed"); return NULL; } diff --git a/lib/filters/filter-mpath.c b/lib/filters/filter-mpath.c index a7dc12eb5..785b51875 100644 --- a/lib/filters/filter-mpath.c +++ b/lib/filters/filter-mpath.c @@ -12,6 +12,7 @@ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "base/memory/zalloc.h" #include "lib/misc/lib.h" #include "lib/filters/filter.h" #include "lib/activate/activate.h" @@ -265,7 +266,7 @@ static void _destroy(struct dev_filter *f) if (f->use_count) log_error(INTERNAL_ERROR "Destroying mpath filter while in use %u times.", f->use_count); - dm_free(f); + free(f); } struct dev_filter *mpath_filter_create(struct dev_types *dt) @@ -278,7 +279,7 @@ struct dev_filter *mpath_filter_create(struct dev_types *dt) return NULL; } - if (!(f = dm_zalloc(sizeof(*f)))) { + if (!(f = zalloc(sizeof(*f)))) { log_error("mpath filter allocation failed"); return NULL; } diff --git a/lib/filters/filter-partitioned.c b/lib/filters/filter-partitioned.c index a85e1821e..34151fd12 100644 --- a/lib/filters/filter-partitioned.c +++ b/lib/filters/filter-partitioned.c @@ -13,6 +13,7 @@ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "base/memory/zalloc.h" #include "lib/misc/lib.h" #include "lib/filters/filter.h" @@ -49,14 +50,14 @@ static void _partitioned_filter_destroy(struct dev_filter *f) if (f->use_count) log_error(INTERNAL_ERROR "Destroying partitioned filter while in use %u times.", f->use_count); - dm_free(f); + free(f); } struct dev_filter *partitioned_filter_create(struct dev_types *dt) { struct dev_filter *f; - if (!(f = dm_zalloc(sizeof(struct dev_filter)))) { + if (!(f = zalloc(sizeof(struct dev_filter)))) { log_error("Partitioned filter allocation failed"); return NULL; } diff --git a/lib/filters/filter-persistent.c b/lib/filters/filter-persistent.c index 8f6d584dd..643c34910 100644 --- a/lib/filters/filter-persistent.c +++ b/lib/filters/filter-persistent.c @@ -13,6 +13,7 @@ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "base/memory/zalloc.h" #include "lib/misc/lib.h" #include "lib/filters/filter.h" #include "lib/config/config.h" @@ -360,10 +361,10 @@ static void _persistent_destroy(struct dev_filter *f) log_error(INTERNAL_ERROR "Destroying persistent filter while in use %u times.", f->use_count); dm_hash_destroy(pf->devices); - dm_free(pf->file); + free(pf->file); pf->real->destroy(pf->real); - dm_free(pf); - dm_free(f); + free(pf); + free(f); } struct dev_filter *persistent_filter_create(struct dev_types *dt, @@ -374,14 +375,14 @@ struct dev_filter *persistent_filter_create(struct dev_types *dt, struct dev_filter *f = NULL; struct stat info; - if (!(pf = dm_zalloc(sizeof(*pf)))) { + if (!(pf = zalloc(sizeof(*pf)))) { log_error("Allocation of persistent filter failed."); return NULL; } pf->dt = dt; - if (!(pf->file = dm_strdup(file))) { + if (!(pf->file = strdup(file))) { log_error("Filename duplication for persistent filter failed."); goto bad; } @@ -393,7 +394,7 @@ struct dev_filter *persistent_filter_create(struct dev_types *dt, goto bad; } - if (!(f = dm_zalloc(sizeof(*f)))) { + if (!(f = zalloc(sizeof(*f)))) { log_error("Allocation of device filter for persistent filter failed."); goto bad; } @@ -414,10 +415,10 @@ struct dev_filter *persistent_filter_create(struct dev_types *dt, return f; bad: - dm_free(pf->file); + free(pf->file); if (pf->devices) dm_hash_destroy(pf->devices); - dm_free(pf); - dm_free(f); + free(pf); + free(f); return NULL; } diff --git a/lib/filters/filter-signature.c b/lib/filters/filter-signature.c index 3aa628453..d4b6ea346 100644 --- a/lib/filters/filter-signature.c +++ b/lib/filters/filter-signature.c @@ -13,6 +13,7 @@ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "base/memory/zalloc.h" #include "lib/misc/lib.h" #include "lib/filters/filter.h" @@ -64,14 +65,14 @@ static void _destroy(struct dev_filter *f) if (f->use_count) log_error(INTERNAL_ERROR "Destroying signature filter while in use %u times.", f->use_count); - dm_free(f); + free(f); } struct dev_filter *signature_filter_create(struct dev_types *dt) { struct dev_filter *f; - if (!(f = dm_zalloc(sizeof(*f)))) { + if (!(f = zalloc(sizeof(*f)))) { log_error("md filter allocation failed"); return NULL; } diff --git a/lib/filters/filter-type.c b/lib/filters/filter-type.c index 8ec302542..63d7c7047 100644 --- a/lib/filters/filter-type.c +++ b/lib/filters/filter-type.c @@ -13,6 +13,7 @@ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "base/memory/zalloc.h" #include "lib/misc/lib.h" #include "lib/filters/filter.h" @@ -36,14 +37,14 @@ static void _lvm_type_filter_destroy(struct dev_filter *f) if (f->use_count) log_error(INTERNAL_ERROR "Destroying lvm_type filter while in use %u times.", f->use_count); - dm_free(f); + free(f); } struct dev_filter *lvm_type_filter_create(struct dev_types *dt) { struct dev_filter *f; - if (!(f = dm_zalloc(sizeof(struct dev_filter)))) { + if (!(f = zalloc(sizeof(struct dev_filter)))) { log_error("LVM type filter allocation failed"); return NULL; } diff --git a/lib/filters/filter-usable.c b/lib/filters/filter-usable.c index 45c78324d..498b90c2a 100644 --- a/lib/filters/filter-usable.c +++ b/lib/filters/filter-usable.c @@ -12,6 +12,7 @@ * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ +#include "base/memory/zalloc.h" #include "lib/misc/lib.h" #include "lib/filters/filter.h" #include "lib/activate/activate.h" @@ -157,15 +158,15 @@ static void _usable_filter_destroy(struct dev_filter *f) if (f->use_count) log_error(INTERNAL_ERROR "Destroying usable device filter while in use %u times.", f->use_count); - dm_free(f->private); - dm_free(f); + free(f->private); + free(f); } struct dev_filter *usable_filter_create(struct dev_types *dt __attribute__((unused)), filter_mode_t mode) { struct dev_filter *f; - if (!(f = dm_zalloc(sizeof(struct dev_filter)))) { + if (!(f = zalloc(sizeof(struct dev_filter)))) { log_error("Usable device filter allocation failed"); return NULL; } @@ -173,9 +174,9 @@ struct dev_filter *usable_filter_create(struct dev_types *dt __attribute__((unus f->passes_filter = _passes_usable_filter; f->destroy = _usable_filter_destroy; f->use_count = 0; - if (!(f->private = dm_zalloc(sizeof(filter_mode_t)))) { + if (!(f->private = zalloc(sizeof(filter_mode_t)))) { log_error("Usable device filter mode allocation failed"); - dm_free(f); + free(f); return NULL; } *((filter_mode_t *) f->private) = mode; |