# HG changeset patch
# User Franz Glasner <fzglas.hg@dom66.de>
# Date 1661071740 -7200
# Node ID 3181a6b5758ebdb3d2ccd99d2f67dc48d89e0415
# Parent  a4fd3bcbbf1c58dccdbdeec9e03cf5da552d9c35
FIX: Using a pipeline within a function correctly with regard to return values:

The last pipeline element determines the status. Account for this.
So instead of a here-doc a pipeline can be used.

diff -r a4fd3bcbbf1c -r 3181a6b5758e sbin/fjail
--- a/sbin/fjail	Sun Aug 21 10:27:14 2022 +0200
+++ b/sbin/fjail	Sun Aug 21 10:49:00 2022 +0200
@@ -91,15 +91,16 @@
 
     _mountpoint="$1"
 
-    while IFS=' '$'\t' read -r _ds _mount _rest ; do
-        if [ "$_mount" = "$_mountpoint" ]; then
-            echo "${_ds}"
-            return 0
-        fi
-    done <<EOF__GDSFM
-$(mount -t zfs -p)
-EOF__GDSFM
-    return 1
+    mount -t zfs -p \
+    | {
+        while IFS=' '$'\t' read -r _ds _mount _rest ; do
+            if [ "$_mount" = "$_mountpoint" ]; then
+                echo "${_ds}"
+                return 0
+            fi
+        done
+        return 1
+    }
 }