While removing an unrelated patch, Alexander noticed we had an unused
temporary talloc context. This patch removes it.
I didn't remove the done: label (in favor of just returning ret), it
cases no harm and future additions to the function might need it again.