There are *very* few places where assuming a tmpl is talloced is a good idea

There are clearly two variants of some tmpl functions, one takes a stack allocated tmpl, the other doesn't.

We don't malloc memory when we don't have to, and we don't have to for tmpls or cursors or maps.
Mark map_verify as unused