[mythtv-commits] Ticket #11341: RemoteFile::Open() can leak MythSockets

[MythTV]

#11341: RemoteFile::Open() can leak MythSockets
----------------------------------+-----------------------------
 Reporter:  paulh <mythtv@…>      |          Owner:  cpinkham
     Type:  Bug Report - General  |         Status:  assigned
 Priority:  minor                 |      Milestone:  unknown
Component:  MythTV - General      |        Version:  Master Head
 Severity:  medium                |     Resolution:
 Keywords:                        |  Ticket locked:  0
----------------------------------+-----------------------------

Comment (by Jim Stichnoth <jstichnoth@…>):

 In [changeset:"ba75244a7f7e412bdb100b7741eaa6a6f0e6649b/mythtv"]:
 {{{
 #!CommitTicketReference repository="mythtv"
 revision="ba75244a7f7e412bdb100b7741eaa6a6f0e6649b"
 Use RemoteFile::isOpen() instead of Open() to check for success.

 Calling Open() was unnecessarily creating extra socket connections to
 the backend, and the original socket connections from calling Open()
 in the constructor were leaking and accumulating in both the frontend
 and backend (refs #11341).  Refs #11618
 }}}

--
Ticket URL: <http://code.mythtv.org/trac/ticket/11341#comment:2>
MythTV <http://www.mythtv.org>
MythTV Media Center