Some mbuf related fixes in icmp_error()

* check mbuf length before doing mtod() and accessing to IP header;
* update oip pointer and all depending pointers after m_pullup();
* remove extra checks and extra parentheses, wrap long lines;

PR:		[[ https://bugs.freebsd.org/222670 | 222670 ]]