Commit c7ab0eb3 authored by Anton Khirnov's avatar Anton Khirnov

examples/decode_video: allocate the packet dynamically

AVPackets on stack are discouraged.
parent 728ea23c
...@@ -94,7 +94,7 @@ int main(int argc, char **argv) ...@@ -94,7 +94,7 @@ int main(int argc, char **argv)
uint8_t *data; uint8_t *data;
size_t data_size; size_t data_size;
int ret; int ret;
AVPacket avpkt; AVPacket *pkt;
if (argc <= 2) { if (argc <= 2) {
fprintf(stderr, "Usage: %s <input file> <output file>\n", argv[0]); fprintf(stderr, "Usage: %s <input file> <output file>\n", argv[0]);
...@@ -105,7 +105,9 @@ int main(int argc, char **argv) ...@@ -105,7 +105,9 @@ int main(int argc, char **argv)
avcodec_register_all(); avcodec_register_all();
av_init_packet(&avpkt); pkt = av_packet_alloc();
if (!pkt)
exit(1);
/* set end of buffer to 0 (this ensures that no overreading happens for damaged MPEG streams) */ /* set end of buffer to 0 (this ensures that no overreading happens for damaged MPEG streams) */
memset(inbuf + INBUF_SIZE, 0, AV_INPUT_BUFFER_PADDING_SIZE); memset(inbuf + INBUF_SIZE, 0, AV_INPUT_BUFFER_PADDING_SIZE);
...@@ -151,7 +153,7 @@ int main(int argc, char **argv) ...@@ -151,7 +153,7 @@ int main(int argc, char **argv)
/* use the parser to split the data into frames */ /* use the parser to split the data into frames */
data = inbuf; data = inbuf;
while (data_size > 0) { while (data_size > 0) {
ret = av_parser_parse2(parser, c, &avpkt.data, &avpkt.size, ret = av_parser_parse2(parser, c, &pkt->data, &pkt->size,
data, data_size, AV_NOPTS_VALUE, AV_NOPTS_VALUE, 0); data, data_size, AV_NOPTS_VALUE, AV_NOPTS_VALUE, 0);
if (ret < 0) { if (ret < 0) {
fprintf(stderr, "Error while parsing\n"); fprintf(stderr, "Error while parsing\n");
...@@ -160,8 +162,8 @@ int main(int argc, char **argv) ...@@ -160,8 +162,8 @@ int main(int argc, char **argv)
data += ret; data += ret;
data_size -= ret; data_size -= ret;
if (avpkt.size) if (pkt->size)
decode(c, picture, &avpkt, outfilename); decode(c, picture, pkt, outfilename);
} }
} }
...@@ -173,6 +175,7 @@ int main(int argc, char **argv) ...@@ -173,6 +175,7 @@ int main(int argc, char **argv)
av_parser_close(parser); av_parser_close(parser);
avcodec_free_context(&c); avcodec_free_context(&c);
av_frame_free(&picture); av_frame_free(&picture);
av_packet_free(&pkt);
return 0; return 0;
} }
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment