ACK: [PATCH] debugger: check for memory allocation failure on object Elements

[Alex Hung]

On 2019-11-27 6:07 a.m., Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> The allocation of Elements may fail, so add a null check to
> avoid any null pointer dereference faulting.
> 
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
>   src/acpica/source/components/debugger/dbconvert.c | 4 ++++
>   1 file changed, 4 insertions(+)
> 
> diff --git a/src/acpica/source/components/debugger/dbconvert.c b/src/acpica/source/components/debugger/dbconvert.c
> index 6b58bf5f..a9014b8f 100644
> --- a/src/acpica/source/components/debugger/dbconvert.c
> +++ b/src/acpica/source/components/debugger/dbconvert.c
> @@ -354,6 +354,10 @@ AcpiDbConvertToPackage (
>   
>       Elements = ACPI_ALLOCATE_ZEROED (
>           DB_DEFAULT_PKG_ELEMENTS * sizeof (ACPI_OBJECT));
> +    if (!Elements)
> +    {
> +        return (AE_NO_MEMORY);
> +    }
>   
>       This = String;
>       for (i = 0; i < (DB_DEFAULT_PKG_ELEMENTS - 1); i++)
> 
Acked-by: Alex Hung <alex.hung at canonical.com>