[rpm] Fixed uninstall script

Some versions of busybox evaluate ~$u/x as / and expand the rm command into rm -fr / if the account has / as its home directory :O
monich · Jul 20, 2022 · 1f0b799 · 1f0b799
1 parent 3f37abe
commit 1f0b799
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/rpm/openrepos-mybackup.spec b/rpm/openrepos-mybackup.spec
@@ -42,8 +42,10 @@ desktop-file-install --delete-original \
 
 %postun
 if [ "$1" == 0 ] ; then
-  for u in $(getent passwd | cut -d: -f1); do
-    eval rm -fr ~$u/.local/share/openrepos-mybackup ||:
+  for d in $(getent passwd | cut -d: -f6) ; do
+    if [ "$d" != "" ] && [ "$d" != "/" ] && [ -d "$d/.local/share/openrepos-mybackup" ] ; then
+      rm -fr "$d/.local/share/openrepos-mybackup" ||:
+    fi
   done
 fi