Bug#455780: pommed: crashes when audio file doesn't exist

[Julien BLACHE]

Romain Beauxis <[EMAIL PROTECTED]> wrote:

Hi,

> pommed daemon crashes when changing volume, without log as it seems,
> when the WAV file given for beep doesn't exist.

Fixed in SVN, revision 425.

JB.

-- 
 Julien BLACHE - Debian & GNU/Linux Developer - <[EMAIL PROTECTED]> 
 
 Public key available on  - KeyID: F5D6 5169 
 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Bug#455780: pommed: crashes when audio file doesn't exist

[Romain Beauxis]

Le Tuesday 11 December 2007 21:35:00 Julien BLACHE, vous avez écrit :
> > pommed daemon crashes when changing volume, without log as it seems, when
> > the WAV file given for beep doesn't exist.
>
> Well actually I can't reproduce the crash. I'd like more details and
> the debug log (pommed -d) if you can reproduce it.
>
> I've found another issue in the beep code that I'll fix tomorrow, but
> it's probably harmless.

Here's the full log after pressing audio up and setting file to bla.wav:
KEY: audio up
Mixer volume: 169
Audio stepping +25 -> 194
DBus audioVolume: 194 169
Processing DBus requests
Checking messages
Done with DBus requests
Processing DBus requests
Checking messages
Done with DBus requests
pommed: mask_inline.h:160: snd_mask_leave:  l'assertion « val <= 64 » a 
échoué.

Then it crashes immediatly


Romain

Bug#455780: pommed: crashes when audio file doesn't exist

[Julien BLACHE]

Romain Beauxis <[EMAIL PROTECTED]> wrote:

Hi,

> pommed: mask_inline.h:160: snd_mask_leave:  l'assertion « val <= 64 » a 
> échoué.
>
> Then it crashes immediatly

It's an assertion failure, triggered by the bogus content of the
buffer passed to it.

That's the problem I found reading the code moments ago, except here
the assertion doesn't trigger.

I'll fix that up and I'll ask you to test the SVN afterwards to be
sure it's OK.

JB.

-- 
 Julien BLACHE <[EMAIL PROTECTED]>  |  Debian, because code matters more 
 Debian & GNU/Linux Developer|   
 Public key available on  - KeyID: F5D6 5169 
 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 

Bug#455780: pommed: crashes when audio file doesn't exist

[Julien BLACHE]

Romain Beauxis <[EMAIL PROTECTED]> wrote:

Hi,

> pommed daemon crashes when changing volume, without log as it seems, when the 
> WAV file given
> for beep doesn't exist.

Well actually I can't reproduce the crash. I'd like more details and
the debug log (pommed -d) if you can reproduce it.

I've found another issue in the beep code that I'll fix tomorrow, but
it's probably harmless.

JB.

-- 
 Julien BLACHE - Debian & GNU/Linux Developer - <[EMAIL PROTECTED]> 
 
 Public key available on  - KeyID: F5D6 5169 
 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Bug#455780: pommed: crashes when audio file doesn't exist

[Julien BLACHE]

Romain Beauxis <[EMAIL PROTECTED]> wrote:

Hi,

> pommed daemon crashes when changing volume, without log as it seems, when the 
> WAV file given
> for beep doesn't exist.

It should be OK as long as the default beep sound exists, otherwise,
indeed, there's a failure mode that slipped through.

JB.

-- 
 Julien BLACHE - Debian & GNU/Linux Developer - <[EMAIL PROTECTED]> 
 
 Public key available on  - KeyID: F5D6 5169 
 GPG Fingerprint : 935A 79F1 C8B3 3521 FD62 7CC7 CD61 4FD7 F5D6 5169 



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Bug#455780: pommed: crashes when audio file doesn't exist

[Romain Beauxis]

Package: pommed
Version: 1.13~dfsg-1
Severity: normal


Hi ! 

pommed daemon crashes when changing volume, without log as it seems, when the 
WAV file given
for beep doesn't exist.

Romain

-- System Information:
Debian Release: lenny/sid
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: amd64 (x86_64)

Kernel: Linux 2.6.23.1-mactel (SMP w/2 CPU cores; PREEMPT)
Locale: LANG=fr_FR, LC_CTYPE=fr_FR (charmap=ISO-8859-1)
Shell: /bin/sh linked to /bin/bash

Versions of packages pommed depends on:
ii  eject   2.1.5-6  ejects CDs and operates CD-Changer
ii  libasound2  1.0.15-2 ALSA library
ii  libaudiofile0   0.2.6-7  Open-source version of SGI's audio
ii  libc6   2.7-4GNU C Library: Shared libraries
ii  libconfuse0 2.5-3Library for parsing configuration 
ii  libdbus-1-3 1.1.2-1  simple interprocess messaging syst
ii  libsmbios1  0.13.10-1Provide access to (SM)BIOS informa
ii  zlib1g  1:1.2.3.3.dfsg-7 compression library - runtime

Versions of packages pommed recommends:
ii  dbus  1.1.2-1simple interprocess messaging syst

-- no debconf information



-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]