> On Dec 15, 2015, at 1:01 AM, Julien Isorce <[email protected]> wrote: > > "image" is not ready yet since it will be set at > the end of the function by: *image = *img; > > Signed-off-by: Julien Isorce <[email protected]> > --- > src/gallium/state_trackers/va/image.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-)
lgtm but you should also add mesa list for this particular patch… Sean > > diff --git a/src/gallium/state_trackers/va/image.c > b/src/gallium/state_trackers/va/image.c > index 58c9ff7..ccc263f 100644 > --- a/src/gallium/state_trackers/va/image.c > +++ b/src/gallium/state_trackers/va/image.c > @@ -264,7 +264,7 @@ vlVaDeriveImage(VADriverContextP ctx, VASurfaceID > surface, VAImage *image) > img->image_id = handle_table_add(drv->htab, img); > > img_buf->type = VAImageBufferType; > - img_buf->size = image->data_size; > + img_buf->size = img->data_size; > img_buf->num_elements = 1; > > pipe_resource_reference(&img_buf->derived_surface.resource, > surfaces[0]->texture); > -- > 1.9.1 > > _______________________________________________ > Libva mailing list > [email protected] > http://lists.freedesktop.org/mailman/listinfo/libva _______________________________________________ Libva mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/libva
