-
-
Notifications
You must be signed in to change notification settings - Fork 966
Closed
tilt-dev/fsnotify
#7Description
Before reporting an issue, please ensure you are using the latest release of fsnotify.
Which operating system (GOOS) and version are you using?
Linux: CentOS7 x86_64
Please describe the issue that occurred.
A race happened when Fsnotify created pipe and epollfd, which didn't live with golang exec. It would lead to unintended fd inheritance. Please checkout https://github.com/golang/go/blob/c659be4dc862cdf14a24134f2cfc16fa81e6d84c/src/os/file_unix.go#L190
Are you able to reproduce the issue? Please provide steps to reproduce and a code sample if possible.
Simply insert codes like below after creating a fsnotify watcher
cmd := exec.Command("sleep", "1000")
cmd.Start()
and checkout /proc/${sleep_pid}/fd, there would be file descriptions inherited from the parent
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels