Mir servers crash if I press the side mouse button (thumb)

[Daniel van Vugt]

Mir servers crash if I press the side mouse button (thumb).

There are two of them on the side of my Intellimouse Explorer 3.0. The kernel calls them BTN_SIDE and BTN_EXTRA. And X calls them buttons 8 and 9.

When the crash happens I see in unity8.log:

[1455268365.782990] <ERROR> MirSurfaceAPI: Caught exception at client library boundary (in mir_surface_release): /build/mir-wMQ7xO/mir-0.19.2+16.04.20160210/src/client/rpc/stream_socket_transport.cpp(168): Throw in function virtual void mir::client::rpc::StreamSocketTransport::send_message(const std::vector<unsigned char>&, const std::vector<mir::Fd>&)
Dynamic exception type: boost::exception_detail::clone_impl<boost::exception_detail::error_info_injector<mir::socket_disconnected_error> >
std::exception::what: Failed to send message to server: Broken pipe
32, "Broken pipe"
*** Error in `unity8': free(): invalid size: 0x00007fcb4c03fe90 ***

Also happens with the Mir demo servers; they segfault but without a usable stack trace due to exception handling.

[Michał Sawicz (saviq)]

This seems similar to bug #1542305, even though that one has different log/.crash results.

[Launchpad Janitor (janitor)]

Status changed to 'Confirmed' because the bug affects multiple users.

[Daniel van Vugt (vanvugt)]

I'm on the fence about the importance of this bug so will err on the side of the worst case. It shouldn't be as easy as pressing one button (which some people will regularly hit by accident) to kill the server.

[Andreas Pokorny (andreas-pokorny)]

libinput treats the ranges BTN_MISC to BTN_GEAR_UP and BTN_DPAS to BTN_TRIGGER_HAPPY40 as pointer buttons since it does not support joystick/gamepads yet. So we get a pointer button down event... so there are probably two places to fix. libinput should grow to support joysticks, and mir should support more mouse buttons. And convert all the others as key events until we support joysticks/gamepads too.

[Andreas Pokorny (andreas-pokorny)]

And yes the problem is the conversion routine only deals with BTN_LEFT _RIGHT, _MIDDLE _BACK _FORWARD .. but not BTN_SIDE or BTN_EXTRA and BTN_TASK which are all in the same range of densely defined constants.

[Daniel van Vugt (vanvugt)]

Dropped importance. Not everyone feels this is a release blocker.

[PS Jenkins bot (ps-jenkins)]

Fix committed into lp:mir at revision None, scheduled for release in mir, milestone 0.21.0

[Daniel van Vugt (vanvugt)]

On second thoughts, the prerequisite commit r3317 that the fix r3324 depends on is huge. It can wait till Mir 0.21.

[Daniel van Vugt (vanvugt)]

The fix is definitely not in lp:mir/0.20 yet. And perhaps should not be due to the large prereq (see comment #8)

[Daniel van Vugt (vanvugt)]

Now it is...

Fix committed to lp:mir/0.20 at revision 3328, scheduled for release in Mir milestone 0.20.1

[Launchpad Janitor (janitor)]

This bug was fixed in the package mir - 0.20.1+16.04.20160225.1-0ubuntu1

---------------
mir (0.20.1+16.04.20160225.1-0ubuntu1) xenial; urgency=medium

  [ Kevin DuBois ]
  * New upstream release 0.20.1 (https://launchpad.net/mir/+milestone/0.20.1)
    - ABI summary:
      . No ABI changes in any libraries. Bugfix release only.
    - Bugs fixed:
      . On arale; top panel drop down menu is semi-transparent and items are
        unreadable. Accommodate system image device name change for arale
        device that broke android quirk detection (LP: #1549226)
      . Crash when side mouse buttons are pressed (LP: #1544878)

  [ CI Train Bot ]
  * No-change rebuild.

 -- Kevin DuBois <email address hidden> Thu, 25 Feb 2016 12:03:41 +0000