Do not manually free materials for shapes (#215)

* Do not manually free materials for shapes

* remove unnecessary junk

Cargo.toml

@@ -1,2 +1,3 @@
 [workspace]
+resolver = "2"
 members = ["physx", "physx-sys", "physx-sys/pxbind"]

physx/src/material.rs

@@ -140,7 +140,7 @@ pub trait Material: Class<physx_sys::PxMaterial> + UserData {
     }
 
     /// Set the restitution.
-    /// - Restitution must be in [0.0 ..= 1.0], values outside tyhe range are clamped.
+    /// - Restitution must be in [0.0 ..= 1.0], values outside the range are clamped.
     /// - A reitution of 0.0 minimizes bouncing, higher values mean more bounce.
     #[inline]
     fn set_restitution(&mut self, mut restitution: f32) {

physx/src/shape.rs

@@ -62,9 +62,6 @@ unsafe impl<U, M: Material> UserData for PxShape<U, M> {
 impl<U, M: Material> Drop for PxShape<U, M> {
     fn drop(&mut self) {
         unsafe {
-            for material in self.get_materials_mut() {
-                drop_in_place(material as *mut _);
-            }
             drop_in_place(self.get_user_data_mut());
             use crate::base::RefCounted;
             self.release();

physx/tests/bug-180.rs

@@ -0,0 +1,24 @@
+use physx::{base::RefCounted, prelude::*};
+
+type PxMaterial = physx::material::PxMaterial<()>;
+type PxShape = physx::shape::PxShape<(), PxMaterial>;
+
+#[test]
+fn test_double_free() {
+    let mut physics = PhysicsFoundation::<_, PxShape>::default();
+
+    let mut material = physics.create_material(0.5, 0.5, 0.6, ()).unwrap();
+    let geometry = PxBoxGeometry::new(1., 1., 1.);
+    let flags =
+        ShapeFlags::SceneQueryShape | ShapeFlags::SimulationShape | ShapeFlags::Visualization;
+
+    let shape = physics
+        .create_shape(&geometry, &mut [&mut material], false, flags, ())
+        .unwrap();
+
+    assert_eq!(material.get_reference_count(), 2);
+    drop(shape);
+
+    assert_eq!(material.get_reference_count(), 1);
+    drop(material);
+}