Re: [PATCH 1/2] drm/radeon: Using unsigned long instead of unsigned to fix possible overflow

2022-11-06 Thread Hanjun Guo
On 2022/11/5 2:31, Alex Deucher wrote: On Fri, Nov 4, 2022 at 6:05 AM Hanjun Guo wrote: VBIOSImageOffset in struct UEFI_ACPI_VFCT is ULONG (unsigned long), but it will be assigned to "unsigned offset", so use unsigned long instead of unsigned for the offset, to avoid possibl

[PATCH 1/2] drm/radeon: Using unsigned long instead of unsigned to fix possible overflow

2022-11-04 Thread Hanjun Guo
VBIOSImageOffset in struct UEFI_ACPI_VFCT is ULONG (unsigned long), but it will be assigned to "unsigned offset", so use unsigned long instead of unsigned for the offset, to avoid possible overflow. Signed-off-by: Hanjun Guo --- drivers/gpu/drm/radeon/radeon_bios.c | 2 +- 1 file

[PATCH 2/2] drm/radeon: Add the missed acpi_put_table() to fix memory leak

2022-11-04 Thread Hanjun Guo
fetch (v3)") Signed-off-by: Hanjun Guo --- drivers/gpu/drm/radeon/radeon_bios.c | 18 +++--- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/drivers/gpu/drm/radeon/radeon_bios.c b/drivers/gpu/drm/radeon/radeon_bios.c index 520d1d6..16730c1 100644 --- a/drivers/gpu/

[PATCH v2] ACPI: Fix acpi_evaluate_object() return value check

2014-01-17 Thread Hanjun Guo
On 2014-1-17 9:29, Yijing Wang wrote: > Fix acpi_evaluate_object() return value check, > shoud acpi_status not int. > > Signed-off-by: Yijing Wang > --- > > v1->v2: Add CC to the related subsystem MAINTAINERS. > > --- > drivers/gpu/drm/i915/intel_acpi.c | 13 +++-- > dri