Fix: attrd: memory leak in CIB callback

This fixes a memory leak detected via routine valgrind testing.
Previously, attrd's CIB callback would free() the supplied user_data,
but that would not catch cases where the callback was implicitly
removed.

Now, attrd uses the new register_callback_full() method to supply
a free function instead.