Fix inconsistent convention on when to destroy tx_data

[bennylp]

The convention now is inconsistent about when tx_data reference counter should be decremented on failure. The dialog send() function *always* decrements the reference counter no matter what, while transaction only decrements it if it is sent successfully.

The good solution perhaps to always decrement reference counter no matter what, to prevent memory leaks, but need to consult everybody about this.