[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