Opened 9 years ago

Closed 9 years ago

#8024 closed patch (fixed)

SendReceiveStringList fails after successful backend reconnect

Reported by: Jeff Lu <jll544@…> Owned by: Isaac Richards
Priority: minor Milestone: unknown
Component: MythTV - General Version: head
Severity: medium Keywords:
Cc: Ticket locked: no

Description

SendReceiveStringList? incorporates logic to reconnect the backend socket if it dies. However, it still fails after a successful reconnect because the strlist to be written to the backend was cleared by readStringList. Attached patch creates a separate variable for writeStringList so that it can be reused after reconnection.

Refs #7847

Attachments (1)

8024-strlist.patch (792 bytes) - added by Jeff Lu <jll544@…> 9 years ago.

Download all attachments as: .zip

Change History (2)

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

Attachment: 8024-strlist.patch added

comment:1 Changed 9 years ago by cpinkham

Resolution: fixed
Status: newclosed

(In [23723]) Copy the send QStringList in case we need to resend it. MythSocket::readStringList() clears the passed-in list, so if the read failed and we had to resend after a reconnect, we'd end up sending an empty list.

Closes #8024 using patch by Jeff Lu.

Note: See TracTickets for help on using tickets.