Skip to content
  • Pekka Paalanen's avatar
    tests: fix all leaks in devices-test · 591fa3b9
    Pekka Paalanen authored
    This fixes all ASan reported leaks in this test.
    
    This test program has several tests named *_multiple that just run
    another test function 30 times. Previously without cleanup all the
    created clients would be left lingering, but now they are torn down. Ths
    might cause a change in test behaviour, although that was never the
    intention:
    
    > It is intentional to run it so many times, but it is not intentional
    > to run a hundred clients at a time.  The problem is that currently we
    > have no destroy function for client.  However, the clients do not run
    > simultaneously but serially, so the effect should be the same as if
    > we'd destroy them (after the client finishes its body, it just 'is'
    > and does nothing until the process exits)
    
    - the original review discussion in
    https://lists.freedesktop.org/archives/wayland-devel/2015-March/020957.html
    
    
    
    The intention for the repeat testing is that as the Weston instance
    remains from test to another, each test needs to undo its changes to the
    devices. Failing to correcntly undo would accumulate devices.
    
    Signed-off-by: default avatarPekka Paalanen <pekka.paalanen@collabora.com>
    591fa3b9
Loading