On Mon, May 01, 2017 at 05:49:53PM +0100, Sean Young wrote: >On Thu, Apr 27, 2017 at 10:34:03PM +0200, David Härdeman wrote: >> The device core infrastructure is based on the presumption that >> once a driver calls device_add(), it must be ready to accept >> userspace interaction. >> >> This requires splitting rc_setup_rx_device() into two functions >> and reorganizing rc_register_device() so that as much work >> as possible is performed before calling device_add(). >> > >With this patch applied, I'm no longer getting any scancodes from >my rc devices. > >David, please can you test your patches before submitting. I have to go >over them meticulously because I cannot assume you've tested them. I did test this patch and I just redid the tests, both with rc-loopback and with a mceusb receiver. I'm seeing scancodes on the input device as well as pulse-space readings on the lirc device in both tests. I did the tests with only this patch applied and the lirc-use-after-free (v3). What hardware did you test with? Meanwhile, I'll try rebasing my patches to the latest version of the media-master git tree and test again. -- David Härdeman
