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 10 months ago

Closed 10 months ago

Last modified 10 months ago

#11596 closed Patch - Bug Fix (fixed)

Patch for a feasible divide by zero in ClassicLogoDetector.cpp - fixes coverity 746806 and 746807

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

Description

Static analysis (coverity) detected a feasible (although not likely) code path such that the variables testEdges and testNotEdges could be zero, which would result in a divide by zero. Add in a check for the zero case.

Fixes coverity 746806 and 746807

github link:

https://github.com/garybuhrmaster/mythtv/commit/ddfad1f933a64f52f91aac72b741874edd1b5455

or, for the git-am format:

https://github.com/garybuhrmaster/mythtv/commit/ddfad1f933a64f52f91aac72b741874edd1b5455.patch

Attachments (0)

Change History (2)

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

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

In e63d70bc478fee4d79a0f5387e857078ff8ff067/mythtv:

[PATCH] Patch for feasible divide by zero

Static analysis (coverity) detected a feasible (although
not likely) code path such that the variables testEdges
and testNotEdges could be zero, which would result in
a divide by zero. Add in a check for the zero case.

Fixes coverity 746806 and 746807

Fixes #11596.

Signed-off-by: Chris Pinkham <cpinkham@…>

comment:2 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.