* [FFmpeg-devel] [PATCH] avcodec/ralf: Cleanup generically on init failure
@ 2022-02-08 10:54 Andreas Rheinhardt
2022-02-11 9:24 ` Andreas Rheinhardt
0 siblings, 1 reply; 2+ messages in thread
From: Andreas Rheinhardt @ 2022-02-08 10:54 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Andreas Rheinhardt
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
libavcodec/ralf.c | 26 +++++++-------------------
1 file changed, 7 insertions(+), 19 deletions(-)
diff --git a/libavcodec/ralf.c b/libavcodec/ralf.c
index bb80119b0c..0c51f49939 100644
--- a/libavcodec/ralf.c
+++ b/libavcodec/ralf.c
@@ -163,47 +163,35 @@ static av_cold int decode_init(AVCodecContext *avctx)
for (i = 0; i < 3; i++) {
ret = init_ralf_vlc(&ctx->sets[i].filter_params, filter_param_def[i],
FILTERPARAM_ELEMENTS);
- if (ret < 0) {
- decode_close(avctx);
+ if (ret < 0)
return ret;
- }
ret = init_ralf_vlc(&ctx->sets[i].bias, bias_def[i], BIAS_ELEMENTS);
- if (ret < 0) {
- decode_close(avctx);
+ if (ret < 0)
return ret;
- }
ret = init_ralf_vlc(&ctx->sets[i].coding_mode, coding_mode_def[i],
CODING_MODE_ELEMENTS);
- if (ret < 0) {
- decode_close(avctx);
+ if (ret < 0)
return ret;
- }
for (j = 0; j < 10; j++) {
for (k = 0; k < 11; k++) {
ret = init_ralf_vlc(&ctx->sets[i].filter_coeffs[j][k],
filter_coeffs_def[i][j][k],
FILTER_COEFFS_ELEMENTS);
- if (ret < 0) {
- decode_close(avctx);
+ if (ret < 0)
return ret;
- }
}
}
for (j = 0; j < 15; j++) {
ret = init_ralf_vlc(&ctx->sets[i].short_codes[j],
short_codes_def[i][j], SHORT_CODES_ELEMENTS);
- if (ret < 0) {
- decode_close(avctx);
+ if (ret < 0)
return ret;
- }
}
for (j = 0; j < 125; j++) {
ret = init_ralf_vlc(&ctx->sets[i].long_codes[j],
long_codes_def[i][j], LONG_CODES_ELEMENTS);
- if (ret < 0) {
- decode_close(avctx);
+ if (ret < 0)
return ret;
- }
}
}
@@ -539,5 +527,5 @@ const AVCodec ff_ralf_decoder = {
AV_CODEC_CAP_DR1,
.sample_fmts = (const enum AVSampleFormat[]) { AV_SAMPLE_FMT_S16P,
AV_SAMPLE_FMT_NONE },
- .caps_internal = FF_CODEC_CAP_INIT_THREADSAFE,
+ .caps_internal = FF_CODEC_CAP_INIT_THREADSAFE | FF_CODEC_CAP_INIT_CLEANUP,
};
--
2.32.0
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avcodec/ralf: Cleanup generically on init failure
2022-02-08 10:54 [FFmpeg-devel] [PATCH] avcodec/ralf: Cleanup generically on init failure Andreas Rheinhardt
@ 2022-02-11 9:24 ` Andreas Rheinhardt
0 siblings, 0 replies; 2+ messages in thread
From: Andreas Rheinhardt @ 2022-02-11 9:24 UTC (permalink / raw)
To: ffmpeg-devel
Andreas Rheinhardt:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
> libavcodec/ralf.c | 26 +++++++-------------------
> 1 file changed, 7 insertions(+), 19 deletions(-)
>
> diff --git a/libavcodec/ralf.c b/libavcodec/ralf.c
> index bb80119b0c..0c51f49939 100644
> --- a/libavcodec/ralf.c
> +++ b/libavcodec/ralf.c
> @@ -163,47 +163,35 @@ static av_cold int decode_init(AVCodecContext *avctx)
> for (i = 0; i < 3; i++) {
> ret = init_ralf_vlc(&ctx->sets[i].filter_params, filter_param_def[i],
> FILTERPARAM_ELEMENTS);
> - if (ret < 0) {
> - decode_close(avctx);
> + if (ret < 0)
> return ret;
> - }
> ret = init_ralf_vlc(&ctx->sets[i].bias, bias_def[i], BIAS_ELEMENTS);
> - if (ret < 0) {
> - decode_close(avctx);
> + if (ret < 0)
> return ret;
> - }
> ret = init_ralf_vlc(&ctx->sets[i].coding_mode, coding_mode_def[i],
> CODING_MODE_ELEMENTS);
> - if (ret < 0) {
> - decode_close(avctx);
> + if (ret < 0)
> return ret;
> - }
> for (j = 0; j < 10; j++) {
> for (k = 0; k < 11; k++) {
> ret = init_ralf_vlc(&ctx->sets[i].filter_coeffs[j][k],
> filter_coeffs_def[i][j][k],
> FILTER_COEFFS_ELEMENTS);
> - if (ret < 0) {
> - decode_close(avctx);
> + if (ret < 0)
> return ret;
> - }
> }
> }
> for (j = 0; j < 15; j++) {
> ret = init_ralf_vlc(&ctx->sets[i].short_codes[j],
> short_codes_def[i][j], SHORT_CODES_ELEMENTS);
> - if (ret < 0) {
> - decode_close(avctx);
> + if (ret < 0)
> return ret;
> - }
> }
> for (j = 0; j < 125; j++) {
> ret = init_ralf_vlc(&ctx->sets[i].long_codes[j],
> long_codes_def[i][j], LONG_CODES_ELEMENTS);
> - if (ret < 0) {
> - decode_close(avctx);
> + if (ret < 0)
> return ret;
> - }
> }
> }
>
> @@ -539,5 +527,5 @@ const AVCodec ff_ralf_decoder = {
> AV_CODEC_CAP_DR1,
> .sample_fmts = (const enum AVSampleFormat[]) { AV_SAMPLE_FMT_S16P,
> AV_SAMPLE_FMT_NONE },
> - .caps_internal = FF_CODEC_CAP_INIT_THREADSAFE,
> + .caps_internal = FF_CODEC_CAP_INIT_THREADSAFE | FF_CODEC_CAP_INIT_CLEANUP,
> };
Will apply this later tonight unless there are objections.
- Andreas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-02-11 9:24 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-08 10:54 [FFmpeg-devel] [PATCH] avcodec/ralf: Cleanup generically on init failure Andreas Rheinhardt
2022-02-11 9:24 ` Andreas Rheinhardt
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git