[FFmpeg-devel] [PATCH 1/7] parse_packet: reset pkt->pos after it has been used

Michael Niedermayer michaelni at gmx.at
Wed Jul 25 00:43:06 CEST 2012


On Wed, Jul 25, 2012 at 12:36:15AM +0200, Stefano Sabatini wrote:
> On date Tuesday 2012-07-24 18:15:33 +0200, Michael Niedermayer encoded:
> > This should have no effect as it was not used in this case
> > 
> > Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> > ---
> >  libavformat/utils.c |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/libavformat/utils.c b/libavformat/utils.c
> > index 5bc79f4..e348fbd 100644
> > --- a/libavformat/utils.c
> > +++ b/libavformat/utils.c
> > @@ -1184,6 +1184,7 @@ static int parse_packet(AVFormatContext *s, AVPacket *pkt, int stream_index)
> >                                 pkt->pts, pkt->dts, pkt->pos);
> >  
> >          pkt->pts = pkt->dts = AV_NOPTS_VALUE;
> > +        pkt->pos = -1;
> >          /* increment read pointer */
> >          data += len;
> >          size -= len;
> 
> LGTM, assuming it has no side effects and it is useful.

applied

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

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120725/9a5368fa/attachment.asc>


More information about the ffmpeg-devel mailing list