pf: pull icmp-nested headers into struct pf_pdesc

Rather than copying headers nested in an ICMP message into separate stack
variables copy them into struct pf_pdesc (pd2). That's one of the things the
struct is for.
This saves a trivial amount of stack space, but also makes future refactoring
easier.

Sponsored by: Rubicon Communications, LLC ("Netgate")