[Pkg-lirc-maint] Bug#872375: lirc: irrecord segfaults when recording a button

[Alec Leamas]

Hi!

On Tue, 22 Aug 2017 17:06:29 +0200 (CEST) Francois Gouget 
<fgouget at free.fr> wrote:
 > On Fri, 18 Aug 2017, Gianfranco Costamagna wrote:
 >
 > > Hello,
 > >
 > > > Dear Maintainer,
 > > >
 > > > When trying to create a configuration file for my remote, irrecord
 > > > crashes when it gets to the 'Now hold down button Xxx' step:
 > > can you please try 0.10.0 from unstable?
 >
 > I tried with 0.10.0-2 but I'm still getting a segfault. Note that the
 > segfault happens *before* I press the button on the remote.
 >
 > # dpkg -l | grep lirc
 > ii liblirc-client0:amd64 0.10.0-2 amd64 infra-red remote control 
support - client library
 > ii liblirc0:amd64 0.10.0-2 amd64 Infra-red remote control support - 
Run-time libraries
 > ii lirc 0.10.0-2 amd64 Infra-red remote control support - daemons and 
utils
 >
 > As before I get:
 > # irrecord -H devinput -f -d /dev/input/event13 /tmp/foo.conf

This is actually not sane, and a prominent note is displayed when you 
start irrecord like this [1]

 > [...]
 > Hold down an arbitrary key
 > 
................................................................................
 > Found gap (115997 us)
 >
 > Please enter the name for the next button (press <ENTER> to finish
 > recording)
 > KEY_1
 >
 > Now hold down button "KEY_1".
 > Segmentation fault

A segmentation fault is obviously bad. However, I cannot reproduce this. 
For me the gap detection sort of works after a looong time, and the 
recording actually results in a file.  Havn't bothered to test it, this 
is just not the correct way to create a config file for the devinput driver.

I'm leaning to close this as not-a-bug. After all, the usecase is not 
supported. On top of that, the crash is not reproducible.

Perhaps the message [1] should be augmented with info on possible 
crashes, dunno

Cheers!
--alec


[1]
Usually you should not create a new config file for devinput
devices. LIRC is installed with a devinput.lircd.conf file which
is built for the current system which works with all remotes
supported by the kernel. There might be a need to update
this file so it matches the current kernel. For this, use the
lirc-make-devinput(1) script.

Press RETURN to continue.