vsnprintf may of returned random crap, but talloc is consistent
Only set ARP dev if interface is known
Emit warnings when were degrading to single threaded build
Move to talloc

The edited arrays should really be statically allocated for a REQUEST,
and re-used.  Re-allocing memory is bad.
Rules for clang analyzer

do "make scan" for a LONG build, with lots of scanning,
and (for now) a number of errors.
Use clang analyzer macro to tell it about asserts
Clean rules for scan
Be more careful about initialization in a loop
One more minor assert
Realm struct to talloc
Don't need to assign result of talloc_steal
Removed duplicate code
Partial conversion to talloc.

It still needs to establish the best / correct parent / child
relationship.  But for now it works
Remove unnecessary assignment
Removed unnecessary assignment
Removed unused assignments
Assert to quiet clang analyzer
Assert to quiet clang analyzer
Rearrange code to quiet clang analyzer
Remove unused assignment