Project

General

Profile

Actions

Emulator Issues #8630

closed

SDL: Dolphin ignores SIGTERM and SIGINT

Added by flacs over 9 years ago.

Status:
Fixed
Priority:
Normal
Assignee:
Category:
Controls
% Done:

0%

Operating system:
N/A
Issue type:
Bug
Milestone:
Regression:
Yes
Relates to usability:
No
Relates to performance:
No
Easy:
Yes
Relates to maintainability:
No
Regression start:
Fixed in:

Description

What's the problem? Describe what went wrong in few words.
Dolphin does not terminate on SIGTERM or SIGINT.

What did you expect to happen instead?
Dolphin should terminate.

What steps will reproduce the problem?

  1. Start Dolphin.
  2. Run "killall dolphin-emu" or press Ctrl+C.
  3. Nothing happens.

Which versions of Dolphin did you test on?
master (4.0-6571, 97382ec90a8c960d12836aa5b979e70da40bb9f5)

What are your PC specifications?
Linux with SDL 2.0.3

Is there any other relevant information?
This issue is caused by SDL. SDL_INIT_JOYSTICK implies SDL_INIT_EVENTS, which installs signal handlers for SIGTERM and SIGINT and translates them into SDL_QUIT events. http://hg.libsdl.org/SDL/rev/0979c3e31211 (to be released in SDL 2.0.4) provides a way to prevent this:

SDL_SetHint(SDL_HINT_NO_SIGNAL_HANDLERS, "1");


Related issues 1 (0 open1 closed)

Blocked by Emulator - Emulator Issues #7486: SDL/Linux: segfault on second game launch if haptic compatible controller plugged in.Fixedphire

Actions
Actions #1

Updated by flacs over 9 years ago

  • Status changed from New to Work started
Actions #3

Updated by flacs over 9 years ago

  • Status changed from Work started to Fix pending
Actions #4

Updated by Buddybenj over 9 years ago

  • Status changed from Fix pending to Fixed
Actions #5

Updated by PEmu over 9 years ago

PR #2530 was reverted in #2565. Should this be still be marked as Fixed?

Actions #6

Updated by JosJuice over 9 years ago

  • Status changed from Fixed to Accepted
Actions #7

Updated by phire over 9 years ago

  • Status changed from Accepted to Work started

I should be removing SDL in a few days anyway, that will fix this issue.

Actions #8

Updated by phire over 9 years ago

  • Status changed from Work started to Fix pending
Actions #9

Updated by phire over 9 years ago

  • Status changed from Fix pending to Fixed
Actions

Also available in: Atom PDF