Fix: attrd: memory leak in legacy attrd's CIB callback

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

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