Opened 10 years ago

Closed 10 years ago

#6588 closed patch (fixed)

frontend segfault if QUERY_LOAD or QUERY_MEMSTATS fail

Reported by: Jeff Lu <jll544@…> Owned by: stuartm
Priority: minor Milestone: 0.22
Component: MythTV - General Version: head
Severity: medium Keywords: remoteutil.cpp
Cc: Ticket locked: no

Description

If QUERY_LOAD or QUERY_MEMSTATS fail for some reason, backend sends an error message (a single element stringlist) in response.

Frontend assumes that it always receives a multi-element stringlist, resulting in segfault. This patch adds checks strlist.count() before parsing.

Attachments (1)

remoteutil.patch (791 bytes) - added by Jeff Lu <jll544@…> 10 years ago.

Download all attachments as: .zip

Change History (3)

Changed 10 years ago by Jeff Lu <jll544@…>

Attachment: remoteutil.patch added

comment:1 Changed 10 years ago by stuartm

Milestone: unknown0.22
Owner: changed from Isaac Richards to stuartm
Status: newaccepted

comment:2 Changed 10 years ago by stuartm

Resolution: fixed
Status: acceptedclosed

(In [21513]) Prevent a frontend segfault if certain protocol queries fail and return unexpected results. This does things a little differently from the supplied patch from Jeff Lu by creating a common error response which SendReceiveStringlist?() will look for and handle accordingly. Closes #6588

Note: See TracTickets for help on using tickets.