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

Submitted by:  Enrico Lehmann <elehmann>
Submitted on:  Thu 03 May 2012 08:46:52 PM UTC  
 
Category: IPv6Severity: 3 - Normal
Item Group: Faulty BehaviourStatus: Invalid
Privacy: PublicAssigned to: None
Open/Closed: ClosedPlanned 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>
Project 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>

 

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
  • -unavailable- added by goldsimon (Updated the item)
  • -unavailable- added by idelamer (Posted a comment)
  • -unavailable- added by elehmann (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 3 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Fri 11 May 2012 08:57:30 PM UTCgoldsimonOpen/ClosedOpen=>Closed
    Fri 04 May 2012 11:39:06 PM UTCidelamerStatusNone=>Invalid
    Thu 03 May 2012 08:46:52 PM UTCelehmannAttached File-=>Added nd6.patch, #25775

    Back to the top


    Powered by Savane 3.1-cleanup1