Opened 3 years ago
Closed 3 years ago
#10426 closed defect (fixed)
The flag for the key frame on MPEG video is not set.
| Reported by: | Aleksoid1978 | Owned by: | |
|---|---|---|---|
| Priority: | normal | Component: | avcodec |
| Version: | git-master | Keywords: | |
| Cc: | Blocked By: | ||
| Blocking: | Reproduced by developer: | no | |
| Analyzed by developer: | yes |
Description
Summary of the bug:
Bug in line
https://git.ffmpeg.org/gitweb/ffmpeg.git/blob/HEAD:/libavcodec/mpegvideo_dec.c#l347
and
https://git.ffmpeg.org/gitweb/ffmpeg.git/blob/HEAD:/libavcodec/mpegvideo_dec.c#l349
Should be s->current_picture_ptr->f
instead of s->current_picture.f
Change History (8)
comment:1 by , 3 years ago
comment:2 by , 3 years ago
No - this happened recently, after switching to using flags + AV_FRAME_FLAG_KEY.
comment:4 by , 3 years ago
Replying to Aleksoid1978:
Developers generally come here, look?
Sometimes. It appears such tickets with actual change require you submitting a patch to a mailing list.
comment:6 by , 3 years ago
Replying to Aleksoid1978:
How can i submit patch to a mailing list ?
First join the mailing list, i.e. subscribe to it. It will not work if you are not subscribed. Attach git created diff patch and rename it from .patch to .txt (VERY important). Use google mail, not Outlook. Check pathwork.ffmpeg.org where it will compile and test FATE.
comment:7 by , 3 years ago
Not needed, fix is here: https://patchwork.ffmpeg.org/project/ffmpeg/patch/20230714123132.55249-1-jamrial@gmail.com/
comment:8 by , 3 years ago
| Analyzed by developer: | set |
|---|---|
| Resolution: | → fixed |
| Status: | new → closed |
| Version: | unspecified → git-master |
Fixed in e047d9c62bff5920074c516da966952cee20da66.



Is this an issie that is visable with ffprove? Like #8515 and #9863?