/snap-confine

Use sanity timeouts around blocking operations #146

Merged
merged 1 commit into from Sep 17, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@zyga @jdstrand
@zyga
Collaborator

zyga commented Sep 14, 2016

This patch uses the sanity timeout mechanism around both locking
flock(2) calls and around the eventfd_read(2) call. This ensures that
even if the call blocks due to unknown bugs in the code, snap-confine
will die at within a predictable amount of time.

Signed-off-by: Zygmunt Krynicki zygmunt.krynicki@canonical.com
@zyga
Use sanity timeouts around blocking operations 
This patch uses the sanity timeout mechanism around both locking
flock(2) calls and around the eventfd_read(2) call. This ensures that
even if the call blocks due to unknown bugs in the code, snap-confine
will die at within a predictable amount of time.

Signed-off-by: Zygmunt Krynicki <zygmunt.krynicki@canonical.com>
1a5994f
@jdstrand
Contributor

jdstrand commented Sep 16, 2016

+1

@zyga zyga merged commit bbc9b54 into master Sep 17, 2016
1 check passed

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@zyga zyga deleted the enable-sanity-timeouts branch Sep 17, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment