Opened 7 years ago

Closed 7 years ago

#12233 closed Patch - Bug Fix (fixed)

Patch to check return values on seek - Fixes Coverity 746749 & 1073815

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


In libmythtv/ringbuffer.cpp Coverity reports two seeks with uncheck return values. While the comment says the seek(s) can never fail, static analysis suggests checking is goodness (and "impossible" failures do occur). The included patch checks the return values, and logs an error if the impossible occurs.

Fixes Coverity 746749 & 1073815

github ref: github git-am ref:

Change History (3)

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

Corrected patch to resolve static analysis detected potential issue with uninitialized value.

github ref: github git-am ref:

comment:2 Changed 7 years ago by stuartm

Milestone: unknown0.28
Owner: set to stuartm
Status: newaccepted

comment:3 Changed 7 years ago by Gary Buhrmaster <gary.buhrmaster@…>

Resolution: fixed
Status: acceptedclosed

In 8a37d0af58797a0f7a99bd751ecf59061c94a86a/mythtv:

Check the return values from seek requests

Fixes Coverity 746749 & 1073815
Fixes #12233

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

Note: See TracTickets for help on using tickets.