diff options
| author | Joel Klinghed <the_jk@spawned.biz> | 2025-10-29 09:24:28 +0100 |
|---|---|---|
| committer | Joel Klinghed <the_jk@spawned.biz> | 2025-10-29 09:25:46 +0100 |
| commit | 42794079bf98ef6a7ce3854fc443f539bac7e9ce (patch) | |
| tree | 29596a8686f1d3cfd08daab08e78d216a3b0a110 /src | |
| parent | 0513cdd9511423ea97b12378035bb562316a7626 (diff) | |
bt: Avoid crashes for MediaPlayerTrack
Doing two things:
* Remove experimental and unused field ImgHandle field
* Handle variant being empty
Diffstat (limited to 'src')
| -rw-r--r-- | src/bt.cc | 9 |
1 files changed, 6 insertions, 3 deletions
@@ -48,7 +48,6 @@ struct MediaPlayerTrack { std::string Genre; uint32_t NumberOfTracks; uint32_t Duration; - std::string ImgHandle; }; } // namespace @@ -59,7 +58,7 @@ struct sdbus::signature_of<Void> : sdbus::signature_of<void> {}; #pragma clang diagnostic push #pragma clang diagnostic ignored "-Wunused-function" SDBUSCPP_REGISTER_STRUCT_NO_THROW(MediaPlayerTrack, Title, Artist, Album, Genre, - NumberOfTracks, Duration, ImgHandle); + NumberOfTracks, Duration); #pragma clang diagnostic pop namespace bt { @@ -726,7 +725,11 @@ class MediaPlayerProxy : public BaseProxy, public Player { break; } case 1: - track_ = value.get<MediaPlayerTrack>(); + if (value.isEmpty()) { + track_ = MediaPlayerTrack{}; + } else { + track_ = value.get<MediaPlayerTrack>(); + } break; case 2: device_ = value.get<sdbus::ObjectPath>(); |
