feat: verify range proofs

Cargo.toml

@@ -61,7 +61,7 @@ firewood-triehash = { path = "triehash", version = "0.0.12" }
 
 # common dependencies
 aquamarine = "0.6.0"
-bytemuck = "1.23.1"
+bytemuck = { version = "1.23.1", features = ["must_cast"] }
 bytemuck_derive = "1.10.0"
 clap = { version = "4.5.41", features = ["derive"] }
 coarsetime = "0.1.36"
@@ -75,7 +75,7 @@ metrics-util = "0.20.0"
 nonzero_ext = "0.3.0"
 rand_distr = "0.5.1"
 sha2 = "0.10.9"
-smallvec = "1.15.1"
+smallvec = { version = "1.15.1", features = ["write", "union"] }
 test-case = "3.3.1"
 thiserror = "2.0.12"
 

ffi/firewood.go

@@ -152,8 +152,7 @@ func (db *Database) Propose(keys, vals [][]byte) (*Proposal, error) {
 		return nil, err
 	}
 
-	val := C.fwd_propose_on_db(db.handle, kvp)
-	return newProposal(db.handle, &val)
+	return getProposalFromProposalResult(C.fwd_propose_on_db(db.handle, kvp), db)
 }
 
 // Get retrieves the value for the given key. It always returns a nil error.
@@ -231,3 +230,25 @@ func (db *Database) Revision(root []byte) (*Revision, error) {
 
 	return &Revision{database: db, root: root}, nil
 }
+
+// Close releases the memory associated with the Database.
+//
+// This is not safe to call while there are any outstanding Proposals. All proposals
+// must be freed or committed before calling this.
+//
+// This is safe to call if the pointer is nil, in which case it does nothing. The
+// pointer will be set to nil after freeing to prevent double free. However, it is
+// not safe to call this method concurrently from multiple goroutines.
+func (db *Database) Close() error {
+	if db.handle == nil {
+		return nil
+	}
+
+	if err := getErrorFromVoidResult(C.fwd_close_db(db.handle)); err != nil {
+		return fmt.Errorf("unexpected error when closing database: %w", err)
+	}
+
+	db.handle = nil // Prevent double free
+
+	return nil
+}