On Mon, 2015-09-14 at 18:07 +0200, Peter Meerwald wrote: > From: Peter Meerwald <p.meerwald at bct-electronic.com> > > CID 1323599 > > if ioctl(SNDCTL_DSP_SETFMT) is successful, use the format > --- > src/modules/oss/oss-util.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/modules/oss/oss-util.c b/src/modules/oss/oss-util.c > index 42ee7f5..ad03ddc 100644 > --- a/src/modules/oss/oss-util.c > +++ b/src/modules/oss/oss-util.c > @@ -189,7 +189,7 @@ int pa_oss_auto_format(int fd, pa_sample_spec > *ss) { > } else > ss->format = PA_SAMPLE_U8; > } else > - ss->format = f == AFMT_S16_LE ? PA_SAMPLE_S16LE : > PA_SAMPLE_S16BE; > + ss->format = f; Is this actually correct? ss->format is a pa_sample_format_t but f is an OSS format enum type, which is why the old code was written that way . -- Arun