fix freeing m.key_values.all_deleted, when there were no deletions, u…

…se a single unsafe{} block for clarity
vlang · Aug 31, 2024 · 4acbdbd · 4acbdbd
1 parent 67a111a
commit 4acbdbd
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/vlib/builtin/map.v b/vlib/builtin/map.v
@@ -318,9 +318,13 @@ pub fn (mut m map) clear() {
 	m.even_index = 0
 	m.key_values.len = 0
 	m.key_values.deletes = 0
-	unsafe { free(m.key_values.all_deleted) }
-	unsafe { vmemset(m.key_values.keys, 0, m.key_values.key_bytes * m.key_values.cap) }
-	unsafe { vmemset(m.metas, 0, sizeof(u32) * (m.even_index + 2 + m.extra_metas)) }
+	unsafe {
+		if m.key_values.all_deleted != 0 {
+			free(m.key_values.all_deleted)
+		}
+		vmemset(m.key_values.keys, 0, m.key_values.key_bytes * m.key_values.cap)
+		vmemset(m.metas, 0, sizeof(u32) * (m.even_index + 2 + m.extra_metas))
+	}
 }
 
 @[inline]