ioctl crash in mpg123

ioctl crash in mpg123

[JonY]

[-- Attachment #1: Type: text/plain, Size: 681 bytes --]

Hi,

I am debugging a crash in Cygwin 1.7.29-2 ioctl but I am not familiar
with the OSS sounds API. I traced the ioctl calls and made this example,
is it supposed to work?

#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/ioctl.h>
#include <sys/soundcard.h>

int main(){
  int fmt = AFMT_MU_LAW;
  int chan = 0;
  int fd = open("/dev/dsp", O_WRONLY);
  ioctl(fd, SNDCTL_DSP_RESET, NULL);
  ioctl(fd, SNDCTL_DSP_RESET, NULL);
  ioctl(fd, SNDCTL_DSP_RESET, NULL);
  ioctl(fd, SNDCTL_DSP_SETFMT, &fmt);
  ioctl(fd, SNDCTL_DSP_STEREO, &chan); /* crashes here */
  close(fd);
  exit(0);
}


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 834 bytes --]

Re: ioctl crash in mpg123

[Christopher Faylor]

On Fri, May 02, 2014 at 07:16:25PM +0800, JonY wrote:
>Hi,
>
>I am debugging a crash in Cygwin 1.7.29-2 ioctl but I am not familiar
>with the OSS sounds API. I traced the ioctl calls and made this example,
>is it supposed to work?

If you're asking if Cygwin is supposed to SEGV in certain situations the
answer is "no".

This should be fixed in the next snapshot.

Thanks for the test case.

cgf

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

Re: ioctl crash in mpg123

[JonY]

[-- Attachment #1: Type: text/plain, Size: 530 bytes --]

On 5/2/2014 23:15, Christopher Faylor wrote:
> On Fri, May 02, 2014 at 07:16:25PM +0800, JonY wrote:
>> Hi,
>>
>> I am debugging a crash in Cygwin 1.7.29-2 ioctl but I am not familiar
>> with the OSS sounds API. I traced the ioctl calls and made this example,
>> is it supposed to work?
> 
> If you're asking if Cygwin is supposed to SEGV in certain situations the
> answer is "no".
> 
> This should be fixed in the next snapshot.
> 
> Thanks for the test case.
> 
> cgf
> 

Thanks for fixing this quickly!



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 834 bytes --]