Re: [PATCH xserver] present: Free the fake_present OsTimerPtr

2016-08-15 Thread Adam Jackson
On Tue, 2016-08-09 at 09:49 +0900, Michel Dänzer wrote: > On 09/08/16 12:25 AM, Olivier Fourdan wrote: > > > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=97065 > > Signed-off-by: Olivier Fourdan > > Reviewed-by: Michel Dänzer remote: I: patch #103848 updated using rev de5291c04b05

Re: [PATCH xserver] present: Free the fake_present OsTimerPtr

2016-08-08 Thread Michel Dänzer
On 09/08/16 12:25 AM, Olivier Fourdan wrote: > Plug a leak in present_fake_queue_vblank() where the OsTimer would not > be freed. > > 492,608 (482,816 direct, 9,792 indirect) bytes in 15,088 blocks > are definitely lost in loss record 3,954 of 3,954 > at 0x4C2ABDE: malloc (in vgpreload_memche

[PATCH xserver] present: Free the fake_present OsTimerPtr

2016-08-08 Thread Olivier Fourdan
Plug a leak in present_fake_queue_vblank() where the OsTimer would not be freed. 492,608 (482,816 direct, 9,792 indirect) bytes in 15,088 blocks are definitely lost in loss record 3,954 of 3,954 at 0x4C2ABDE: malloc (in vgpreload_memcheck-amd64-linux.so) by 0x586B19: TimerSet (WaitFor.c:4