[FFmpeg-devel] [PATCH 03/10] avformat/utils: use av_packet_alloc() to allocate packets

Michael Niedermayer michael at niedermayer.cc
Thu Feb 4 00:36:42 EET 2021


On Wed, Feb 03, 2021 at 07:25:29PM -0300, James Almer wrote:
> On 2/3/2021 7:23 PM, Michael Niedermayer wrote:
> > On Mon, Feb 01, 2021 at 07:44:14PM -0300, James Almer wrote:
> > > Signed-off-by: James Almer <jamrial at gmail.com>
> > > ---
> > > av_get_packet() and av_read_frame() were allowed to be called with
> > > uninitialized packets in stack, so to keep supporting that we need to leave the
> > > calls to av_init_packet() in place during the deprecation period.
> > > 
> > >   libavformat/internal.h |   1 +
> > >   libavformat/options.c  |  13 +++--
> > >   libavformat/utils.c    | 105 +++++++++++++++++++++++------------------
> > >   3 files changed, 68 insertions(+), 51 deletions(-)
> > 
> > valgrind dislikes this
> 
> Did you apply patch 10/10 first, like i mentioned in another reply?
> I mistakenly sent that patch, which introduces an FF_API define, before the
> patches that make use of it (This one being the first doing so).

I probably applied them in order yesterday on their numbers and testing today
if that was wrong ill test again next time this is posted i guess
iam a little behind today with things i wanted to do

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20210203/af0671ff/attachment.sig>


More information about the ffmpeg-devel mailing list