Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#11571 closed Patch - Bug Fix (fixed)

Patch to initialize chanid in mythtranscode (static analysis false positive)

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

Description

Static analysis (scan-build) detects that the variable chanid could be used without initialization. This appears to actually be a false positive, since the use is protected by the found_chanid variable, but initializing the variable should be benign, and closes the scan-build static analysis issue.

Patch: https://github.com/garybuhrmaster/mythtv/commit/49617bc0c174be8b58e37ded693aa8d037ba7975

Change History (2)

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

Owner: set to Gary Buhrmaster <gary.buhrmaster@…>
Resolution: fixed
Status: newclosed

In 4fdd1432684c070e45bbd7c889e876602c57a98d/mythtv:

Initialize chanid in mythtranscode

Static analysis (scan-build) detects that the variable
chanid could be used without initialization. This is
actually a false positive, since the use is protected
by the found_chanid variable, but initializing the
variable should be benign, and eliminates that static
analysis detected issue.
(cherry picked from commit 49617bc0c174be8b58e37ded693aa8d037ba7975)

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

Fixes #11571

comment:2 Changed 11 years ago by paulh

Milestone: unknown0.27
Note: See TracTickets for help on using tickets.