[pulseaudio-tickets] [Bug 70096] New: sndfile-util.c crashes on 24 bit files

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=70096

          Priority: medium
            Bug ID: 70096
                CC: lennart at poettering.net
          Assignee: pulseaudio-bugs at lists.freedesktop.org
           Summary: sndfile-util.c crashes on 24 bit files
        QA Contact: pulseaudio-bugs at lists.freedesktop.org
          Severity: normal
    Classification: Unclassified
                OS: All
          Reporter: tanuk at iki.fi
          Hardware: Other
            Status: NEW
           Version: unspecified
         Component: core
           Product: PulseAudio

Reported in IRC:

<kiraank> Hi, I am using PA 2.1 and getting this error when I try to play some
wav file using paplay : "Code should not be reached at
pulsecore/sndfile-util.c:391, function pa_sndfile_readf_function(). Aborting.

I had a look at pa_sndfile_readf_function(). It handles a limited set of sample
formats. I then checked what kind of sample specs paplay can give that
function. It looks like the sample spec is coming from
pa_sndfile_read_sample_spec(), and that function can return format
PA_SAMPLE_S24NE, which is not handled by pa_sndfile_readf_function().

We couldn't figure out with kiraank what the sample format of the test file was
("file test.wav" didn't show the bit depth for some reason). I then tried to
create a test file with this command:

    parecord --device=alsa_output.pci-0000_00_1b.0.analog-stereo.monitor
--format=s24le > test.wav

but that crashed too, in a similar way but not exactly the same:

    Code should not be reached at pulsecore/sndfile-util.c:413, function
pa_sndfile_writef_function(). Aborting.
    Aborted

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/pulseaudio-bugs/attachments/20131003/56b766c0/attachment.html>