On Thu, Oct 8, 2020 at 10:10 AM Christian König <ckoenig.leichtzumer...@gmail.com> wrote: > > Instead of manually calculating the structure size. > > Signed-off-by: Christian König <christian.koe...@amd.com>
Reviewed-by: Daniel Vetter <daniel.vet...@ffwll.ch> > --- > drivers/dma-buf/dma-resv.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/dma-buf/dma-resv.c b/drivers/dma-buf/dma-resv.c > index 1c8f2581cb09..bb5a42b10c29 100644 > --- a/drivers/dma-buf/dma-resv.c > +++ b/drivers/dma-buf/dma-resv.c > @@ -63,7 +63,7 @@ static struct dma_resv_list *dma_resv_list_alloc(unsigned > int shared_max) > { > struct dma_resv_list *list; > > - list = kmalloc(offsetof(typeof(*list), shared[shared_max]), > GFP_KERNEL); > + list = kmalloc(struct_size(list, shared, shared_max), GFP_KERNEL); > if (!list) > return NULL; > > -- > 2.17.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel