From: "Tomas Härdin" <git@haerdin.se> To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org> Subject: Re: [FFmpeg-devel] [PATCH 8/8] Make mime-type award a bonus probe score Date: Fri, 21 Feb 2025 10:15:56 +0100 Message-ID: <ed9d13deddb784fbbaf38aa281a5714dfe36c99f.camel@haerdin.se> (raw) In-Reply-To: <20250220210820.GK4991@pb2> tor 2025-02-20 klockan 22:08 +0100 skrev Michael Niedermayer: > On Thu, Feb 13, 2025 at 10:29:33PM +0100, Tomas Härdin wrote: > > Might be better to leverage afl-fuzz since it is more wily in its > > tricks to provoke different program behavior. Then exit(1) whenever > > the > > test program probes something incorrectly. For example you could > > start > > with a small, valid MPEG-PS file and have afl-fuzz generate > > slightly > > different versions of it that don't probe as such > > A real fuzzer will make every probe, probe incorrectly. Maybe i > misunderstood > what you suggested What I'm getting at is that we can use fuzzing to generate files that straddle the line between valid and not-valid, and then deliberately decide not to support them. Maybe this isn't as useful as I initially thought though. > what we want is that > 1. Random binary, random ascii, randon utf8 and intermediates do not > get > detected as any format (thats what probetest does) Right, probing /dev/urandom shouldn't really return meaningful scores. Except probetest is deterministic but whatever. > 2. that format A is detected more as format A than format B where B > != A > we and our users test this by simply using ffmpeg and fate A and B are not always cleanly separated. In fact some formats are deliberately designed to be polyglots. MXF is the obvious example. In these cases what is important is workflows, not formats. > having a really large corpus of real world odd files and test probing > on them > seems the "ideal" way to test probing to me Probably yeah. This is more of a philosophical point anyway I think. Anyway to get back to the point of this patch: I think MIME type is an excellent hint of user intent. The patch also doesn't break any existing tests. If I don't see any explicit objection I'll push this patch as well and we'll see how it plays out in the real world. If anyone objects then they need to at least provide samples, and preferably also explain their workflow. /Tomas _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
next prev parent reply other threads:[~2025-02-21 9:16 UTC|newest] Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top 2025-02-05 14:18 [FFmpeg-devel] [PATCH 1/8] avformat/http: Return EIO for prematurely broken connection Tomas Härdin 2025-02-05 14:19 ` [FFmpeg-devel] [PATCH 2/8] libavcodec/wmadec: Return AVERROR_INVALIDDATA on decoding errors Tomas Härdin 2025-02-05 16:28 ` Marth64 2025-02-05 14:20 ` [FFmpeg-devel] [PATCH 3/8] libavformat/flacdec: Export samples md5 as metadata Tomas Härdin 2025-02-06 15:07 ` Michael Niedermayer 2025-02-12 10:56 ` Tomas Härdin 2025-02-12 21:55 ` Michael Niedermayer 2025-02-13 11:41 ` Tomas Härdin 2025-02-13 11:52 ` Gyan Doshi 2025-02-12 11:14 ` Andreas Rheinhardt 2025-02-12 11:27 ` Tomas Härdin 2025-02-12 12:27 ` Andreas Rheinhardt 2025-02-12 13:32 ` Tomas Härdin 2025-02-05 14:20 ` [FFmpeg-devel] [PATCH 4/8] avformat/flacdec: Return correct error-codes on read-failure Tomas Härdin 2025-02-06 15:01 ` Michael Niedermayer 2025-02-05 14:21 ` [FFmpeg-devel] [PATCH 5/8] rtmp: Set correct message stream id when writing as server Tomas Härdin 2025-02-05 14:22 ` [FFmpeg-devel] [PATCH 6/8] GOL-1361: Remove invalid CTTS sample_offset check Tomas Härdin 2025-02-12 11:11 ` Tomas Härdin 2025-02-05 14:23 ` [FFmpeg-devel] [PATCH 7/8] avformat/mp3dec: Subtract known padding from duration Tomas Härdin 2025-02-05 14:24 ` [FFmpeg-devel] [PATCH 8/8] Make mime-type award a bonus probe score Tomas Härdin 2025-02-06 14:58 ` Michael Niedermayer 2025-02-12 11:03 ` Tomas Härdin 2025-02-12 22:03 ` Michael Niedermayer 2025-02-13 11:40 ` Tomas Härdin 2025-02-13 12:03 ` Michael Niedermayer 2025-02-13 21:29 ` Tomas Härdin 2025-02-20 21:08 ` Michael Niedermayer 2025-02-21 9:15 ` Tomas Härdin [this message] 2025-02-18 14:43 ` [FFmpeg-devel] [PATCH 1/8] avformat/http: Return EIO for prematurely broken connection Tomas Härdin
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=ed9d13deddb784fbbaf38aa281a5714dfe36c99f.camel@haerdin.se \ --to=git@haerdin.se \ --cc=ffmpeg-devel@ffmpeg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel This inbox may be cloned and mirrored by anyone: git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git # If you have public-inbox 1.1+ installed, you may # initialize and index your mirror using the following commands: public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \ ffmpegdev@gitmailbox.com public-inbox-index ffmpegdev Example config snippet for mirrors. AGPL code for this site: git clone https://public-inbox.org/public-inbox.git