Le tridi 3 brumaire, an CCXXIII, Timothy Gu a écrit :
> Those URLs are very explicit, while `/dev/` can mean anything,
> including `/dev/stdin`.

Andrey quoted these bits of code to show that checking filename for NULL is
not necessary. I believe his proof is conclusive for that point.

wm4 :
> IMHO it should check whether it's really a video device.

Yes, it should. Do you intend to propose the patch that invokes stat() on
all supported operating systems, check that can not devolve into a security
issue in the context of probing, find the relevant list of major/minor
pairs and a way of testing the dynamically allocated ones soon, or are you
just bikeshedding.

Because if you are just bikeshedding, I believe the patch with a lower
score, EXTENSION instead of MAX maybe, would be fine.

Regards,

-- 
  Nicolas George

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to