From e746ce11899b6ad2ba46643ccc038e758f5912cd Mon Sep 17 00:00:00 2001
From: Hans-Christoph Steiner <eighthave@users.sourceforge.net>
Date: Sat, 29 Oct 2011 18:58:39 +0000
Subject: fixed crasher bug where it wasn't checking whether the arguments were
 in bounds when set by an inlet, fixes bug #3429877
 https://sourceforge.net/tracker/?func=detail&atid=478070&aid=3429877&group_id=55736

svn path=/trunk/externals/smlib/; revision=15680
---
 prevl.c | 27 +++++++++++++++++----------
 1 file changed, 17 insertions(+), 10 deletions(-)

(limited to 'prevl.c')

diff --git a/prevl.c b/prevl.c
index 2ba39ce..d43a2df 100644
--- a/prevl.c
+++ b/prevl.c
@@ -14,6 +14,7 @@ typedef struct _prevl
 	float *m_buffer;  // circular buffer
 } t_prevl;
 
+static void prevl_set(t_prevl *x, t_float lo, t_float hi, t_float size);
 
 static void prevl_perform_float(t_prevl *x, t_float f)
 {
@@ -30,6 +31,7 @@ static void prevl_bang(t_prevl *x)
 	t_atom *ap,*app;
 	float *buffer, *bp;
 
+    prevl_set(x, x->m_lo, x->m_hi, x->m_buffer_size);
 	lo=(int)x->m_lo;
 	hi=(int)x->m_hi;
 
@@ -94,37 +96,42 @@ static void prevl_set(t_prevl *x, t_float lo, t_float hi, t_float size)
 	if (size<1)
 	{
 		size=1;
-		post("prevl: size is minimum 1...");
+		logpost(x, 2, "[prevl] size is minimum 1");
 	}
 	if (hi>size)
 	{
-		post("prevl: higher bound cannot be higher than the buffer size...");	
+		logpost(x, 2, "[prevl] higher bound (%g) cannot be greater than the buffer size (%g)",
+                hi, size);	
 		hi=size;
 	}
 	if (lo<0)
 	{
-		post("prevl: lower bound cannot be negative...");	
+		logpost(x, 2, "[prevl] lower bound cannot be negative");	
 		lo=0;
 	}
 	if (hi<1)
 	{
-		post("prevl: higher bound cannot be smaller than one...");	
+		logpost(x, 2, "[prevl] higher bound (%g) cannot be smaller than 1", hi);
 		hi=1;
 	}
 	if (hi<=lo)
 	{
-		post("prevl: higher bound must be higher than lower bound...");	
+		logpost(x, 2, "[prevl] higher bound (%g) must be greater than lower bound (%g)",
+                hi, lo);	
 		lo=hi-1.0f;
 	}
 
-	freebytes(x->m_buffer, x->m_buffer_size);
 
 	x->m_hi=(float)((int)hi);
 	x->m_lo=(float)((int)lo);
-	x->m_buffer_size=(int)size;
-    x->m_buffer = (float*)getbytes(sizeof(float)*x->m_buffer_size);
-	prevl_clear(x);
-	x->m_buffer_index=0;
+    if (x->m_buffer_size != size)
+    {
+        x->m_buffer_size=(int)size;
+        freebytes(x->m_buffer, x->m_buffer_size);
+        x->m_buffer = (float*)getbytes(sizeof(float)*x->m_buffer_size);
+        prevl_clear(x);
+        x->m_buffer_index=0;
+    }
 }
 
 static void *prevl_new(t_float lo, t_float hi, t_float size)
-- 
cgit v1.2.1