1473824 - Do not try to unregister HAL observers during shutdown

Status: RESOLVED FIXED

(Core :: Hardware Abstraction Layer (HAL), enhancement)

Description

[Gabriele Svelto [:gsvelto]]

While refactoring HAL observers I noticed that we do a couple of weird things:

- Observer lists are instanced lazily but if we unregister an observer from a list that has not been instanced already we don't instance it

- We delete the lists asyncrhonously when removing the last observers

I originally thought both decisions made perfect sense but besides the added complexity they hide a significant issue: we have code that tries to unregister observer which have never been registered very late during shutdown. That code happens to work because of the above but should be considered buggy. This bug is for fixing the affected code and removing the explicit observer list allocation/deallocation.

Comment 1

[Gabriele Svelto [:gsvelto]]

This was fixed in bug 1476250. We're not deleting the lists asynchronously anymore and if one tries to deregisters an observer after the HAL has been shut down it will assert.