buglwIP - A Lightweight TCP/IP stack - Bugs: bug #56017, The addrlen get from getaddrinfo...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

bug #56017: The addrlen get from getaddrinfo was wrong for IPv6 domain name translation

Submitter:  Gao Qingshui <kevingao>
Submitted:  Thu 28 Mar 2019 07:43:51 AM UTC
   
 
Category:  DNS Severity:  3 - Normal
Item Group:  Faulty Behaviour Status:  Wont Fix
Privacy:  Public Assigned to:  None
Open/Closed:  Closed Planned Release:  None
lwIP version:  git head

Tue 07 May 2019 02:05:05 AM UTC, comment #2: 

why the addrlen should be set to the sizeof(sockaddr_storage)? I checked the GLibc and found the addrlen was set based on the address family, pls refer the source file getaddrinfo.c in glibc.

Gao Qingshui <kevingao>
Thu 28 Mar 2019 10:25:48 AM UTC, comment #1: 

The correct way is to use sockaddr_storage here. This is what Git HEAD is doing, it was fixed in commit cca758d332.

Sylvain Rochet <gradator>
Group Member
Thu 28 Mar 2019 07:43:51 AM UTC, original submission:  

The addrlen should be set based on the address family, for AF_INET, it should be sizeof(sockaddr_in), for AF_IENT6, it should be sizeof(sockaddr_in6)

Gao Qingshui <kevingao>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attached Files
file #46654:  set-addrlen-based-on-address-family.patch added by kevingao (1013B - application/octet-stream)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by dziegel (Updated the item)
  • -email is unavailable- added by gradator (Posted a comment)
  • -email is unavailable- added by kevingao (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 3 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-04-30 dziegel StatusNone Wont Fix
        Open/ClosedOpen Closed
    2019-03-28 kevingao Attached File- Added set-addrlen-based-on-address-family.patch, #46654

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code