Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#5140 closed defect (fixed)

QFile::open() of device should be checked for failure in MpegRecorder::StartRecording

Reported by: Erik Hovland <erik@…> Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: mythtv Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

QFile::open() can fail. It should be checked in case it does. MpegRecorder::StartRecording?() uses a QFile for the device file object.

Attachments (1)

libs_libmythtv_mpegrecorder.cpp-make-sure-device-file-desc-is-open.patch (925 bytes) - added by Erik Hovland <erik@…> 12 years ago.
check QFile::open()

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by Erik Hovland <erik@…>

check QFile::open()

comment:1 Changed 12 years ago by manwe <nettibug@…>

Could not confirm yet, but this might be a solution to recording, but no file shows up in system bug in #5260
Erratic behaviour and hard to diagnose...

  • Manwe

comment:2 Changed 12 years ago by danielk

Resolution: fixed
Status: newclosed

(In [17269]) Fixes #5140. Check for failure to reopen ivtv device after error.

comment:3 in reply to:  1 Changed 12 years ago by Manwe <nettibug@…>

Replying to manwe <nettibug@profnetti.fi>:

Could not confirm yet, but this might be a solution to recording, but no file shows up in system bug in #5260
Erratic behaviour and hard to diagnose...

  • Manwe


this does not fix #5260, that problem still exists

  • Manwe

Note: See TracTickets for help on using tickets.