Skip to content

Commit

Permalink
cppcheck: Fix "possible null pointer dereference" in musicplayer.cpp.
Browse files Browse the repository at this point in the history
Fix the bounds check to ensure no out of bounds access.
  • Loading branch information
linuxdude42 committed Jul 10, 2022
1 parent 1675200 commit f7ba32c
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion mythplugins/mythmusic/mythmusic/musicplayer.cpp
Expand Up @@ -613,7 +613,7 @@ void MusicPlayer::customEvent(QEvent *event)
pause();
else if (list[2] == "SET_VOLUME")
{
if (list.size() >= 3)
if (list.size() > 3)
{
int volume = list[3].toInt();
if (volume >= 0 && volume <= 100)
Expand Down

0 comments on commit f7ba32c

Please sign in to comment.