[FFmpeg-devel] [PATCH 11/48] avcodec/tests/avpacket: use av_packet_alloc() to allocate packets

James Almer jamrial at gmail.com
Fri Mar 5 18:33:02 EET 2021


Signed-off-by: James Almer <jamrial at gmail.com>
---
 libavcodec/tests/avpacket.c | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

diff --git a/libavcodec/tests/avpacket.c b/libavcodec/tests/avpacket.c
index 90b72341f4..7a70ade4c3 100644
--- a/libavcodec/tests/avpacket.c
+++ b/libavcodec/tests/avpacket.c
@@ -63,9 +63,6 @@ static int initializations(AVPacket* avpkt)
     const static uint8_t* data = "selftest for av_packet_clone(...)";
     int ret = 0;
 
-    /* initialize avpkt */
-    av_init_packet(avpkt);
-
     /* set values for avpkt */
     avpkt->pts = 17;
     avpkt->dts = 2;
@@ -82,16 +79,24 @@ static int initializations(AVPacket* avpkt)
 
 int main(void)
 {
-    AVPacket avpkt;
+    AVPacket *avpkt = NULL;
     AVPacket *avpkt_clone = NULL;
     int ret = 0;
 
-    if(initializations(&avpkt) < 0){
+    /* test av_packet_alloc */
+    avpkt = av_packet_alloc();
+    if(!avpkt) {
+        av_log(NULL, AV_LOG_ERROR, "av_packet_alloc failed to allcoate AVPacket\n");
+        return 1;
+    }
+
+    if (initializations(avpkt) < 0) {
         printf("failed to initialize variables\n");
+        av_packet_free(&avpkt);
         return 1;
     }
     /* test av_packet_clone*/
-    avpkt_clone = av_packet_clone(&avpkt);
+    avpkt_clone = av_packet_clone(avpkt);
 
     if(!avpkt_clone) {
         av_log(NULL, AV_LOG_ERROR,"av_packet_clone failed to clone AVPacket\n");
@@ -121,7 +126,7 @@ int main(void)
     }
     /*clean up*/
     av_packet_free(&avpkt_clone);
-    av_packet_unref(&avpkt);
+    av_packet_free(&avpkt);
 
 
     return ret;
-- 
2.30.1



More information about the ffmpeg-devel mailing list