shithub: sox

Info  •  Files  •  Log  •  Branches

delay effect: more syntax checking

Make sure that there are at most as many delay values given as there are
channels. (This could lead to strange effects otherwise, since the
excess arguments were considered when computing the maximum value.)

--- a/src/delay.c

+++ b/src/delay.c

@@ -74,6 +74,10 @@

   if (!p->max_arg)

     return SOX_EFF_NULL;

+  if (p->argc > effp->in_signal.channels) {

+    lsx_fail("too few input channels");

+    return SOX_EOF;

+  }

   if (effp->flow < p->argc)

     lsx_parsesamples(effp->in_signal.rate, p->argv[effp->flow], &p->buffer_size, 't');

   lsx_parsesamples(effp->in_signal.rate, p->max_arg, &max_delay, 't');