diff options
| author | William Wilgus <wilgus.william@gmail.com> | 2024-05-30 20:25:35 -0400 |
|---|---|---|
| committer | William Wilgus <me.theuser@yahoo.com> | 2024-05-31 23:02:28 -0400 |
| commit | d1aeb7db9dbc3cdd0db25ab709b9e2d2322ef8d2 (patch) | |
| tree | a4206b281d04a43d3843335aaa631138ffe82cd0 /apps/playback.c | |
| parent | eaeaac598d5c06a138a5e8ea012f7a959461e473 (diff) | |
| download | rockbox-d1aeb7db9dbc3cdd0db25ab709b9e2d2322ef8d2.tar.gz rockbox-d1aeb7db9dbc3cdd0db25ab709b9e2d2322ef8d2.zip | |
[Coverity] playback.c audio_auto_change_frequency check for valid id3
Change-Id: Ia213134c690f4c1de16afba38eea740825558280
Diffstat (limited to 'apps/playback.c')
| -rw-r--r-- | apps/playback.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/apps/playback.c b/apps/playback.c index 7ea44a047f..18f62fb69b 100644 --- a/apps/playback.c +++ b/apps/playback.c | |||
| @@ -4054,6 +4054,8 @@ static unsigned long audio_guess_frequency(struct mp3entry *id3) | |||
| 4054 | 4054 | ||
| 4055 | static bool audio_auto_change_frequency(struct mp3entry *id3, bool play) | 4055 | static bool audio_auto_change_frequency(struct mp3entry *id3, bool play) |
| 4056 | { | 4056 | { |
| 4057 | if (!valid_mp3entry(id3)) | ||
| 4058 | return false; | ||
| 4057 | unsigned long guessed_frequency = global_settings.play_frequency == 0 ? audio_guess_frequency(id3) : 0; | 4059 | unsigned long guessed_frequency = global_settings.play_frequency == 0 ? audio_guess_frequency(id3) : 0; |
| 4058 | if (guessed_frequency && mixer_get_frequency() != guessed_frequency) | 4060 | if (guessed_frequency && mixer_get_frequency() != guessed_frequency) |
| 4059 | { | 4061 | { |