Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix signal handler #29

Closed
elfring opened this issue Sep 25, 2015 · 3 comments
Closed

Fix signal handler #29

elfring opened this issue Sep 25, 2015 · 3 comments
Labels
Code Quality

Comments

@elfring
Copy link

elfring commented Sep 25, 2015

The function "printf" does not belong to the list of async-signal-safe functions.
I guess that a different design will be needed for your function "alarm_handler".
@tmcnag
Copy link
Contributor

tmcnag commented Sep 25, 2015

I'll defer to @jfrickson for the final decision, but since this handler is used to exit on a timeout I can't see the harm in using a synchronous function.

@jfrickson
Copy link
Contributor

Fixed in commit ce6c775

@elfring
Copy link
Author

elfring commented Nov 16, 2015

Are there a few update candidates left over?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Code Quality
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@elfring @tmcnag @jfrickson