avformat/flacdec/flac_read_timestamp: dont modify AVStream state
authorMichael Niedermayer <michaelni@gmx.at>
Tue, 15 Apr 2014 21:53:31 +0000 (23:53 +0200)
committerMichael Niedermayer <michaelni@gmx.at>
Tue, 15 Apr 2014 21:53:31 +0000 (23:53 +0200)
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>
libavformat/flacdec.c

index f640fec..1caa7b0 100644 (file)
@@ -167,17 +167,19 @@ static av_unused int64_t flac_read_timestamp(AVFormatContext *s, int stream_inde
 {
     AVPacket pkt, out_pkt;
     AVStream *st = s->streams[stream_index];
+    AVCodecParserContext *parser;
     int ret;
+    int64_t pts = AV_NOPTS_VALUE;
 
     if (avio_seek(s->pb, *ppos, SEEK_SET) < 0)
         return AV_NOPTS_VALUE;
 
     av_init_packet(&pkt);
-    st->parser = av_parser_init(st->codec->codec_id);
-    if (!st->parser){
+    parser = av_parser_init(st->codec->codec_id);
+    if (!parser){
         return AV_NOPTS_VALUE;
     }
-    st->parser->flags |= PARSER_FLAG_USE_CODEC_TS;
+    parser->flags |= PARSER_FLAG_USE_CODEC_TS;
 
     for (;;){
         ret = ff_raw_read_partial_packet(s, &pkt);
@@ -188,22 +190,24 @@ static av_unused int64_t flac_read_timestamp(AVFormatContext *s, int stream_inde
                 return AV_NOPTS_VALUE;
         }
         av_init_packet(&out_pkt);
-        ret = av_parser_parse2(st->parser, st->codec,
+        ret = av_parser_parse2(parser, st->codec,
                                &out_pkt.data, &out_pkt.size, pkt.data, pkt.size,
                                pkt.pts, pkt.dts, *ppos);
 
         if (out_pkt.size){
             int size = out_pkt.size;
             av_free_packet(&out_pkt);
-            if (st->parser->pts != AV_NOPTS_VALUE){
+            if (parser->pts != AV_NOPTS_VALUE){
                 // seeking may not have started from beginning of a frame
                 // calculate frame start position from next frame backwards
-                *ppos = st->parser->next_frame_offset - size;
-                return st->parser->pts;
+                *ppos = parser->next_frame_offset - size;
+                pts = parser->pts;
+                break;
             }
         }
     }
-    return AV_NOPTS_VALUE;
+    av_parser_close(parser);
+    return pts;
 }
 
 AVInputFormat ff_flac_demuxer = {