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

Last modified 10 months ago

#11552 closed Patch - Bug Fix (fixed)

Patch for static analysis detected (potential) uninitialized variable usage in mythcommflag

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 (scan-build) indicated that there is a feasible code path such that the variable currentFrameNumber could be used before assigning it a value. This patch initializes the variable to 0 at declaration.

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

Attachments (0)

Change History (2)

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

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

In c6ffdf44acb5238d7a1fe1a0b9713ea98e8b8159/mythtv:

Fix static analysis detected use without assignment

In ClassicCommDetector?.cpp, static analysis indicates that
the variable currentFrameNumber could be used without
previously assigning it a value if certain code feasible
code paths are taken. Assign it the value 0 at declaration.
(cherry picked from commit d2a021c29cf7d8bb4c13dd340654e61b4a16c8da)

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

Fixes #11552

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.