From fa2dd0dbd9eeb4456ee7e5440143385ce6d1febe Mon Sep 17 00:00:00 2001
From: Thijs Cadier <thijs@appsignal.com>
Date: Mon, 10 May 2021 23:10:05 +0200
Subject: [PATCH] Fix memory leak in Bscon::new

See https://github.com/appsignal/mongo-rust-driver/issues/66 for
details. Thanks to @austinjones for figuring this out.
---
 src/bsonc.rs | 14 +++++++++++++-
 1 file changed, 13 insertions(+), 1 deletion(-)

diff --git a/src/bsonc.rs b/src/bsonc.rs
index 3dc00b7..c4c42e5 100644
--- a/src/bsonc.rs
+++ b/src/bsonc.rs
@@ -16,7 +16,12 @@ pub struct Bsonc {
 
 impl Bsonc {
     pub fn new() -> Bsonc {
-        Bsonc::from_ptr(unsafe { bindings::bson_new() })
+        let inner: *const bindings::bson_t = unsafe { bindings::bson_new() };
+        assert!(!inner.is_null());
+        Bsonc {
+            inner: inner as *mut bindings::bson_t,
+            destroy_inner_on_drop: true,
+        }
     }
 
     /// Create a bsonc from a raw pointer. Does not run cleanup
@@ -111,6 +116,13 @@ impl Drop for Bsonc {
 
 #[cfg(test)]
 mod tests {
+    #[test]
+    fn test_bsonc_new() {
+        for _ in 0..100 {
+            let _ = super::Bsonc::new();
+        }
+    }
+
     #[test]
     fn test_bsonc_from_and_as_document() {
         let document = doc! { "key": "value" };