Free allocated memory in test applications

pablodelara · pablodelara · commit bf18da677033 · 2025-06-11T12:17:43.000+01:00
Signed-off-by: Pablo de Lara &lt;pablo.de.lara.guarch@intel.com&gt;
diff --git a/crc/crc16_t10dif_copy_perf.c b/crc/crc16_t10dif_copy_perf.c
@@ -81,5 +81,10 @@ main(int argc, char *argv[])
         perf_print(start, (long long) TEST_LEN);
 
         printf("finish 0x%x\n", crc);
+
+        // Free allocated memory
+        aligned_free(src);
+        aligned_free(dst);
+
         return 0;
 }
diff --git a/crc/crc16_t10dif_copy_test.c b/crc/crc16_t10dif_copy_test.c
@@ -133,7 +133,6 @@ main(int argc, char *argv[])
         dst_raw = (u8 *) malloc(TEST_LEN);
         if (NULL == dst_raw) {
                 printf("alloc error: Fail");
-                aligned_free(src_raw);
                 return -1;
         }
         src = src_raw;
diff --git a/crc/crc16_t10dif_op_perf.c b/crc/crc16_t10dif_op_perf.c
@@ -114,5 +114,10 @@ main(int argc, char *argv[])
         perf_print(start, (long long) sizeof(blks->data) * NBLOCKS);
 
         printf("finish 0x%x\n", crc);
+
+        // Free allocated memory
+        aligned_free(blks);
+        aligned_free(blks_ext);
+
         return 0;
 }
diff --git a/crc/crc16_t10dif_perf.c b/crc/crc16_t10dif_perf.c
@@ -77,5 +77,7 @@ main(int argc, char *argv[])
         perf_print(start, (long long) TEST_LEN);
 
         printf("finish 0x%x\n", crc);
+
+        aligned_free(buf);
         return 0;
 }
diff --git a/crc/crc32_gzip_refl_perf.c b/crc/crc32_gzip_refl_perf.c
@@ -88,5 +88,6 @@ main(int argc, char *argv[])
 
         printf("finish 0x%x\n", crc);
 
+        aligned_free(buf);
         return 0;
 }
diff --git a/crc/crc32_ieee_perf.c b/crc/crc32_ieee_perf.c
@@ -77,5 +77,7 @@ main(int argc, char *argv[])
         perf_print(start, (long long) TEST_LEN);
 
         printf("finish 0x%x\n", crc);
+
+        aligned_free(buf);
         return 0;
 }
diff --git a/crc/crc32_iscsi_perf.c b/crc/crc32_iscsi_perf.c
@@ -77,5 +77,9 @@ main(int argc, char *argv[])
         perf_print(start, (long long) TEST_LEN);
 
         printf("finish 0x%x\n", crc);
+
+        // Free allocated memory
+        aligned_free(buf);
+
         return 0;
 }
diff --git a/crc/crc64_funcs_perf.c b/crc/crc64_funcs_perf.c
@@ -105,5 +105,8 @@ main(int argc, char *argv[])
                 printf("finish 0x%" PRIx64 "\n", crc);
         }
 
+        // Free allocated memory
+        aligned_free(buf);
+
         return 0;
 }
diff --git a/erasure_code/erasure_code_base_perf.c b/erasure_code/erasure_code_base_perf.c
@@ -174,5 +174,14 @@ main(int argc, char *argv[])
         perf_print(start, (long long) (TEST_LEN(m)) * (k + nerrs));
 
         printf("done all: Pass\n");
+
+        // Free allocated buffers
+        for (i = 0; i < m; i++) {
+                aligned_free(buffs[i]);
+        }
+        for (i = 0; i < (m - k); i++) {
+                aligned_free(temp_buffs[i]);
+        }
+
         return 0;
 }
diff --git a/erasure_code/erasure_code_base_test.c b/erasure_code/erasure_code_base_test.c
@@ -763,5 +763,16 @@ main(int argc, char *argv[])
         }
 
         printf("done EC tests: Pass\n");
+
+        for (i = 0; i < TEST_SOURCES; i++) {
+                aligned_free(buffs[i]);
+        }
+        for (i = 0; i < TEST_SOURCES; i++) {
+                aligned_free(temp_buffs[i]);
+        }
+        free(encode_matrix);
+        free(decode_matrix);
+        free(invert_matrix);
+        free(g_tbls);
         return 0;
 }
diff --git a/erasure_code/gf_vect_dot_prod_base_test.c b/erasure_code/gf_vect_dot_prod_base_test.c
@@ -295,5 +295,14 @@ main(int argc, char *argv[])
         }
 
         printf("done all: Pass\n");
+
+        // Free allocated memory
+        for (i = 0; i < TEST_SOURCES; i++) {
+                aligned_free(buffs[i]);
+        }
+        aligned_free(dest);
+        aligned_free(dest_ref);
+        aligned_free(temp_buff);
+
         return 0;
 }
