Opened 15 years ago
Closed 15 years ago
#6715 closed defect (fixed)
SIGABRT in RemoteEnocder::GetFrameRate()
Reported by: | Owned by: | danielk | |
---|---|---|---|
Priority: | minor | Milestone: | 0.22 |
Component: | MythTV - General | Version: | head |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
changeset:18098 introduced a possible error path assertion failure in RemoteEncoder::GetFrameRate()
at remoteencoder.cpp:167. Here is how the assertion is triggered:
RemoteEnocder::GetFrameRate()
createsstrlist
RemoteEnocder::GetFrameRate()
passesstrlist
toRemoteEnocder::SendReceiveStringList()
RemoteEnocder::SendReceiveStringList()
passesstrlist
toMythSocket::readStringList()
MythSocket::readStringList()
callsstrlist.clear()
MythSocket::readStringList()
fails in some way (e.g., times out) and returnsfalse
RemoteEnocder::SendReceiveStringList()
returnsfalse
- The variable
ok
inRemoteEnocder::GetFrameRate()
remainsfalse
, so line 167 is executed - Line 167 tries to access element 0 of
strlist
, butstrlist
is empty - Qt aborts because 0 is out of the range of valid indexes into
strlist
Attachments (1)
Change History (3)
Changed 15 years ago by
Attachment: | mythtv_assert_fix.patch added |
---|
comment:1 Changed 15 years ago by
Milestone: | unknown → 0.22 |
---|---|
Owner: | changed from Isaac Richards to danielk |
Status: | new → assigned |
comment:2 Changed 15 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
(In [20881]) Fixes #6715. Improve debugging output on GetFrameRate?() when backend socket handling dies on us.
Note: See
TracTickets for help on using
tickets.
possible fix