Re: [Libusb-devel] OpenUSB Fixes

[Artem E. <ar...@gm...>]

Dear Michael,

I have now been staring at the following patch, but can't get my head around
what is its purpose...

 void openusb_free_raw_desc(uint8_t *buffer)
>  {
> -       if (buffer == NULL)
> -               return;
> +       if (buffer != NULL)
> +       {
> +               free(buffer);
> +               buffer = NULL;
> +       }
>
> -       free(buffer);
> +       return;
>  }
>
>
"buffer = NULL" doesn't have any consequence as buffer is a local pointer
variable (and there's a "return" following anyway).

"return" at the end of a procedure doesn't make sense either.

What am I missing?

-Artem