diff options
| author | Ganesh Ajjanagadde <gajjanagadde@gmail.com> | 2015-10-15 19:36:22 -0400 |
|---|---|---|
| committer | Ganesh Ajjanagadde <gajjanagadde@gmail.com> | 2015-10-18 15:17:58 -0400 |
| commit | 07d4fe3a871f3ea2f3e475bc84c3e85102ec75ba (patch) | |
| tree | 331d7f2f5ecbf93ab6818a736bca9b28ff0ba96d /libavutil/cast5.c | |
| parent | 94f7c97e05737c2856bb08e2172b012a56568bdc (diff) | |
| download | ffmpeg-07d4fe3a871f3ea2f3e475bc84c3e85102ec75ba.tar.gz | |
avutil: use EINVAL instead of -1 for the return code of crypto related init functions
These functions return an error typically when the key size is an
incorrect number. AVERROR(EINVAL) is more specific than -1.
Reviewed-by: Ronald S. Bultje <rsbultje@gmail.com>
Reviewed-by: Michael Niedermayer <michael@niedermayer.cc>
Signed-off-by: Ganesh Ajjanagadde <gajjanagadde@gmail.com>
Diffstat (limited to 'libavutil/cast5.c')
| -rw-r--r-- | libavutil/cast5.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libavutil/cast5.c b/libavutil/cast5.c index 98aa19d630..a47697b2f8 100644 --- a/libavutil/cast5.c +++ b/libavutil/cast5.c @@ -459,7 +459,7 @@ av_cold int av_cast5_init(AVCAST5* cs, const uint8_t *key, int key_bits) int i; uint32_t p[4], q[4]; if (key_bits % 8 || key_bits < 40 || key_bits > 128) - return -1; + return AVERROR(EINVAL); memset(newKey, 0, sizeof(newKey)); memcpy(newKey, key, key_bits >> 3); |