diff options
author | Anton Khirnov <anton@khirnov.net> | 2015-05-22 21:50:14 +0300 |
---|---|---|
committer | Vittorio Giovara <vittorio.giovara@gmail.com> | 2015-05-28 17:38:43 +0300 |
commit | 1b1bb2c4efc126d74d44d8c421860c85f932ecb1 (patch) | |
tree | f0cd5a786852326a75ab665fbc7f73b5490111a1 /libavcodec | |
parent | 324e50ee95929a9491b855c5e15451145bd5d1ec (diff) |
rl: Add error checking to ff_rl_init().
Diffstat (limited to 'libavcodec')
-rw-r--r-- | libavcodec/rl.c | 26 | ||||
-rw-r--r-- | libavcodec/rl.h | 2 |
2 files changed, 21 insertions, 7 deletions
diff --git a/libavcodec/rl.c b/libavcodec/rl.c index c0ec8ec296..5bc1f91563 100644 --- a/libavcodec/rl.c +++ b/libavcodec/rl.c @@ -34,8 +34,8 @@ void ff_rl_free(RLTable *rl) } } -av_cold void ff_rl_init(RLTable *rl, - uint8_t static_store[2][2 * MAX_RUN + MAX_LEVEL + 3]) +av_cold int ff_rl_init(RLTable *rl, + uint8_t static_store[2][2 * MAX_RUN + MAX_LEVEL + 3]) { int8_t max_level[MAX_RUN + 1], max_run[MAX_LEVEL + 1]; uint8_t index_run[MAX_RUN + 1]; @@ -43,7 +43,7 @@ av_cold void ff_rl_init(RLTable *rl, /* If table is static, we can quit if rl->max_level[0] is not NULL */ if (static_store && rl->max_level[0]) - return; + return 0; /* compute max_level[], max_run[] and index_run[] */ for (last = 0; last < 2; last++) { @@ -70,20 +70,34 @@ av_cold void ff_rl_init(RLTable *rl, } if (static_store) rl->max_level[last] = static_store[last]; - else + else { rl->max_level[last] = av_malloc(MAX_RUN + 1); + if (!rl->max_level[last]) + goto fail; + } memcpy(rl->max_level[last], max_level, MAX_RUN + 1); if (static_store) rl->max_run[last] = static_store[last] + MAX_RUN + 1; - else + else { rl->max_run[last] = av_malloc(MAX_LEVEL + 1); + if (!rl->max_run[last]) + goto fail; + } memcpy(rl->max_run[last], max_run, MAX_LEVEL + 1); if (static_store) rl->index_run[last] = static_store[last] + MAX_RUN + MAX_LEVEL + 2; - else + else { rl->index_run[last] = av_malloc(MAX_RUN + 1); + if (!rl->index_run[last]) + goto fail; + } memcpy(rl->index_run[last], index_run, MAX_RUN + 1); } + return 0; + +fail: + ff_rl_free(rl); + return AVERROR(ENOMEM); } av_cold void ff_rl_init_vlc(RLTable *rl) diff --git a/libavcodec/rl.h b/libavcodec/rl.h index 8244968898..fe4efef972 100644 --- a/libavcodec/rl.h +++ b/libavcodec/rl.h @@ -53,7 +53,7 @@ typedef struct RLTable { * @param static_store static uint8_t array[2][2*MAX_RUN + MAX_LEVEL + 3] which will hold * the level and run tables, if this is NULL av_malloc() will be used */ -void ff_rl_init(RLTable *rl, uint8_t static_store[2][2*MAX_RUN + MAX_LEVEL + 3]); +int ff_rl_init(RLTable *rl, uint8_t static_store[2][2*MAX_RUN + MAX_LEVEL + 3]); void ff_rl_init_vlc(RLTable *rl); /** |