On Tue, Oct 27, 2020 at 9:18 AM Jason A. Donenfeld <[email protected]> wrote:
...

> @@ -5721,16 +5720,18 @@ growwgdata(size_t by)
>         if (wg_aip != NULL)
>                 wg_aip = (void *)wg_interface + aip_offset;
>
> -       ret = (void *)wg_interface + wgdata.wgd_size - by;
> -       bzero(ret, by);
> -
> -       return ret;
> +       bzero((void *)wg_interface + wgdata.wgd_size - by, by);
>  }
>

Total side note, but this caught my eye as relying on the gcc extension to
do pointer arithmetic on "void *" pointers.  At least historically we've
avoided relying on that, which is easy in this case by just casting to
"char *" instead.

Philip Guenther

Reply via email to