Merge pull request #2375 from vyos/mergify/bp/sagitta/pr-2374

pmacct: T5232: Fixed socket parameters for trigger-packets (backport #2374)

1 files changed, 4 insertions, 3 deletions

diff --git a/src/system/uacctd_stop.py b/src/system/uacctd_stop.py
index 7fbac0566..a1b57335b 100755
--- a/src/system/uacctd_stop.py
+++ b/src/system/uacctd_stop.py
@@ -21,7 +21,7 @@
 # send some packets to pmacct to wake it up
 
 from argparse import ArgumentParser
-from socket import socket
+from socket import socket, AF_INET, SOCK_DGRAM
 from sys import exit
 from time import sleep
 
@@ -42,11 +42,12 @@ def stop_process(pid: int, timeout: int) -> None:
     uacctd.terminate()
 
     # create a socket
-    trigger = socket()
+    trigger = socket(AF_INET, SOCK_DGRAM)
 
     first_cycle: bool = True
     while uacctd.is_running() and timeout:
-        trigger.sendto(b'WAKEUP', ('127.0.254.0', 0))
+        print('sending a packet to uacctd...')
+        trigger.sendto(b'WAKEUP', ('127.0.254.0', 1))
         # do not sleep during first attempt
         if not first_cycle:
             sleep(1)