diff options
| author |   Laine Stump <laine@redhat.com> | 2011-12-07 13:52:01 -0500 |
|---|---|---|
| committer |   Laine Stump <laine@laine.org> | 2011-12-08 16:20:31 -0500 |
| commit | f21d49144fadf6fbe2fa0d846228a4f7e3713953 (patch) | |
| tree | ef645968825967b33eafc5332d84e25e8f41cb0d /bootstrap.conf | |
| parent | spec: fix logic bug in deciding to turn on cgconfig (diff) | |
| download | libvirt-f21d49144fadf6fbe2fa0d846228a4f7e3713953.tar.gz libvirt-f21d49144fadf6fbe2fa0d846228a4f7e3713953.tar.bz2 libvirt-f21d49144fadf6fbe2fa0d846228a4f7e3713953.zip | |
test: fix potential lock corruption in test driver
In some error situations, the function testDomainRestoreFlags() could
unlock the test driver mutex without first locking it. This patch
moves the lock operation earlier, so that it occurs before any
potential jump down to the unlock call.
I found this problem while auditing the test driver lock usage to
determine the cause of a hang while running the following test:
cd tests; while true; do printf x; ./undefine; done
This patch *does not* solve that problem, but we now understand its
actual source, and danpb is working on a patch.
Diffstat (limited to 'bootstrap.conf')
0 files changed, 0 insertions, 0 deletions