Re: memory leak in gpt.c (Fixed the correct way)

[Håkon Løvdal]

On 12/26/05, Harley D. Eades III <address@hidden> wrote:
> +static void
> +pth_free (GuidPartitionTableHeader_t* pth) {
> +       if (pth->Reserved2)
> +               ped_free (pth->Reserved2);
> +
> +       if (pth)
> +               ped_free (pth);
> +}

There is no point in the last "if (pth)" test; if pth is NULL the
program will crash
at the pth->Reserved2 reference. On the question of whether passing NULL
to a function like this should be valid or not I tend to think it
should not, and therefore
suggest adding PED_ASSERT(pth != NULL, return) at the beginning.

BR Håkon Løvdal