Repository navigation
[BUG] Crash when trying to search lyrics on the internet #649
Description
Describe the bug
ncmpcpp crashes when trying to search lyrics from the internet
To Reproduce
Steps to reproduce the behavior:
- Configure lyrics fetchers in
.config/ncmpcpp/config:
lyrics_fetchers = internet
- Go lyrics interface when listeting to a song without saved lyrics and wait.
Expected behavior
To not crash and possibly print an URL pointing to the lyrics.
Screenshots/Logs
The error message is:
ncmpcpp: /usr/include/boost/optional/detail/union_optional.hpp:580: constexpr T& boost::optional<T>::get() [with T = boost::optional<std::__cxx11::basic_string<char> >; reference_type = boost::optional<std::__cxx11::basic_string<char> >&]: Assertion "this->is_initialized()" failed.rnet... zsh: IOT instruction (core dumped) ncmpcpp
Desktop (please complete the following information):
- OS: Linux
- Terminal: [e.g. zsh]
- Version 0.10.1
Activity
shan995-cell commentedon May 21, 2026
I can affirm this bug. It seems that it also happens even with just
lyrics_fetchers = tagsin the config.Using
0.10.1, Artix Linux, fish shell.cornelius-sevald commentedon Jul 12, 2026
I am getting the same error (ncmpcpp v0.10.1, on Arch Linux).
From poking around it seems that if the future value of
m_workerisboost::nonethis linencmpcpp/src/screens/lyrics.cpp
Line 222 in 04f2f43
will try and access the (non-existent) optional value and crash.
I don't know enough about C++ or boost to know why this happens.
I've tried to fix it by changing the
boost::optional<std::string>type to astd::pair<bool, std::string>, the change is here: cornelius-sevald@de286ffIt seems to work, but I'm not very proficient in C++ so judge for yourself. Hope it helps :)
alkino commentedon Sep 21, 2026
Can you confirm that the bug is fixed with the last master?
shan995-cell commentedon Oct 1, 2026
hello @alkino
i compiled ncmpcpp from source and it seems that your merged fixed works, thank you for your contribution !!
hopefully we see a new ncmpcpp release soon