[mythtv] BUG: read of freed memory in themedmenu.cpp

Isaac Richards ijr at po.cwru.edu
Wed Sep 17 13:55:33 EDT 2003


On Wednesday 17 September 2003 10:29 am, Geoffrey Hausheer wrote:
> Another valgrind find...

Applied this and your previous change, thanks =)

> The patch below seems to fix the problem, but if there is any case where
> handleAction doesn't result in rebuilding the screen, then perhaps it
> sn't quite right (seem sto work well for me so far though).  I think the
> actual memory reset happens in parseMenu, which isn't always called from
> handleAction, but the patch doesn't seem to affect the other cases
> either, so I think it is OK.  Basically, I just invalidate lastButtun
> whenever the user hits 'Enter'.

I added a couple more places to invalidate it (hitting escape, when it has to 
scroll the button rows, etc).

Isaac


More information about the mythtv-dev mailing list