A mismatch of buffer types I and O and IOSpec is not checked for

[teodly]

Note that I'm a beginner to Rust so I may be wrong.

The Soxr::process function doesn't require that arguments contain references to slices containing the same data type that was specified in IOSpec::new. It means that libsoxr may accidentally read or write beyond allocated memory if it was created with IOSpec specifying 64-bit float and a 32-bit float slice is given as an argument.

Also, it doesn't support split (planar) channels while original libsoxr does. The problem is that it gets input and output lengths for libsoxr from input & output slices lengths. It won't work if the arguments contain references to slices of references to slices (similarly to libsoxr API which excepts array of arrays) because the outer slice's length is in such case the number of channels, not number of samples.

Both issues could be solved (if I think correctly) by making the whole Soxr type generic over input and output types and removing IOSpec argument from the constructor.

I won't be able to propose a pull request myself because I don't know Rust well enough and implementing it requires fluency in generic programming.

[lrbalt]

I need to look into this. Would be nice to create a testcase to confirm that a mismatch between datatype of the slice and the IOSpec will lead to write beyond allocated memory or if some check will prevent that

[lrbalt]

I've fixed the problem with split channels, see #7. Leaving this open for the issue of mismatch between IOSpec and type of buffer