Opened 18 years ago

Closed 18 years ago

Last modified 18 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@…> 18 years ago.
check QFile::open()

Download all attachments as: .zip

Change History (4)

by Erik Hovland <erik@…>, 18 years ago

Attachment: libs_libmythtv_mpegrecorder.cpp-make-sure-device-file-desc-is-open.patch​ added

check QFile::open()

comment:1 by manwe <nettibug@…>, 18 years ago

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 by danielk, 18 years ago

Resolution: → fixed
Status: new → closed

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

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

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.