[Bug 572057] New: ident code has ipv6 and ipv4 port parsing reversed

[Red Hat Bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: ident code has ipv6 and ipv4 port parsing reversed

https://bugzilla.redhat.com/show_bug.cgi?id=572057

           Summary: ident code has ipv6 and ipv4 port parsing reversed
           Product: Fedora
           Version: 12
          Platform: All
        OS/Version: Linux
            Status: NEW
          Severity: medium
          Priority: low
         Component: xgridfit
        AssignedTo: nicolas.mailhot(a)laposte.net
        ReportedBy: redhat3(a)zacglen.com
         QAContact: extras-qa(a)fedoraproject.org
                CC: nicolas.mailhot(a)laposte.net,
                    fonts-bugs(a)lists.fedoraproject.org
    Classification: Fedora


Created an attachment (id=399012)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=399012)
Swap reversed IPV4 and IPV6 code fragments

Description of problem:
There is code in xinetd/ident.c which wrongly reverses the treatment of ipv4 vs
ipv6. I have attached a patch but to find the code just search for the first
AP_INET and read.

Version-Release number of selected component (if applicable):
xinetd-2.3.14-29

How reproducible:
Always

Steps to Reproduce:
1. see patch

Actual results:
No problem. Code seems to work ok.

Expected results:
No problem.

Additional info:
The first AF_INET and AF_INET6 should be swapped

It is a fortunately coincidence that the sin_port and sin6_port elements are
always the first element and always have the same size.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.