Mark Wielaard <mjw@redhat.com> writes:

> -      Dwarf_Attribute attributes[proto->nforms];
> +      Dwarf_Attribute *attributes;
> +      Dwarf_Attribute *attributesp = NULL;
> +      Dwarf_Attribute nattributes[8];
> +      if (unlikely (proto->nforms > 8))
> +	{
> +	  attributesp = malloc (sizeof (Dwarf_Attribute) * proto->nforms);
> +	  if (attributesp == NULL)
> +	    {
> +	      __libdw_seterrno (DWARF_E_NOMEM);
> +	      return -1;
> +	    }
> +	  attributes = attributesp;
> +	}
> +      else
> +	attributes = &nattributes[0];
> +
>        for (Dwarf_Word i = 0; i < proto->nforms; ++i)
>  	{

There's a return in this loop that needs free (attributesp) as well.

Thanks,
Petr