intel / thermal_daemon

Thermal daemon for IA
GNU General Public License v2.0
539 stars 117 forks source link

thd_dbus_on_bus_acquired: fix segfault #429

Closed c4rlo closed 7 months ago

c4rlo commented 7 months ago

Fixes #428.

c4rlo commented 7 months ago

I have confirmed that this fixes the crash for me.

spandruvada commented 7 months ago

Please write a comment in the commit description, why this should prevent crash? This is not obvious how assigning NULL will help.

spandruvada commented 7 months ago

The original author of the code fixed this issue https://github.com/intel/thermal_daemon/commit/e49e4baf6ca12c647e8a4bc4e50743bc475d316a

Thanks for the pull request. This issue is addressed. If not fixed, please send again.