Opened 10 years ago

Closed 9 years ago

#8349 closed patch (fixed)

Better cleanup of ffmpeg process when web streaming is cancelled

Reported by: brandon.d.shields+mythtv@… Owned by: Rob Smith
Priority: minor Milestone: unknown
Component: Plugin - MythWeb Version: 0.23rc2
Severity: low Keywords:
Cc: Ticket locked: no

Description

I use lighttpd, and when the ffmpeg process is spawned by stream_flv.pl it is executed via a 'sh' process (see below). The method used in stream_flv.pl to cleanup ffmpeg is insufficient. The attached patch forces a new process group id, and uses that to terminate the ffmpeg process.

 5249 ?        00:00:00       lighttpd
 5846 ?        00:00:00         php-cgi <defunct>
 5849 ?        00:00:00         php-cgi <defunct>
 6006 ?        00:00:00         php-cgi
 6116 ?        00:00:01           php-cgi
 7019 ?        00:00:00         php-cgi
 7020 ?        00:00:00           php-cgi
 7022 ?        00:00:00         perl
 7024 ?        00:00:00           sh
 7025 ?        00:00:05             ffmpeg

Attachments (1)

stream_flv.pl.diff (953 bytes) - added by brandon.d.shields+mythtv@… 10 years ago.

Download all attachments as: .zip

Change History (2)

Changed 10 years ago by brandon.d.shields+mythtv@…

Attachment: stream_flv.pl.diff added

comment:1 Changed 9 years ago by Rob Smith

Resolution: fixed
Status: newclosed

(In [24926]) Fixes #8349, More agressive ffmpeg post stream cleanup

Note: See TracTickets for help on using tickets.