Opened 20 years ago

Closed 20 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@… 20 years ago.
videoout_quartz.cpp patch for NULL pointer

Download all attachments as: .zip

Change History (5)

by mythtv@…, 20 years ago

Attachment: videoout_quartz.cpp.patch​ added

videoout_quartz.cpp patch for NULL pointer

comment:1 by anonymous, 20 years ago

Type: defect → patch

comment:2 by Nigel, 20 years ago

Owner: changed from Isaac Richards to Nigel
Status: new → assigned

comment:3 by Nigel, 20 years ago

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 by Nigel, 20 years ago

Resolution: → fixed
Status: assigned → closed

(In [9439]) Closes #1539

Note: See TracTickets for help on using tickets.