e2fsprogs: fix blkid whole-disk scanning heuristics

PATCH New Eric Sandeen 1 participant 1 message Nov 29, 2007

Share & tools

Eric Sandeen

author

2007-11-29 19:31 UTC #0 lore raw

PATCH 1/1 e2fsprogs: fix blkid whole-disk scanning heuristics

The heuristics in blkid/devname.c probe_all() for scanning whole disks with no partitions assume that a device name with no digit on the end will always be present as a delineator, i.e.:

sda
sda1
sdb
sdc

In this case, when sdc is seen, it's the clue to go back and scan sdb.

however, for something like:

sda
sda1
sdb
loop0

this falls down, and sdb is never scanned.
(thanks to Karel Zak for pointing this out).

I believe the following patch will fix this up, but when messing with partition scanning, I don't mind a second opinion. :)

Thanks,

-Eric

---

Addresses-Red-Hat-Bugzilla: #400321

Signed-off-by Eric Sandeen <[email protected]>

lib/blkid/devname.c +15 -7

Index: e2fsprogs-1.40.2/lib/blkid/devname.c
===================================================================
@@ -490,10 +490,11 @@ static int probe_all(blkid_cache cache, 
 
 		DBG(DEBUG_DEVNAME, printf("read partition name %s\n", ptname));
 
-		/* Skip whole disk devs unless they have no partitions
-		 * If we don't have a partition on this dev, also
+		/* Skip whole disk devs unless they have no partitions.
+		 * If base name of device has changed, also
 		 * check previous dev to see if it didn't have a partn.
-		 * heuristic: partition name ends in a digit.
+		 * heuristic: partition name ends in a digit, & partition
+		 * names contain whole device name as substring.
 		 *
 		 * Skip extended partitions.
 		 * heuristic: size is 1

@@ -502,6 +503,8 @@ static int probe_all(blkid_cache cache, 
 		 */
 
 		lens[which] = strlen(ptname);
+
+		/* ends in a digit, clearly a partition, so check */
 		if (isdigit(ptname[lens[which] - 1])) {
 			DBG(DEBUG_DEVNAME,
 			    printf("partition dev %s, devno 0x%04X\n",

@@ -510,10 +513,15 @@ static int probe_all(blkid_cache cache, 
 			if (sz > 1)
 				probe_one(cache, ptname, devs[which], 0, 
 					  only_if_new);
-			lens[which] = 0;
-			lens[last] = 0;
-		} else if (lens[last] && strncmp(ptnames[last], ptname,
-						 lens[last])) {
+			lens[which] = 0;	/* mark as checked */
+		}
+
+		/*
+		 * If last was not checked because it looked like a whole-disk
+		 * dev, and the device's base name has changed,
+		 * check last as well.
+		 */
+		if (lens[last] && strncmp(ptnames[last], ptname, lens[last])) {
 			DBG(DEBUG_DEVNAME,
 			    printf("whole dev %s, devno 0x%04X\n",
 				   ptnames[last], (unsigned int) devs[last]));