[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[lwip-devel] [bug #26487] igmp_ip_output_if uses IP_HDRINCL at the wrong

From: Simon Goldschmidt
Subject: [lwip-devel] [bug #26487] igmp_ip_output_if uses IP_HDRINCL at the wrong place
Date: Thu, 07 May 2009 18:09:11 +0000
User-agent: Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; de; rv: Gecko/2009042315 Firefox/3.0.10


                 Summary: igmp_ip_output_if uses IP_HDRINCL at the wrong
                 Project: lwIP - A Lightweight TCP/IP stack
            Submitted by: goldsimon
            Submitted on: Do 07 Mai 2009 18:09:09 GMT
                Category: IPv4
                Severity: 3 - Normal
              Item Group: Faulty Behaviour
                  Status: None
                 Privacy: Public
             Assigned to: None
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: 
            lwIP version: CVS Head



pbuf_header(p, IP_HLEN) is executed even if IP_HDRINCL is given, which
results in a wrong payload.

The reason for posting this here and not fixing directly is I'd rather solve
this more cleanly: I think we should provide a function in ip.c to include IP
options in an IP header (which is what igmp_ip_output_if needs) instead of
duplicating the ip_output_if code in igmp_ip_output_if.

This would both make the code smaller and prevent bugs getting fixed in one
place while staying in the other. Also, the current igmp_ip_output_if code
omits stats, snmp and loopback!

I'd add the function ip_output_if_options() that adds 2 arguments (void
*options, int optlen) to ip_output_if().


Reply to this item at:


  Nachricht geschickt von/durch Savannah

reply via email to

[Prev in Thread] Current Thread [Next in Thread]