media_utils: fix segfault when stopping a recording

RPCS3 · Jan 12, 2024 · 9dd0a2e · 9dd0a2e
1 parent 17aeefe
commit 9dd0a2e
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 15 deletions.
diff --git a/rpcs3/Emu/Cell/Modules/cellAdec.cpp b/rpcs3/Emu/Cell/Modules/cellAdec.cpp
@@ -371,7 +371,6 @@ class AudioDecoder : public ppu_thread
 		}
 		if (ctx)
 		{
-			avcodec_close(ctx);
 			avcodec_free_context(&ctx);
 		}
 		if (io_buf)

diff --git a/rpcs3/Emu/Cell/Modules/cellVdec.cpp b/rpcs3/Emu/Cell/Modules/cellVdec.cpp
@@ -280,7 +280,6 @@ struct vdec_context final
 
 	~vdec_context()
 	{
-		avcodec_close(ctx);
 		avcodec_free_context(&ctx);
 		sws_freeContext(sws);
 	}

diff --git a/rpcs3/util/media_utils.cpp b/rpcs3/util/media_utils.cpp
@@ -302,22 +302,13 @@ namespace utils
 			if (sws)
 				sws_freeContext(sws);
 			if (audio.context)
-			{
-				avcodec_close(audio.context);
 				avcodec_free_context(&audio.context);
-			}
 			if (video.context)
-			{
-				avcodec_close(video.context);
 				avcodec_free_context(&video.context);
-			}
 			// AVCodec is managed by libavformat, no need to free it
 			// see: https://stackoverflow.com/a/18047320
 			if (format_context)
-			{
 				avformat_close_input(&format_context);
-				avformat_free_context(format_context);
-			}
 			//if (stream)
 			//	av_free(stream);
 			if (kill_callback)
@@ -1266,9 +1257,9 @@ namespace utils
 			{
 				media_log.error("video_encoder: avformat_write_header failed. Error: %d='%s'", err, av_error_to_string(err));
 
-				if (int err = avio_close(av.format_context->pb); err != 0)
+				if (int err = avio_closep(&av.format_context->pb); err != 0)
 				{
-					media_log.error("video_encoder: avio_close failed. Error: %d='%s'", err, av_error_to_string(err));
+					media_log.error("video_encoder: avio_closep failed. Error: %d='%s'", err, av_error_to_string(err));
 				}
 
 				has_error = true;
@@ -1645,9 +1636,9 @@ namespace utils
 				media_log.error("video_encoder: av_write_trailer failed. Error: %d='%s'", err, av_error_to_string(err));
 			}
 
-			if (int err = avio_close(av.format_context->pb); err != 0)
+			if (int err = avio_closep(&av.format_context->pb); err != 0)
 			{
-				media_log.error("video_encoder: avio_close failed. Error: %d='%s'", err, av_error_to_string(err));
+				media_log.error("video_encoder: avio_closep failed. Error: %d='%s'", err, av_error_to_string(err));
 			}
 		});
 	}