buglwIP - A Lightweight TCP/IP stack - Bugs: bug #38165, socket with mulicast

 
 

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

bug #38165: socket with mulicast

Submitter:  hanhui <hanhui03>
Submitted:  Fri 25 Jan 2013 03:34:30 AM UTC
   
 
Category:  sockets/netconn Severity:  3 - Normal
Item Group:  Feature Request Status:  Fixed
Privacy:  Public Assigned to:  goldsimon
Open/Closed:  Closed Planned Release:  None
lwIP version:  CVS Head

Wed 25 Feb 2015 08:31:12 PM UTC, comment #3: 

2) Fixed by tracking IP_ADD/DROP_MEMBERSHIP calls and dropping from lwip_close().

1) is OK for me.

Simon Goldschmidt <goldsimon>
Group administrator
Wed 26 Feb 2014 08:44:22 PM UTC, comment #2: 


> when close a socket which has already join a group, lwip MUST
> first leave this group then delete socket.


This is not a simple thing to do, as it would require to keep an array of joined groups addresses/netif per socket (a pointer to the calling socket per group does not work as multiple sockets can join the group).

As such, personally, I'd be happy with dropping this in favour for being lightweight: I wouldn't expect lwIP to be used this way...

Simon Goldschmidt <goldsimon>
Group administrator
Wed 26 Feb 2014 08:38:35 PM UTC, comment #1: 


> setsockopt() to join or leave a group, must use:
> netconn_join_leave_group();
> and do NOT use igmp_joingroup() or igmp_leavegroup()!


Why? Isn't the IPv4 code the same? And as far as I read it, IP_ADD_MEMBERSHIP/IP_DROP_MEMBERSHIP are meant for IPv4 only, not for IPv6 as netconn_join_leave_group() handles it.

Simon Goldschmidt <goldsimon>
Group administrator
Fri 25 Jan 2013 03:34:30 AM UTC, original submission:  

1:

setsockopt() to join or leave a group, must use:

netconn_join_leave_group();

and do NOT use igmp_joingroup() or igmp_leavegroup()!

2:

when close a socket which has already join a group, lwip MUST

first leave this group then delete socket.

hanhui <hanhui03>

 

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

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by goldsimon (Posted a comment)
  • -email is unavailable- added by hanhui03 (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 4 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2015-02-25 goldsimon StatusNeed Info Fixed
        Assigned toNone goldsimon
        Open/ClosedOpen Closed
    2014-02-26 goldsimon StatusNone Need Info

    Back to the top

    Powered by Savane 3.13-d3ae.
    Corresponding source code