]> git.ipfire.org Git - thirdparty/pdns.git/commitdiff
dnsdist: Fix a race in the async regression tests 13583/head
authorRemi Gacogne <remi.gacogne@powerdns.com>
Thu, 7 Dec 2023 14:05:43 +0000 (15:05 +0100)
committerRemi Gacogne <remi.gacogne@powerdns.com>
Fri, 8 Dec 2023 07:45:39 +0000 (08:45 +0100)
We need to make sure the listener is alive during the duration of
the test, and not destroyed by the garbage collector.

regression-tests.dnsdist/test_Async.py

index 34ebf018c90ba02e01c9f2669c6259fdad199de1..eda4a5903c33ea1c959e2585abd0bea903bcb018 100644 (file)
@@ -6,6 +6,7 @@ import sys
 import threading
 import unittest
 import dns
+import dns.message
 import doqclient
 
 from dnsdisttests import DNSDistTest, pickAvailablePort
@@ -459,8 +460,8 @@ class TestAsyncFFI(DNSDistTest, AsyncTests):
       end
     end
 
-    local asyncResponderEndpoint = newNetworkEndpoint('%s')
-    local listener = newNetworkListener()
+    asyncResponderEndpoint = newNetworkEndpoint('%s')
+    listener = newNetworkListener()
     listener:addUnixListeningEndpoint('%s', 0, gotAsyncResponse)
     listener:start()
 
@@ -588,8 +589,8 @@ class TestAsyncLua(DNSDistTest, AsyncTests):
       end
     end
 
-    local asyncResponderEndpoint = newNetworkEndpoint('%s')
-    local listener = newNetworkListener()
+    asyncResponderEndpoint = newNetworkEndpoint('%s')
+    listener = newNetworkListener()
     listener:addUnixListeningEndpoint('%s', 0, gotAsyncResponse)
     listener:start()