Use-after-free if sendCmdBuffer fails

The call class deletes itself on failure so we really can't continue
with the method. Needs to abort immediately. (Seriously, the class is
calling delete this? That's just asking for pain).

Caught by clang-analyzer.
You cannot call computeStat after abortCall.

call::abortCall also does delete this. Which means you can't call
call::computeStat safely after the fact. Another use-after-free bug.

Caught by clang-analyzer.
Correct the return from connect_to_peer

connect_to_peer is returning stack-allocated memory. The stack allocated
T_peer_infos structure technically means a copy from the iterator,
meaning that we can't really return the address of peer_socket safely.

Caught by clang-analyzer.
Fix garbage/undefined load on E_Message_Server_IP

SOCK_ADDR_SIZE can't be used here because server_sockaddr. The
structure is uninitialized so we can't look up the ss_family.

It's just fine calling getsockname with sizeof the full
sockaddr_storage structure, the macro is overkill. I wonder if that's
the case in other places.

Caught by clang-analyzer.
Fix potential memory leak after second strtok

Memory is allocated before we know if anything is there to copy into
it. Flat out rewritten.

Caught by clang-analyzer.
Fix potential memory leak when HAVE_PCAP_NEXT_EX isn't defined

We malloc pkthdr, we must free it too.

Caught by clang-analyzer.
Free the existing M_regularExpression if its already set

Potential memory leak.

Caught by clang-analyzer.