Expand use of std::unique_ptr

[gaul]

This eliminates the potentially-unsafe heap deallocation in the signal handler.

[spandruvada]

merged to thermald-next

[spandruvada]

Caused coverity error:

[spandruvada]

	gboolean thd_dbus_interface_reinit(PrefObject *obj, GError **error) {
		bool exclusive_control = false;
	

(1) Event cond_true:	Condition "thd_engine->get_control_mode() == EXCLUSIVE", taking true branch.

[spandruvada]

The three pull requests in thermald-next caused coverity errors:
Without them
Defect occurrences found : 5 Total
1 COPY_INSTEAD_OF_MOVE
2 PASS_BY_VALUE
2 USE_AFTER_MOVE

"Make borrowing of pointer clearer for cppcheck"
"Expand use of std::unique_ptr"
"Convert pthread_mutex_t to C++11 std::mutex"

With those three commits :
Defect occurrences found : 9 Total
1 COPY_INSTEAD_OF_MOVE
1 LOCK_EVASION
2 MISSING_LOCK
2 PASS_BY_VALUE
1 RESOURCE_LEAK
2 USE_AFTER_MOVE

So please recheck the three commits. I can remove and reapply.

[gaul]

How can I access the Coverity report?

[gaul]

I might understand the RESOURCE_LEAK without the report. I expect the global destructor for thd_engine to run but perhaps the call to exit confuses Coverity? It would be better if thermald interrupted the event loop to exit but let me make the smaller change first.