Opened 14 years ago

Closed 14 years ago

Last modified 13 years ago

#34 closed defect (fixed)

change 'system' to 'myth_system' in scheduler.cpp for properly spawning external commands

Reported by: david.collett@… 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

Attachments (1)

scheduler.diff (1.3 KB) - added by david.collett@… 14 years ago.
patch for #34

Download all attachments as: .zip

Change History (2)

Changed 14 years ago by david.collett@…

Attachment: scheduler.diff added

patch for #34

comment:1 Changed 14 years ago by jdonavan

Resolution: fixed
Status: newclosed

(In [6762]) Patch from David Collett to use myth_system in place of system calls in the scheduler.

This closes #34

Note: See TracTickets for help on using tickets.