From 8cbac84c969fd53e3aec8e4853ba8f677421c6d3 Mon Sep 17 00:00:00 2001
From: itsmattkc <itsmattkc@gmail.com>
Date: Mon, 6 Apr 2020 17:07:38 +1000
Subject: [PATCH] samplebuffer: added an assert to ensure valid allocations
 occur

---
 app/codec/samplebuffer.cpp | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/app/codec/samplebuffer.cpp b/app/codec/samplebuffer.cpp
index 1c5053f33..1dde4fbae 100644
--- a/app/codec/samplebuffer.cpp
+++ b/app/codec/samplebuffer.cpp
@@ -1,4 +1,4 @@
-/***
+/***
 
   Olive - Non-Linear Video Editor
   Copyright (C) 2019 Olive Team
@@ -64,7 +64,7 @@ SampleBufferPtr SampleBuffer::CreateFromPackedData(const AudioRenderingParams &a
   const float* packed_data = reinterpret_cast<const float*>(bytes.constData());
 
   for (int i=0;i<total_samples;i++) {
-    int channel = i%audio_params.channel_count();
+    int channel = i % audio_params.channel_count();
     int index = i / audio_params.channel_count();
 
     buffer->data_[channel][index] = packed_data[i];
@@ -256,6 +256,8 @@ QByteArray SampleBuffer::toPackedData() const
 
 void SampleBuffer::allocate_sample_buffer(float ***data, int nb_channels, int nb_samples)
 {
+  Q_ASSERT(nb_samples > 0);
+
   *data = new float* [nb_channels];
 
   for (int i=0;i<nb_channels;i++) {