@scrypted/libav 1.0.11 → 1.0.13

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@scrypted/libav",
-  "version": "1.0.11",
+  "version": "1.0.13",
   "dependencies": {
     "node-addon-api": "^8.1.0",
     "prebuild-install": "npm:@scrypted/prebuild-install@^7.1.8"

package/src/decode.cpp

@@ -83,6 +83,7 @@ public:
         // Decode video frames
         while (av_read_frame(fmt_ctx_, packet) >= 0)
         {
+            fprintf(stdout, "read a frame\n");
             if (packet->stream_index != videoStreamIndex)
             {
                 continue;
@@ -99,6 +100,7 @@ public:
             }
             av_packet_unref(packet); // Reset the packet for the next frame
         }
+        fprintf(stdout, "finished reading a frame\n");
 
         av_frame_free(&frame);
         av_packet_free(&packet);
@@ -141,7 +143,6 @@ public:
 private:
     static Napi::FunctionReference constructor;
     AVFormatContext *fmt_ctx_;
-    bool owns_context_; // Flag to indicate if we should free the context
     int videoStreamIndex;
     AVCodecContext *codecContext;
     AVBufferRef *hw_device_ctx;
@@ -200,7 +201,6 @@ Napi::Object AVFormatContextObject::Init(Napi::Env env, Napi::Object exports)
 
 AVFormatContextObject::AVFormatContextObject(const Napi::CallbackInfo &info)
     : Napi::ObjectWrap<AVFormatContextObject>(info),
-      owns_context_(true),
       videoStreamIndex(-1),
       codecContext(nullptr),
       hw_device_ctx(nullptr)
@@ -221,7 +221,6 @@ AVFormatContextObject::AVFormatContextObject(const Napi::CallbackInfo &info)
         }
 
         fmt_ctx_ = reinterpret_cast<AVFormatContext *>(value);
-        owns_context_ = false; // We don't own this context, so we shouldn't free it
     }
     else
     {
@@ -232,7 +231,6 @@ AVFormatContextObject::AVFormatContextObject(const Napi::CallbackInfo &info)
             Napi::Error::New(env, "Failed to allocate AVFormatContext").ThrowAsJavaScriptException();
             return;
         }
-        owns_context_ = true; // We own this context, so we should free it when done
     }
 }
 
@@ -514,18 +512,11 @@ Napi::Value AVFormatContextObject::Open(const Napi::CallbackInfo &info)
 
 AVFormatContextObject::~AVFormatContextObject()
 {
-    if (owns_context_ && fmt_ctx_)
-    {
-        av_log(NULL, AV_LOG_INFO, "Destroyng AVFormatContext\n");
-        avformat_close_input(&fmt_ctx_);
-        avformat_free_context(fmt_ctx_);
-        fmt_ctx_ = nullptr;
-    }
 }
 
 Napi::Value AVFormatContextObject::Close(const Napi::CallbackInfo &info)
 {
-    if (owns_context_ && fmt_ctx_)
+    if (fmt_ctx_)
     {
         av_log(NULL, AV_LOG_INFO, "Closing AVFormatContext\n");
         avformat_close_input(&fmt_ctx_);

package/src/filter.cpp

@@ -63,11 +63,6 @@ AVFilterGraphObject::AVFilterGraphObject(const Napi::CallbackInfo &info)
 
 AVFilterGraphObject::~AVFilterGraphObject()
 {
-    if (filterGraph)
-    {
-        avfilter_graph_free(&filterGraph);
-        filterGraph = nullptr;
-    }
 }
 
 Napi::Value AVFilterGraphObject::Destroy(const Napi::CallbackInfo &info)

package/src/frame.cpp

@@ -82,11 +82,6 @@ AVFrameObject::AVFrameObject(const Napi::CallbackInfo &info)
 
 AVFrameObject::~AVFrameObject()
 {
-    if (frame_)
-    {
-        av_frame_free(&frame_);
-        frame_ = nullptr;
-    }
 }
 
 Napi::Value AVFrameObject::ToJPEG(const Napi::CallbackInfo &info)