[TS-4070] RemapProcessor Forward Mapping with Recv Port failing with TS-2157 changes

• Type: Bug
• Status: Closed
• Priority: Critical
• Resolution: Fixed
• Fix Version/s: 6.1.0
• Component/s: Network
• Labels: regression
• Backport to Version: 6.0.1
• Assignee: Eric Schwartz
• Reporter: Craig Schomburg
• Created: 11/Dec/15 20:55
• Updated: 22/Jan/16 19:21
• Resolved: 28/Dec/15 19:55

Description

During the rework of RemapProcessor.cc, RemapProcessor::setup_for_remap() as part of the TS-2157 changeset, the port access API appears to have been incorrectly modified to use the client_info.src_addr.host_order_port() API [source port, host order] instead of the client_info.dst_addr.port() [destination/receive port, network
order] API. This caused port based remapping based on the receive port to
fail with ATS 6.0.0.

Functionality was previously working with ATS 4.0.1.

See attached patch that was used to restore the ATS 4.0.1 functionality.

Looking for confirmation that this fix/patch was correct.