Skip to content

Commit

Permalink
Merge pull request #3844 from fluent/issue3627-plug-fd-leak
Browse files Browse the repository at this point in the history
child_process: Plug file descriptor leak
  • Loading branch information
ashie committed Aug 10, 2022
2 parents 0ee8322 + 788cf89 commit acfbb09
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions lib/fluent/plugin_helper/child_process.rb
Expand Up @@ -346,6 +346,9 @@ def child_process_execute_once(
if cb
cb.call(process_info.exit_status) rescue nil
end
process_info.readio&.close rescue nil
process_info.writeio&.close rescue nil
process_info.stderrio&.close rescue nil
end
thread[:_fluentd_plugin_helper_child_process_running] = true
thread[:_fluentd_plugin_helper_child_process_pid] = pid
Expand Down

0 comments on commit acfbb09

Please sign in to comment.