[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#804175: potential workaround



I added a check for extended partitions to 50mounted-tests. However it is rather ugly as it parses file -s $partition output and I do not know whether this test is valid in all circumstances. In any case I got rid of all those kernel errors with this.
diff --git a/os-probes/common/50mounted-tests b/os-probes/common/50mounted-tests
index 561163b..47da77f 100755
--- a/os-probes/common/50mounted-tests
+++ b/os-probes/common/50mounted-tests
@@ -4,7 +4,6 @@ set -e
 partition="$1"
 
 . /usr/share/os-prober/common.sh
-
 types="$(fs_type "$partition")" || types=NOT-DETECTED
 if [ "$types" = NOT-DETECTED ]; then
 	debug "$1 type not recognised; skipping"
@@ -20,6 +19,15 @@ elif [ "$types" = ntfs ]; then
 		types='ntfs-3g ntfs'
 	fi
 elif [ -z "$types" ]; then
+	if type file >/dev/null 2>&1 && \
+	   [ -n "${file_info="$(file -s $partition)"}" ]; then
+		case "$file_info" in
+			*"extended partition table"*)
+				debug "$partition is an extended partition; skipping"
+				exit 0
+				;;
+		esac
+	fi
 	if type cryptsetup >/dev/null 2>&1 && \
 	   cryptsetup luksDump "$partition" >/dev/null 2>&1; then
 		debug "$1 is a LUKS partition; skipping"

Reply to: