pf: Use pf_map_addr() only once when choosing source port and address

[vegeta_tuxpowered.net]

Summary

When choosing source port and address for NAT operations the proper order of
operations is:

1. Try to get them from udp_mapping if rule has PF_POOL_ENDPI. This might be enough to return.
2. Get IP address from pf_map_addr_sn()
3. Look for free ports for the IP address
4. Get another IP address from pf_map_addr() if no ports are free

Calling pf_map_addr_sn() before checking udp_mappings is not necessary,
remove the first call. Since now a rule can have multiple pools, don't
hardcode pools anymore, always use the pool given in pf_get_sport() call.

[kp]

Approved too, but make sure to use your @FreeBSD.org e-mail address as author, otherwise the pre-commit triggers will complain.