On Wed, Feb 21, 2018 at 02:17:37PM +0100, Hans Verkuil wrote: > On 02/21/18 13:49, Sakari Ailus wrote: > > On Mon, Feb 19, 2018 at 11:38:01AM +0100, Hans Verkuil wrote: > >> MEDIA_IOC_SETUP_LINK didn't zero the reserved field of the media_link_desc > >> struct. Do so in media_device_setup_link(). > >> > >> MEDIA_IOC_ENUM_LINKS didn't zero the reserved field of the media_links_enum > >> struct. Do so in media_device_enum_links(). > >> > >> Signed-off-by: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > > > Acked-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> > > > > If you haven't sent a pull request including your patch "media-device: zero > > reserved media_links_enum field", could you add it to the next version of > > this set (or the same pull request)? > > It's folded into this patch. It made no sense to have that in a separate patch. > Ah, indeed it is. Then please ignore my comment apart from the ack. -- Sakari Ailus e-mail: sakari.ailus@xxxxxx