On Tuesday 01 September 2009 11:56:19 ext Laurent Pinchart wrote: > On Monday 31 August 2009 13:58:54 Tuukka.O Toivonen wrote: > > When user space queues a buffer using VIDIOC_QBUF, the kernel > > should set flags in struct v4l2_buffer as specified in the V4L2 > > documentation. > > You forgot your SoB line. My bad. > > + __u32 buffer_flags = b->flags; > > Is that safe ? What if userspace sets bogus flags ? What else userspace should expect than garbage in, garbage out? Now this sets and clears exactly what V4L2 spec says it should, should we clear all other flags also? What about other fields that are not mentioned in the VIDIOC_QBUF documentation for input buffers, like bytesused, timestamp, etc.? Should VIDIOC_QBUF clear those also? - Tuukka -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
