[mythtv] Ticket #6948: LiveTV browse mode starts on wrong channel

[Daniel Kristjansson]

On Mon, 2011-05-23 at 22:36 +0100, Simon Kenyon wrote:
> On 05/18/11 00:51, MythTV wrote:
> > #6948: LiveTV browse mode starts on wrong channel
> there are a whole clatter of different bug reports associated with this:
> #6208, #6948, #8211, #9025 (not at all sure about that one), #9137
Heh, I'd be greatful if someone checked if those were all fixed as well.

> one that i raised #9161 is not fixed by this change
> 
> but #9540 has a nice simple fix
> it is just one line
> could it be applied?
> 
> --- a/mythtv/libs/libmythtv/tvbrowsehelper.cpp
> +++ b/mythtv/libs/libmythtv/tvbrowsehelper.cpp
> @@ -80,6 +80,8 @@ bool TVBrowseHelper::BrowseStart(PlayerContext *ctx,
> bool skip_browse)
>                        ctx->playingInfo->GetScheduledStartTime(ISODate));
>          ctx->UnlockPlayingInfo(__FILE__, __LINE__);
> 
> +        m_chanid = ctx->playingInfo->GetChanID();
> +
>          if (!skip_browse)
>          {
>              locker.unlock();

No, this accesses the playing info outside the lock and it does nothing
anyway since that line of code exists inside the lock four lines
earlier..

-- Daniel