On Wed, 2017-11-22 at 15:34 +0200, Luiz Augusto von Dentz wrote:
> From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>
>
> The value stored in the hashtable is the path not the actual pointer
> of
> the item.
> ---
> profiles/audio/player.c | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/profiles/audio/player.c b/profiles/audio/player.c
> index e499e6757..d12969bf9 100644
> --- a/profiles/audio/player.c
> +++ b/profiles/audio/player.c
> @@ -1885,6 +1885,7 @@ struct media_item
> *media_player_set_playlist_item(struct media_player *mp,
> {
> struct media_folder *folder = mp->playlist;
> struct media_item *item;
> + char *path;
const char *? So it's clear why you don't free it.
> DBG("%" PRIu64 "", uid);
>
> @@ -1903,7 +1904,8 @@ struct media_item
> *media_player_set_playlist_item(struct media_player *mp,
> mp->track = g_hash_table_ref(item->metadata);
> }
>
> - if (item == g_hash_table_lookup(mp->track, "Item"))
> + path = g_hash_table_lookup(mp->track, "Item");
> + if (!g_strcmp0(path, item->path))
> return item;
>
> if (mp->process_id == 0) {
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
