Opened 20 years ago
Closed 20 years ago
Last modified 18 years ago
#34 closed defect (fixed)
change 'system' to 'myth_system' in scheduler.cpp for properly spawning external commands
Reported by: | Owned by: | Isaac Richards | |
---|---|---|---|
Priority: | minor | Milestone: | unknown |
Component: | mythtv | Version: | head |
Severity: | low | Keywords: | |
Cc: | Ticket locked: | no |
Description
Hi all, This trivial patch makes the scheduler use the myth_system function rather than regular system when spawning external processes (eg. startup command, shutdown command etc).
myth_system correcly closes open file descriptors after forking and before execing. If a user script spawns other background processes, they too will inherit fd's without this change, this causes problems for me and prevents my system from suspending.
While I'm sure there are other solutions for me (my script could close the fd's itself), I think this is the way the code should be anyway.
Thanks, Dave
patch for #34