Move hmac_key to "inst"

There's no reason to have it a global
Add TALLOC_CTX to pairlist_read()
simplify module detach

Now that pairlist_read() takes a talloc context, modules
which have nothing but PAIR_LISTs in their config don't need
a detach method.  The pairlists are free'd when the instance
data is free'd