Commit 774239be authored by Timothy Gu's avatar Timothy Gu Committed by Michael Niedermayer

ffmpeg_opt: get_preset_file_2(): fix avio_open2() return code check

avio_open2() only return < 0 when error.
Signed-off-by: 's avatarTimothy Gu <timothygu99@gmail.com>
Signed-off-by: 's avatarMichael Niedermayer <michaelni@gmx.at>
parent 75f6ed8d
...@@ -978,14 +978,14 @@ static uint8_t *get_line(AVIOContext *s) ...@@ -978,14 +978,14 @@ static uint8_t *get_line(AVIOContext *s)
static int get_preset_file_2(const char *preset_name, const char *codec_name, AVIOContext **s) static int get_preset_file_2(const char *preset_name, const char *codec_name, AVIOContext **s)
{ {
int i, ret = 1; int i, ret = -1;
char filename[1000]; char filename[1000];
const char *base[3] = { getenv("AVCONV_DATADIR"), const char *base[3] = { getenv("AVCONV_DATADIR"),
getenv("HOME"), getenv("HOME"),
AVCONV_DATADIR, AVCONV_DATADIR,
}; };
for (i = 0; i < FF_ARRAY_ELEMS(base) && ret; i++) { for (i = 0; i < FF_ARRAY_ELEMS(base) && ret < 0; i++) {
if (!base[i]) if (!base[i])
continue; continue;
if (codec_name) { if (codec_name) {
...@@ -993,7 +993,7 @@ static int get_preset_file_2(const char *preset_name, const char *codec_name, AV ...@@ -993,7 +993,7 @@ static int get_preset_file_2(const char *preset_name, const char *codec_name, AV
i != 1 ? "" : "/.avconv", codec_name, preset_name); i != 1 ? "" : "/.avconv", codec_name, preset_name);
ret = avio_open2(s, filename, AVIO_FLAG_READ, &int_cb, NULL); ret = avio_open2(s, filename, AVIO_FLAG_READ, &int_cb, NULL);
} }
if (ret) { if (ret < 0) {
snprintf(filename, sizeof(filename), "%s%s/%s.avpreset", base[i], snprintf(filename, sizeof(filename), "%s%s/%s.avpreset", base[i],
i != 1 ? "" : "/.avconv", preset_name); i != 1 ? "" : "/.avconv", preset_name);
ret = avio_open2(s, filename, AVIO_FLAG_READ, &int_cb, NULL); ret = avio_open2(s, filename, AVIO_FLAG_READ, &int_cb, NULL);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment