Modify
Warning Please read the Ticket HowTo before creating or commenting on a ticket. Failure to do so may cause your ticket to be rejected or result in a slower response.

Opened 11 months ago

Closed 10 months ago

Last modified 10 months ago

#11592 closed Patch - Bug Fix (fixed)

Patch to correct check for null pointer from function in audiooutputpulse.cpp - fixes coverity 700280

Reported by: Gary Buhrmaster <gary.buhrmaster@…> Owned by: jyavenard
Priority: minor Milestone: 0.27
Component: MythTV - Audio Output Version: Master Head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

In 1e1e550 code was added to call pa_channel_map_init_auto and check the result. This patch corrects that check for a null pointer.

Fixes coverity 700280 ("Operands don't affect result" because the previous comparison was always false)

Patch: https://github.com/garybuhrmaster/mythtv/commit/78ddf38354312cdeecccd6030becb78b86b797bf.patch

Attachments (0)

Change History (4)

comment:1 Changed 11 months ago by Gary Buhrmaster <gary.buhrmaster@…>

This patch should also fix coverity 700382 (I missed the second defect in report)

comment:2 Changed 11 months ago by Gary Buhrmaster <gary.buhrmaster@…>

Should also fix coverity 700298 (I missed the third defect in the report).

comment:3 Changed 10 months ago by Gary Buhrmaster <gary.buhrmaster@…>

  • Resolution set to fixed
  • Status changed from new to closed

In 98614122e6658c9b5c9c2d79c7756b4bffd5f538/mythtv:

Correct check for null pointer return

In 1e1e5500 code was added to call pa_channel_map_init_auto,
and check the result. This patch corrects that check for
a null pointer.

Fixes coverity 700280
(cherry picked from commit 78ddf38354312cdeecccd6030becb78b86b797bf)

Fixes #11592

Signed-off-by: Stuart Morgan <smorgan@…>

comment:4 Changed 10 months ago by paulh

  • Milestone changed from unknown to 0.27

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'new'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.