Commit e45d5579 authored by Matt Domsch's avatar Matt Domsch Committed by Linus Torvalds

[PATCH] EDD: fix too short array

Bingo...  edd_devices[] was too short.  When we keep more than 6
signatures, it overruns the end.  Also, I rewrote edd_num_devices to be
clearer about its goal. 

This patch is necessary even after the last edd.S patch was reverted.

Thanks to Christian Kujau for testing and narrowing it down enough to
debug this. 

It still doesn't explain why Christian's BIOS reports more devices than
he has, that's still up in the air, so don't re-apply the edd.S patch
just reverted. 
Signed-off-by: default avatarMatt Domsch <Matt_Domsch@dell.com>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent e9398715
...@@ -70,7 +70,7 @@ struct edd_attribute { ...@@ -70,7 +70,7 @@ struct edd_attribute {
static int edd_dev_is_type(struct edd_device *edev, const char *type); static int edd_dev_is_type(struct edd_device *edev, const char *type);
static struct pci_dev *edd_get_pci_dev(struct edd_device *edev); static struct pci_dev *edd_get_pci_dev(struct edd_device *edev);
static struct edd_device *edd_devices[EDDMAXNR]; static struct edd_device *edd_devices[EDD_MBR_SIG_MAX];
#define EDD_DEVICE_ATTR(_name,_mode,_show,_test) \ #define EDD_DEVICE_ATTR(_name,_mode,_show,_test) \
struct edd_attribute edd_attr_##_name = { \ struct edd_attribute edd_attr_##_name = { \
...@@ -728,9 +728,9 @@ edd_device_register(struct edd_device *edev, int i) ...@@ -728,9 +728,9 @@ edd_device_register(struct edd_device *edev, int i)
static inline int edd_num_devices(void) static inline int edd_num_devices(void)
{ {
return min_t(unsigned char, return max_t(unsigned char,
max_t(unsigned char, edd.edd_info_nr, edd.mbr_signature_nr), min_t(unsigned char, EDD_MBR_SIG_MAX, edd.mbr_signature_nr),
max_t(unsigned char, EDD_MBR_SIG_MAX, EDDMAXNR)); min_t(unsigned char, EDDMAXNR, edd.edd_info_nr));
} }
/** /**
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment