[libc-commits] [libc] [libc] Fix memory leak in MPFRWrapper cospif with MPFR pre 4.2. (PR #114415)

via libc-commits libc-commits at lists.llvm.org
Thu Oct 31 08:15:02 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libc

Author: None (lntue)

<details>
<summary>Changes</summary>



---
Full diff: https://github.com/llvm/llvm-project/pull/114415.diff


1 Files Affected:

- (modified) libc/utils/MPFRWrapper/MPFRUtils.cpp (+2-1) 


``````````diff
diff --git a/libc/utils/MPFRWrapper/MPFRUtils.cpp b/libc/utils/MPFRWrapper/MPFRUtils.cpp
index 60e4abadb5e3c8..4fa6d2f9291993 100644
--- a/libc/utils/MPFRWrapper/MPFRUtils.cpp
+++ b/libc/utils/MPFRWrapper/MPFRUtils.cpp
@@ -262,6 +262,7 @@ class MPFRNumber {
 
       int d = mpz_tstbit(integer, 0);
       mpfr_set_si(result.value, d ? -1 : 1, mpfr_rounding);
+      mpz_clear(integer);
       return result;
     }
 
@@ -271,7 +272,7 @@ class MPFRNumber {
     mpfr_cos(result.value, value_pi.value, mpfr_rounding);
 
     return result;
-#endif
+// #endif
   }
 
   MPFRNumber erf() const {

``````````

</details>


https://github.com/llvm/llvm-project/pull/114415


More information about the libc-commits mailing list