This set of patches is the result of my taking a second look at the sbus
code. One of the driving factors was the removal of talloc_reference()
given the uncertainty about this API upstream.
During the review I decided to rationalize the use of names, combined
server and connection structures so that one single set of watch
functions can be used without the need of additional placeholder
structures (I've done this in 2 steps with a placeholder structure
first).
Finally I found some minor bugs and potential memory leaks I resolved in
patches. One notable bug that was hiding in the code is that the
responder context was holding only one sbus_connection structure and we
were assigning to it both the monitor and the dp connection structures.
We don't reference the monitor connection after we instantiate it so if
the monitor connection was set up first we wouldn't notice it.
With my patches I started seeing that the dp connection was set up first
some times and this led to failures as messages intended for the data
provider were received by the monitor, and it would reply with a Method
Unknown error of course.
Simo.