You've already forked FFmpeg
							
							
				mirror of
				https://github.com/FFmpeg/FFmpeg.git
				synced 2025-10-30 23:18:11 +02:00 
			
		
		
		
	vf_hwupload: Add missing return value check
Add missing return value checks to suppress build warning and remove noop ff_formats_unref() calling. Note: most filters using ff_formats_ref() didn't have a suitable error handling, it's a potential memory leak issue. Signed-off-by: Jun Zhao <jun.zhao@intel.com> Signed-off-by: Mark Thompson <sw@jkqxz.net>
This commit is contained in:
		| @@ -74,17 +74,15 @@ static int hwupload_query_formats(AVFilterContext *avctx) | ||||
|     if (input_pix_fmts) { | ||||
|         for (i = 0; input_pix_fmts[i] != AV_PIX_FMT_NONE; i++) { | ||||
|             err = ff_add_format(&input_formats, input_pix_fmts[i]); | ||||
|             if (err < 0) { | ||||
|                 ff_formats_unref(&input_formats); | ||||
|             if (err < 0) | ||||
|                 goto fail; | ||||
|             } | ||||
|         } | ||||
|     } | ||||
|  | ||||
|     ff_formats_ref(input_formats, &avctx->inputs[0]->out_formats); | ||||
|  | ||||
|     ff_formats_ref(ff_make_format_list(output_pix_fmts), | ||||
|                    &avctx->outputs[0]->in_formats); | ||||
|     if ((err = ff_formats_ref(input_formats, &avctx->inputs[0]->out_formats)) < 0 || | ||||
|         (err = ff_formats_ref(ff_make_format_list(output_pix_fmts), | ||||
|                               &avctx->outputs[0]->in_formats)) < 0) | ||||
|         goto fail; | ||||
|  | ||||
|     av_hwframe_constraints_free(&constraints); | ||||
|     return 0; | ||||
|   | ||||
		Reference in New Issue
	
	Block a user