Opened 18 years ago

Closed 18 years ago

#1539 closed patch (fixed)

NULL Pointer in videoout_quartz.cpp

Reported by: mythtv@… Owned by: Nigel
Priority: minor Milestone: 0.20
Component: mythtv Version: head
Severity: low Keywords:
Cc: Ticket locked: no

Description

We are passing in a NULL to DecompressSequenceFrameWhen? for the parameter that is supposed to be a pointer to the storage for the return flags. This doesn't seem to be causing any problems but according to the docs it is incorrect. The enclosed patch fixes this problem.

Attachments (1)

videoout_quartz.cpp.patch (802 bytes) - added by mythtv@… 18 years ago.
videoout_quartz.cpp patch for NULL pointer

Download all attachments as: .zip

Change History (5)

Changed 18 years ago by mythtv@…

Attachment: videoout_quartz.cpp.patch added

videoout_quartz.cpp patch for NULL pointer

comment:1 Changed 18 years ago by anonymous

Type: defectpatch

comment:2 Changed 18 years ago by Nigel

Owner: changed from Isaac Richards to Nigel
Status: newassigned

comment:3 Changed 18 years ago by Nigel

Thank you for this patch, Taoyama-san. I am very impressed that you found this. I am just compiling and testing it now (against 0-19-fixes).
You can review our code any time, Mino :-)

comment:4 Changed 18 years ago by Nigel

Resolution: fixed
Status: assignedclosed

(In [9439]) Closes #1539

Note: See TracTickets for help on using tickets.