Divide Error Exception - GPT Partition Table Driver

[kallisti5]

Insert a usb stick which used to have a gpt map on it... random kdl's.

Seems likely related to #3064 (looks like the same error on the Intel partition system driver)

Screenshot attached

[kallisti5]

(In #9603) ah. This is a 100% duplicate of my issue :D

#9489 shows an identical error

[phoudoin]

I don't have yet a working Haiku system, but this one should be easy to fix in src/add-ons/kernel/partitioning_systems/gpt/efi_gpt.cpp:

static float
efi_gpt_identify_partition(int fd, partition_data* partition, void** _cookie)
{
+       if (partition->block_size == 0)
+           return -1;
+
	EFI::Header* header = new (std::nothrow) EFI::Header(fd,
		(partition->size - 1) / partition->block_size, partition->block_size);
	status_t status = header->InitCheck();
	if (status != B_OK) {
		delete header;
		return -1;
	}

Could some of you guys check it for me? Thanks.

[phoudoin]

A better fix will to pass the whole partition_data to Header constructor and move the check in InitCheck(), though.

[axeld]

I don't think the VFS should even call that method with an invalid partition. It's error prone to have these kinds of checks in every file and partitioning system.

[phoudoin]

Replying to axeld:

I don't think the VFS should even call that method with an invalid partition. It's error prone to have these kinds of checks in every file and partitioning system.

Somehow agree. Problem is where having a 0 block_size geometry should be considering an error and avoid rescanning partitions? KDiskDeviceManager::_CheckMediaStatus() ? KDiskDeviceManager::_ScanPartitions() ? partition system modules ?

Plus, are all partition systems necessary block-based? The DDM is pretty agnostic here, letting the partition modules use the device geometry information or not...

Due to the risk that any unchecked division in kernel space pose, I'm inclined to prefer paying the price of multiple checks than the one we pay right now, a KDL.

[kallisti5]

Replying to phoudoin:

Due to the risk that any unchecked division in kernel space pose, I'm inclined to prefer paying the price of multiple checks than the one we pay right now, a KDL.

This would be nice place for an assert check (and fixing the code that calls it to not call it with a 0 set.)

[TmTFx]

I have the same problem, but I have it on boot time and no usb keys nor hard disks mapped as gpt my mobo is an Asus M5A97 R2.0 with uefi.

[TmTFx]

I forgot to say that if I boot Haiku from cd-r drive the error won't happen and it boots flawlessy. I can even install it on hard drive, but if I reboot from the fresh installation it does the same error. I post the related screenshots

[jahaiku]

I get this exception only if "Eject when unmounting" in mount options is enabled.

[jessicah]

Fixed in hrev47468.