[mythtv-firehose] mythtv branch master updated by stuartm. v0.28-pre-1984-g5e756a5

Git Repo Owner noreply at mythtv.org
Sat Jul 26 08:19:38 UTC 2014


The branch, master has been updated on the
mythtv repository by gitolite user stuartm.
       via  5e756a5ea2fb34dfb3847b25f01149c9fa6a18cb (commit)
      from  e018bbe6be8018fb860c7b8ce54ea88a15d16338 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 5e756a5ea2fb34dfb3847b25f01149c9fa6a18cb
Author:    Stuart Morgan <smorgan at mythtv.org> at Sat, 26 Jul 2014 09:14:04 +0100
Committer: Stuart Morgan <smorgan at mythtv.org> at Sat, 26 Jul 2014 09:14:09 +0100
URL:       http://code.mythtv.org/cgit/mythtv/commit/?id=5e756a5ea2fb34dfb3847b25f01149c9fa6a18cb

Fix null pointer derefence in protocol error handling. Coverity 1228785
The PlaybackSock object has been deleted at this point and the pbs
pointer is NULL. This would have triggered a backend segfault if a
malformed MYTH_PROTO_VERSION query was received.

Instead use a pointer to the underlying socket (MythSocket)
which is what we really want ultimately. This adds a second
SendErrorResponse which handles MythSocket, so that either a
PlaybackSock or MythSocket pointer may be used.



-----------------------------------------------------------------------

Summary of changes:
 mythtv/programs/mythbackend/mainserver.cpp |   10 ++++++++--
 mythtv/programs/mythbackend/mainserver.h   |    3 ++-
 2 files changed, 10 insertions(+), 3 deletions(-)

-- 



More information about the mythtv-firehose mailing list