buglwIP - A Lightweight TCP/IP stack - Bugs: bug #36390, LWIP_ND6_QUEUING define is missing...

 
 

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

bug #36390: LWIP_ND6_QUEUING define is missing in nd6.c

Submitter:  Enrico Lehmann <elehmann>
Submitted:  Thu 03 May 2012 08:46:52 PM UTC
   
 
Category:  IPv6 Severity:  3 - Normal
Item Group:  Faulty Behaviour Status:  Invalid
Privacy:  Public Assigned to:  None
Open/Closed:  Closed Planned Release:  None
lwIP version:  CVS Head

Sun 06 May 2012 08:56:24 PM UTC, comment #2: 

Sorry - you are right.
I just was looking into an older version of the nd6.h file. The code works correct and there is no existing bug, so this entry can be closed.

Thanks for the review!

Enrico Lehmann <elehmann>
Fri 04 May 2012 11:39:06 PM UTC, comment #1: 

This shouldn't happen, the current code still uses the q field to store a single packet, even when queueing is turned off.

Are you sure you are using GIT Head? This was changed on March 22nd.

Ivan Delamer <idelamer>
Group Member
Thu 03 May 2012 08:46:52 PM UTC, original submission:  

In nd6 the Queuing feature can be switched on or off. Switching it off will create an compiler error since there is a missing #ifdef LWIP_ND6_QUEUING in the nd6_new_router() function. I added a patch which remove this error.

I'm using the GIT master branch.

Enrico Lehmann <elehmann>

 

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

Attached Files
file #25775:  nd6.patch added by elehmann (656B - 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 goldsimon (Updated the item)
  • -email is unavailable- added by idelamer (Posted a comment)
  • -email is unavailable- added by elehmann (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
    2012-05-11 goldsimon Open/ClosedOpen Closed
    2012-05-04 idelamer StatusNone Invalid
    2012-05-03 elehmann Attached File- Added nd6.patch, #25775

    Back to the top

    Powered by Savane 3.13-caa5.
    Corresponding source code