oysteikt/mpd
0
0
Fork 0
Code Issues Pull Requests Releases Activity

decoder/mpg123: use AtScopeExit() to call mpg123_delete() during scan

Browse Source
This commit is contained in:
Max Kellermann 2018-08-07 22:48:11 +02:00
parent cfe34647e1
commit cacc1ffdad
1 changed files with 2 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/decoder/plugins/Mpg123DecoderPlugin.cxx
View File

@ -287,20 +287,19 @@ mpd_mpg123_scan_file(Path path_fs, TagHandler &handler) noexcept
return false;
}
AtScopeExit(handle) { mpg123_delete(handle); };
AudioFormat audio_format;
try {
if (!mpd_mpg123_open(handle, path_fs.c_str(), audio_format)) {
mpg123_delete(handle);
return false;
}
} catch (...) {
mpg123_delete(handle);
return false;
}
const off_t num_samples = mpg123_length(handle);
if (num_samples <= 0) {
mpg123_delete(handle);
return false;
}
@ -308,8 +307,6 @@ mpd_mpg123_scan_file(Path path_fs, TagHandler &handler) noexcept
/* ID3 tag support not yet implemented */
mpg123_delete(handle);
const auto duration =
SongTime::FromScale<uint64_t>(num_samples,
audio_format.sample_rate);