[bluez/bluez] 897689: mpris-proxy: Fix possible crash

[bluez/bluez] 897689: mpris-proxy: Fix possible crash

[hadess]

  Branch: refs/heads/1089698
  Home:   https://github.com/bluez/bluez
  Commit: 897689ffe9c0ac36fbd508b9d754a557d1ee84be
      https://github.com/bluez/bluez/commit/897689ffe9c0ac36fbd508b9d754a557d1ee84be
  Author: Bastien Nocera <hadess@hadess.net>
  Date:   2026-05-05 (Tue, 05 May 2026)

  Changed paths:
    M tools/mpris-proxy.c

  Log Message:
  -----------
  mpris-proxy: Fix possible crash

find_player_by_obex() doesn't check whether session->obex is a valid
pointer before dereferecing it, but all code paths that assign it use
create_obex_session() to assign it, a function that can fail.

Check whether session->obex is null before dereferencing it.

 #0 find_player_by_obex at tools/mpris-proxy.c:2819
 #1 obex_property_changed at tools/mpris-proxy.c:2929
 #2 add_property at gdbus/client.c:373
 #3 update_properties at gdbus/client.c:399
 #5 properties_changed at gdbus/client.c:537
 #6 signal_filter at gdbus/watch.c:416
 #7 message_filter at gdbus/watch.c:566
 #10 message_dispatch at gdbus/mainloop.c:59
 #13 g_main_context_dispatch_unlocked at ../glib/gmain.c:4451
 #14 g_main_context_iterate_unlocked at ../glib/gmain.c:4516

Closes: https://bugzilla.redhat.com/show_bug.cgi?id=2466640


  Commit: 79975d8d2f524ad50eb840c5137eb80a45aa6ea8
      https://github.com/bluez/bluez/commit/79975d8d2f524ad50eb840c5137eb80a45aa6ea8
  Author: Bastien Nocera <hadess@hadess.net>
  Date:   2026-05-05 (Tue, 05 May 2026)

  Changed paths:
    M tools/mpris-proxy.c

  Log Message:
  -----------
  mpris-proxy: Avoid session->obex dereference

Protect against trying to access session->obex variable that might be
null after failure to create the session object.


Compare: https://github.com/bluez/bluez/compare/897689ffe9c0%5E...79975d8d2f52

To unsubscribe from these emails, change your notification settings at https://github.com/bluez/bluez/settings/notifications