hashirama/uvg266
1
0
Fork 0
mirror of https://github.com/ultravideo/uvg266.git synced 2024-12-18 03:04:06 +00:00
Code Issues Actions Projects Releases Wiki Activity

Fix finalizing uninitialized encoder states

Browse source 
Finalization functions for frame and tile encoder states accessed the
frame and tile fields of the encoder state even though they might be
NULL. This is the case when the initialization of an encoder state
fails. Fixed by adding NULL checks.
This commit is contained in:
Arttu Ylä-Outinen 2017-02-07 12:57:28 +09:00
parent 51786eda67
commit 7f7844caad
1 changed files with 4 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/encoder_state-ctors_dtors.c
View file

@ -61,6 +61,8 @@ static int encoder_state_config_frame_init(encoder_state_t * const state) {
} }
static void encoder_state_config_frame_finalize(encoder_state_t * const state) { static void encoder_state_config_frame_finalize(encoder_state_t * const state) {
if (state->frame == NULL) return;
kvz_image_list_destroy(state->frame->ref); kvz_image_list_destroy(state->frame->ref);
FREE_POINTER(state->frame->lcu_stats); FREE_POINTER(state->frame->lcu_stats);
} }
@ -128,6 +130,8 @@ static int encoder_state_config_tile_init(encoder_state_t * const state,
} }
static void encoder_state_config_tile_finalize(encoder_state_t * const state) { static void encoder_state_config_tile_finalize(encoder_state_t * const state) {
if (state->tile == NULL) return;
if (state->tile->hor_buf_before_sao) kvz_yuv_t_free(state->tile->hor_buf_before_sao); if (state->tile->hor_buf_before_sao) kvz_yuv_t_free(state->tile->hor_buf_before_sao);
kvz_yuv_t_free(state->tile->hor_buf_search); kvz_yuv_t_free(state->tile->hor_buf_search);
@ -160,10 +164,6 @@ static int encoder_state_config_slice_init(encoder_state_t * const state,
return 1; return 1;
} }
static void encoder_state_config_slice_finalize(encoder_state_t * const state) {
//Nothing to do (yet?)
}
static int encoder_state_config_wfrow_init(encoder_state_t * const state, static int encoder_state_config_wfrow_init(encoder_state_t * const state,
const int lcu_offset_y) { const int lcu_offset_y) {
@ -171,10 +171,6 @@ static int encoder_state_config_wfrow_init(encoder_state_t * const state,
return 1; return 1;
} }
static void encoder_state_config_wfrow_finalize(encoder_state_t * const state) {
//Nothing to do (yet?)
}
#ifdef KVZ_DEBUG_PRINT_THREADING_INFO #ifdef KVZ_DEBUG_PRINT_THREADING_INFO
static void encoder_state_dump_graphviz(const encoder_state_t * const state) { static void encoder_state_dump_graphviz(const encoder_state_t * const state) {
int i; int i;
@ -692,12 +688,10 @@ void kvz_encoder_state_finalize(encoder_state_t * const state) {
state->lcu_order_count = 0; state->lcu_order_count = 0;
if (!state->parent || (state->parent->wfrow != state->wfrow)) { if (!state->parent || (state->parent->wfrow != state->wfrow)) {
encoder_state_config_wfrow_finalize(state);
FREE_POINTER(state->wfrow); FREE_POINTER(state->wfrow);
} }
if (!state->parent || (state->parent->slice != state->slice)) { if (!state->parent || (state->parent->slice != state->slice)) {
encoder_state_config_slice_finalize(state);
FREE_POINTER(state->slice); FREE_POINTER(state->slice);
} }