From 8113fa522d223a2e6708c345a5f216cbe4a5e758 Mon Sep 17 00:00:00 2001 From: Yao Zongyou Date: Fri, 22 Dec 2017 16:44:31 +0800 Subject: [PATCH] erasure-code: use jerasure_free_schedule to properly free a schedule Signed-off-by: Yao Zongyou --- src/erasure-code/jerasure/ErasureCodeJerasure.cc | 8 ++++++++ src/erasure-code/jerasure/ErasureCodeJerasure.h | 7 +------ 2 files changed, 9 insertions(+), 6 deletions(-) diff --git a/src/erasure-code/jerasure/ErasureCodeJerasure.cc b/src/erasure-code/jerasure/ErasureCodeJerasure.cc index 0369dd87ee5c..ea76b22f0040 100644 --- a/src/erasure-code/jerasure/ErasureCodeJerasure.cc +++ b/src/erasure-code/jerasure/ErasureCodeJerasure.cc @@ -301,6 +301,14 @@ void ErasureCodeJerasureCauchy::prepare_schedule(int *matrix) schedule = jerasure_smart_bitmatrix_to_schedule(k, m, w, bitmatrix); } +ErasureCodeJerasureCauchy::~ErasureCodeJerasureCauchy() +{ + if (bitmatrix) + free(bitmatrix); + if (schedule) + jerasure_free_schedule(schedule); +} + // // ErasureCodeJerasureCauchyOrig // diff --git a/src/erasure-code/jerasure/ErasureCodeJerasure.h b/src/erasure-code/jerasure/ErasureCodeJerasure.h index be9abc5615fa..0b5bfb1c8a94 100644 --- a/src/erasure-code/jerasure/ErasureCodeJerasure.h +++ b/src/erasure-code/jerasure/ErasureCodeJerasure.h @@ -156,12 +156,7 @@ public: DEFAULT_M = "3"; DEFAULT_W = "8"; } - ~ErasureCodeJerasureCauchy() override { - if (bitmatrix) - free(bitmatrix); - if (schedule) - free(schedule); - } + ~ErasureCodeJerasureCauchy() override; void jerasure_encode(char **data, char **coding, -- 2.47.3