summaryrefslogtreecommitdiff
path: root/lib/filters
diff options
context:
space:
mode:
authorJoe Thornber <ejt@redhat.com>2018-06-08 13:40:53 +0100
committerJoe Thornber <ejt@redhat.com>2018-06-08 13:40:53 +0100
commitd5da55ed85248adb066d293c2a1b863ce17d2779 (patch)
treee526dd2f773e3016a3acd3cb8e490ed11ba7c439 /lib/filters
parentc78239d8605f333915543c0e0c3ccf4f4ef5ee8f (diff)
downloadlvm2-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.c11
-rw-r--r--lib/filters/filter-fwraid.c5
-rw-r--r--lib/filters/filter-internal.c5
-rw-r--r--lib/filters/filter-md.c5
-rw-r--r--lib/filters/filter-mpath.c5
-rw-r--r--lib/filters/filter-partitioned.c5
-rw-r--r--lib/filters/filter-persistent.c19
-rw-r--r--lib/filters/filter-signature.c5
-rw-r--r--lib/filters/filter-type.c5
-rw-r--r--lib/filters/filter-usable.c11
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;