diff --git a/examples/ec/ec_piggyback_example.c b/examples/ec/ec_piggyback_example.c
@@ -437,6 +437,18 @@ main(int argc, char *argv[])
         }
         printf(" } done all: Pass\n");
 
+        free(encode_matrix);
+        free(decode_matrix);
+        free(invert_matrix);
+        free(temp_matrix);
+        free(g_tbls);
+        for (i = 0; i < k; i++)
+                free(frag_ptrs[i]);
+        for (i = 0; i < p2; i++)
+                free(parity_ptrs[i]);
+        for (i = 0; i < p2; i++)
+                free(recover_outp[i]);
+
         return 0;
 }
 
diff --git a/examples/ec/ec_simple_example.c b/examples/ec/ec_simple_example.c
@@ -210,6 +210,22 @@ main(int argc, char *argv[])
         }
 
         printf(" } done all: Pass\n");
+
+        // Free allocated memory
+        free(encode_matrix);
+        free(decode_matrix);
+        free(invert_matrix);
+        free(temp_matrix);
+        free(g_tbls);
+
+        for (i = 0; i < m; i++) {
+                free(frag_ptrs[i]);
+        }
+
+        for (i = 0; i < p; i++) {
+                free(recover_outp[i]);
+        }
+
         return 0;
 }
 
diff --git a/mem/mem_zero_detect_perf.c b/mem/mem_zero_detect_perf.c
@@ -55,5 +55,6 @@ main(int argc, char *argv[])
         printf("mem_zero_detect_perf" TEST_TYPE_STR ": ");
         perf_print(start, (long long) TEST_LEN);
 
+        aligned_free(buf);
         return 0;
 }
diff --git a/raid/pq_gen_perf.c b/raid/pq_gen_perf.c
@@ -84,5 +84,8 @@ main(int argc, char *argv[])
         printf("pq_gen" TEST_TYPE_STR ": ");
         perf_print(start, (long long) TEST_MEM);
 
+        for (i = 0; i < TEST_SOURCES + 2; i++)
+                aligned_free(buffs[i]);
+
         return 0;
 }
diff --git a/raid/xor_example.c b/raid/xor_example.c
@@ -75,7 +75,6 @@ main(int argc, char *argv[])
                 ret = -1;
 exit:
         for (i = 0; i < TEST_SOURCES + 1; i++)
-                free(buffs[i]);
-
+                aligned_free(buffs[i]);
         return ret;
 }
diff --git a/raid/xor_gen_perf.c b/raid/xor_gen_perf.c
@@ -88,5 +88,8 @@ main(int argc, char *argv[])
         printf("xor_gen" TEST_TYPE_STR ": ");
         perf_print(start, (long long) TEST_MEM);
 
+        for (i = 0; i < TEST_SOURCES + 1; i++)
+                aligned_free(buffs[i]);
+
         return fail;
 }

Original file line number	Diff line number	Diff line change
`@@ -133,7 +133,6 @@ main(int argc, char *argv[])`
`133`	`133`	`dst_raw = (u8 *) malloc(TEST_LEN);`
`134`	`134`	`if (NULL == dst_raw) {`
`135`	`135`	`printf("alloc error: Fail");`
`136`		`- aligned_free(src_raw);`
`137`	`136`	`return -1;`
`138`	`137`	`}`
`139`	`138`	`src = src_raw;`
Original file line number	Diff line number	Diff line change
`@@ -77,5 +77,7 @@ main(int argc, char *argv[])`
`77`	`77`	`perf_print(start, (long long) TEST_LEN);`
`78`	`78`
`79`	`79`	`printf("finish 0x%x\n", crc);`
	`80`	`+`
	`81`	`+ aligned_free(buf);`
`80`	`82`	`return 0;`
`81`	`83`	`}`
Original file line number	Diff line number	Diff line change
`@@ -88,5 +88,6 @@ main(int argc, char *argv[])`
`88`	`88`
`89`	`89`	`printf("finish 0x%x\n", crc);`
`90`	`90`
	`91`	`+ aligned_free(buf);`
`91`	`92`	`return 0;`
`92`	`93`	`}`
Original file line number	Diff line number	Diff line change
`@@ -77,5 +77,9 @@ main(int argc, char *argv[])`
`77`	`77`	`perf_print(start, (long long) TEST_LEN);`
`78`	`78`
`79`	`79`	`printf("finish 0x%x\n", crc);`
	`80`	`+`
	`81`	`+ // Free allocated memory`
	`82`	`+ aligned_free(buf);`
	`83`	`+`
`80`	`84`	`return 0;`
`81`	`85`	`}`
Original file line number	Diff line number	Diff line change
`@@ -105,5 +105,8 @@ main(int argc, char *argv[])`
`105`	`105`	`printf("finish 0x%" PRIx64 "\n", crc);`
`106`	`106`	`}`
`107`	`107`
	`108`	`+ // Free allocated memory`
	`109`	`+ aligned_free(buf);`
	`110`	`+`
`108`	`111`	`return 0;`
`109`	`112`	`}`