From 059ff0ab5446ab64d020bfdec5a16379374ae84c Mon Sep 17 00:00:00 2001
From: Michael Prokop <mika@debian.org>
Date: Fri, 29 Jan 2021 13:44:43 +0100
Subject: [PATCH] debian/coturn.postrm: fix check for dpkg-statoverride on
 package removal

This fixes a regression introduced in commit b9a4a8.
When trying to remove a dpkg-statoverride, we need to check,
whether it exists. Otherwise it's failing, as can be observed
via piuparts runs:

| dpkg: unrecoverable fatal error, aborting:
|  unknown system group 'turnserver' in statoverride file; the system group got removed
| before the override, which is most probably a packaging bug, to recover you
| can remove the override manually with dpkg-statoverride
| E: Sub-process /usr/bin/dpkg returned an error code (2)

While at it, also move the statoverride removal before the account removal.

Closes: #981330
---
 debian/coturn.postrm | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/debian/coturn.postrm b/debian/coturn.postrm
index a5b41e63..efda361a 100644
--- a/debian/coturn.postrm
+++ b/debian/coturn.postrm
@@ -6,6 +6,11 @@ TURNSERVER_USER=turnserver
 TURNSERVER_GROUP=turnserver
 
 if [ "$1" = "purge" ] ; then
+        TURNSERVER_CONFIG=/etc/turnserver.conf
+        if dpkg-statoverride --list "$TURNSERVER_CONFIG" >/dev/null 2>&1; then
+                dpkg-statoverride --remove "$TURNSERVER_CONFIG"
+        fi
+
         if getent passwd $TURNSERVER_USER >/dev/null; then
                 userdel $TURNSERVER_USER
         fi
@@ -15,11 +20,6 @@ if [ "$1" = "purge" ] ; then
 
         rm -f /var/lib/turn/turndb
         rmdir /var/lib/turn
-
-        TURNSERVER_CONFIG=/etc/turnserver.conf
-        if ! dpkg-statoverride --list "$TURNSERVER_CONFIG" >/dev/null 2>&1; then
-	    dpkg-statoverride --remove "$TURNSERVER_CONFIG"
-        fi
 fi