File util-linux-libblkid-cdrom-autoclose-2.patch of Package util-linux.27364

Overview Repositories Revisions Requests Users Attributes Meta

File util-linux-libblkid-cdrom-autoclose-2.patch of Package util-linux.27364

From dc30fd4383e57a0440cdb0e16ba5c4336a43b290 Mon Sep 17 00:00:00 2001
From: Karel Zak <kzak@redhat.com>
Date: Tue, 7 Jan 2020 16:48:34 +0100
Subject: [PATCH] libblkid: check for medium on CDMROMs probing

The commit 39f5af25982d8b0244000e92a9d0e0e6557d0e17 introduces
O_NONBLOCK to avoid the tray close on open(). The side effect is that
open() is successful when there is no medium.

This is usually no problem for standard tools because the next read()
will fail. Unfortunately, libblkid ignores I/O errors for (and only
for) CDROMs to support some crazy hybrid data+audio disks. The final
result is many I/O errors in system log when O_NONBLOCK is enabled.

This patch add CDROM_DRIVE_STATUS to stop probing when there is no
disk or when the tray is open.

Addresses: https://bugzilla.redhat.com/show_bug.cgi?id=1787973
Signed-off-by: Karel Zak <kzak@redhat.com>
---
 libblkid/src/probe.c | 8 ++++++++
 misc-utils/blkid.c   | 6 +++++-
 2 files changed, 13 insertions(+), 1 deletion(-)

Index: util-linux-2.28/libblkid/src/probe.c
===================================================================
--- util-linux-2.28.orig/libblkid/src/probe.c
+++ util-linux-2.28/libblkid/src/probe.c
@@ -847,8 +847,17 @@ int blkid_probe_set_device(blkid_probe p
 	else if (S_ISBLK(sb.st_mode) &&
 	    !blkid_probe_is_tiny(pr) &&
 	    blkid_probe_is_wholedisk(pr) &&
-	    ioctl(fd, CDROM_GET_CAPABILITY, NULL) >= 0)
+	    ioctl(fd, CDROM_GET_CAPABILITY, NULL) >= 0) {
+# ifdef CDROM_DRIVE_STATUS
+		switch (ioctl(fd, CDROM_DRIVE_STATUS, 0)) {
+		case CDS_TRAY_OPEN:
+		case CDS_NO_DISC:
+			errno = ENOMEDIUM;
+			goto err;
+		}
+# endif
 		pr->flags |= BLKID_FL_CDROM_DEV;
+	}
 #endif
 
 	DBG(LOWPROBE, ul_debug("ready for low-probing, offset=%"PRIu64", size=%"PRIu64"",
Index: util-linux-2.28/misc-utils/blkid.c
===================================================================
--- util-linux-2.28.orig/misc-utils/blkid.c
+++ util-linux-2.28/misc-utils/blkid.c
@@ -489,8 +489,12 @@ static int lowprobe_device(blkid_probe p
 		fprintf(stderr, "error: %s: %m\n", devname);
 		return BLKID_EXIT_NOTFOUND;
 	}
-	if (blkid_probe_set_device(pr, fd, offset, size))
+	errno = 0;
+	if (blkid_probe_set_device(pr, fd, offset, size)) {
+		if (errno)
+			warn(_("error: %s"), devname);
 		goto done;
+	}
 
 	if (chain & LOWPROBE_TOPOLOGY)
 		rc = lowprobe_topology(pr);

Places

File util-linux-libblkid-cdrom-autoclose-2.patch of Package util-linux.27364

Places