add another array length check

missed in previous version.

Author: jnqnfe

pulse-binding/CHANGELOG.md

@@ -1,3 +1,7 @@
+# <unreleased>
+
+ * Added a length check to the stream restore `write()` method (missed in previous version).
+
 # 2.30.0 (April 19th, 2025)
 
  * Added `to_owned()` methods to a bunch of introspection objects (as requested back in 2021 in

pulse-binding/src/context/ext_stream_restore.rs

@@ -139,11 +139,13 @@ impl StreamRestore {
     ///
     /// The callback must accept a `bool`, which indicates success.
     ///
-    /// Panics if the underlying C function returns a null pointer.
+    /// Panics if the underlying C function returns a null pointer, or if the length of the array is
+    /// too long to be communicated to the C function.
     pub fn write<F>(&mut self, mode: proplist::UpdateMode, data: &[&Info],
         apply_immediately: bool, callback: F) -> Operation<dyn FnMut(bool)>
         where F: FnMut(bool) + 'static
     {
+        assert!(data.len() <= u32::MAX as usize);
         let cb_data = box_closure_get_capi_ptr::<dyn FnMut(bool)>(Box::new(callback));
         let ptr = unsafe {
             capi::pa_ext_stream_restore_write(self.context, mode, mem::transmute(data.as_ptr()),