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.

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

Fix anon_inode descriptors leakage #249

Merged
merged 1 commit into from
Jul 9, 2014

Conversation

Witos
Copy link

@Witos Witos commented Jul 9, 2014

There is inode descriptor leakage in a current version.

Steps to reproduce:

  1. Create any observer.
  2. Close the observer.
  3. Check ls /proc/$pid/fd

There is: anon_inode:inotify present, which was opened by watchdog (checked with strace).
This pull request contains fix for that problem.

tamland added a commit that referenced this pull request Jul 9, 2014
Fix anon_inode descriptors leakage
@tamland tamland merged commit be2bff9 into gorakhargosh:master Jul 9, 2014
@tamland
Copy link
Collaborator

tamland commented Jul 9, 2014

Thank! Nice find

@vincyyou
Copy link

#249

CCP-Aporia pushed a commit to CCP-Aporia/watchdog that referenced this pull request Aug 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants