recording: reject a zero-length window

1 files changed, 11 insertions, 0 deletions

diff --git a/src/recording/mod.rs b/src/recording/mod.rs
index 4cd2891..e03df32 100644
--- a/src/recording/mod.rs
+++ b/src/recording/mod.rs
@@ -57,6 +57,9 @@ struct Inner {
 impl Recorder {
     pub fn new(sample_rate: u32, window_seconds: u64) -> Self {
         let samples_per_frame = (sample_rate as usize) / TICKS_PER_SEC;
+        // Defensive: callers validate config, but a zero-length ring would
+        // panic on `% len` in `mix_tick`.
+        let window_seconds = window_seconds.max(1);
         let window_samples = (sample_rate as usize) * (window_seconds as usize);
         // `head` advances by `samples_per_frame` and wraps mod `window_samples`,
         // so the window length must be a whole number of frames.
@@ -342,6 +345,14 @@ mod tests {
     }
 
     #[test]
+    fn zero_window_is_clamped_and_does_not_panic() {
+        let rec = Recorder::new(48_000, 0);
+        let spf = rec.samples_per_frame;
+        rec.mix_tick(&[vec![1i16; spf]]);
+        assert!(!rec.flush().is_empty());
+    }
+
+    #[test]
     fn single_speaker_values_survive() {
         let rec = Recorder::new(48_000, 1);
         let spf = rec.samples_per_frame;