Hi!

On Tue, 22 Aug 2017 17:06:29 +0200 (CEST) Francois Gouget <fgou...@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.

Reply via email to