avformat: Fix AV1 RTP wrong log condition

Chris Hodges · ePirat · commit 68b105341c09 · 2025-04-03T16:08:41.000+02:00
Fixed warning about OBU count being wrong, which can only be
determined when the number of OBUs in the header is non-zero,
not the other way round.

Signed-off-by: Chris Hodges &lt;chris.hodges@axis.com&gt;
diff --git a/libavformat/rtpdec_av1.c b/libavformat/rtpdec_av1.c
@@ -373,7 +373,7 @@ static int av1_handle_packet(AVFormatContext *ctx, PayloadContext *data,
         }
         is_frag_cont = 0;
 
-        if (!rem_pkt_size && !num_obus && (num_obus != obu_cnt)) {
+        if (!rem_pkt_size && num_obus && (num_obus != obu_cnt)) {
             av_log(ctx, AV_LOG_WARNING, "AV1 aggregation header indicated %u OBU elements, was %u\n",
                    num_obus, obu_cnt);
         }

Original file line number	Diff line number	Diff line change
`@@ -373,7 +373,7 @@ static int av1_handle_packet(AVFormatContext ctx, PayloadContext data,`
`373`	`373`	`}`
`374`	`374`	`is_frag_cont = 0;`
`375`	`375`
`376`		`- if (!rem_pkt_size && !num_obus && (num_obus != obu_cnt)) {`
	`376`	`+ if (!rem_pkt_size && num_obus && (num_obus != obu_cnt)) {`
`377`	`377`	`av_log(ctx, AV_LOG_WARNING, "AV1 aggregation header indicated %u OBU elements, was %u\n",`
`378`	`378`	`num_obus, obu_cnt);`
`379`	`379`	`}`