check return value of power_supply_register from Lan Tianyu (rhbz 772730)
This commit is contained in:
parent
28a83dc698
commit
9358096eaf
|
@ -0,0 +1,37 @@
|
|||
In the ac.c, lack check return value of power_supply_register().
|
||||
This may casue that acpi driver's add() ops was called successful
|
||||
while the device may be failed to be initalized. For example, some
|
||||
ugly bios may describe two ACADs in the same dsdt. They use the same
|
||||
name which will cause the second ACAD device can no be registered.
|
||||
And then power_supply_register() failed. But acpi driver's add() ops
|
||||
is called sucessfully. The acpi device also will receive acpi notification
|
||||
and cause oops.
|
||||
https://bugzilla.redhat.com/show_bug.cgi?id=772730
|
||||
|
||||
Signed-off-by: Lan Tianyu <tianyu.lan@intel.com>
|
||||
---
|
||||
drivers/acpi/ac.c | 4 +++-
|
||||
1 files changed, 3 insertions(+), 1 deletions(-)
|
||||
|
||||
diff --git a/drivers/acpi/ac.c b/drivers/acpi/ac.c
|
||||
index 6512b20..d1fcbc0 100644
|
||||
--- a/drivers/acpi/ac.c
|
||||
+++ b/drivers/acpi/ac.c
|
||||
@@ -292,7 +292,9 @@ static int acpi_ac_add(struct acpi_device *device)
|
||||
ac->charger.properties = ac_props;
|
||||
ac->charger.num_properties = ARRAY_SIZE(ac_props);
|
||||
ac->charger.get_property = get_ac_property;
|
||||
- power_supply_register(&ac->device->dev, &ac->charger);
|
||||
+ result = power_supply_register(&ac->device->dev, &ac->charger);
|
||||
+ if (result)
|
||||
+ goto end;
|
||||
|
||||
printk(KERN_INFO PREFIX "%s [%s] (%s)\n",
|
||||
acpi_device_name(device), acpi_device_bid(device),
|
||||
--
|
||||
1.7.6.rc2.8.g28eb
|
||||
|
||||
--
|
||||
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
|
||||
the body of a message to majordomo@vger.kernel.org
|
||||
More majordomo info at http://vger.kernel.org/majordomo-info.html
|
11
kernel.spec
11
kernel.spec
|
@ -54,7 +54,7 @@ Summary: The Linux kernel
|
|||
# For non-released -rc kernels, this will be appended after the rcX and
|
||||
# gitX tags, so a 3 here would become part of release "0.rcX.gitX.3"
|
||||
#
|
||||
%global baserelease 1
|
||||
%global baserelease 2
|
||||
%global fedora_build %{baserelease}
|
||||
|
||||
# base_sublevel is the kernel version we're starting with and patching
|
||||
|
@ -767,6 +767,9 @@ Patch22056: crypto-aesni-intel-fix-wrong-kfree-pointer.patch
|
|||
#rhbz 834318
|
||||
Patch22057: ACPICA-Fix-possible-fault-in-return-package-object-repair-code.patch
|
||||
|
||||
#rhbz 772730
|
||||
Patch22058: ACPI-AC-check-the-return-value-of-power_supply_register.patch
|
||||
|
||||
# END OF PATCH DEFINITIONS
|
||||
|
||||
%endif
|
||||
|
@ -1421,6 +1424,9 @@ ApplyPatch crypto-aesni-intel-fix-wrong-kfree-pointer.patch
|
|||
#rhbz 834318
|
||||
ApplyPatch ACPICA-Fix-possible-fault-in-return-package-object-repair-code.patch
|
||||
|
||||
#rhbz 772730
|
||||
ApplyPatch ACPI-AC-check-the-return-value-of-power_supply_register.patch
|
||||
|
||||
# END OF PATCH APPLICATIONS
|
||||
|
||||
%endif
|
||||
|
@ -2159,6 +2165,9 @@ fi
|
|||
# and build.
|
||||
|
||||
%changelog
|
||||
* Wed Jul 18 2012 Josh Boyer <jwboyer@redhat.com>
|
||||
- check return value of power_supply_register from Lan Tianyu (rhbz 772730)
|
||||
|
||||
* Mon Jul 16 2012 Josh Boyer <jwboyer@redhat.com> - 3.4.5-1
|
||||
- Linux v3.4.5
|
||||
|
||||
|
|
Loading…
Reference in New Issue