buglwIP - A Lightweight TCP/IP stack - Bugs: bug #48359, dns entries are not set to state...

 
 

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

bug #48359: dns entries are not set to state DONE

Submitted by:  Simon Goldschmidt <goldsimon>
Submitted on:  Fri 01 Jul 2016 11:17:20 AM UTC  
 
Category: NoneSeverity: 3 - Normal
Item Group: Crash ErrorStatus: Fixed
Privacy: PublicAssigned to: Simon Goldschmidt <goldsimon>
Open/Closed: ClosedPlanned Release: 2.0.0 Beta2
lwIP version: git head

Mon 04 Jul 2016 11:17:40 AM UTC, comment #2:

Have seen a problem with the 1.4.1 code after setting the state to DONE and then calling the 'found' callback. If this callback re-enters the dns code then dns_equeue might reuse the entry but after the return from this callback the state might be set to UNUSED. It is not clear if this could happen in the current code but it seems possible?

Douglas <iosen>
Mon 04 Jul 2016 08:22:48 AM UTC, comment #1:

Fixed, thanks for reporting.

Simon Goldschmidt <goldsimon>
Project AdministratorIn charge of this item.
Fri 01 Jul 2016 11:17:20 AM UTC, original submission:

Found by Sandra Gilge on lwip-devel:
This has been introduced when fixing bug #47459 (in march 2016): dns_correct_response() does not set the entry's state to DNS_STATE_DONE, so dns_tmr() tries to send the requests again (which fails because the pcb is already removed).

Simon Goldschmidt <goldsimon>
Project AdministratorIn charge of this item.

 

No files currently attached

 

Depends on the following items: None found

Items that depend on this one: None found

 

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

    Date Changed By Updated Field Previous Value => Replaced By
    Mon 04 Jul 2016 08:22:48 AM UTCgoldsimonItem GroupNone=>Crash Error
      StatusNone=>Fixed
      Assigned toNone=>goldsimon
      Open/ClosedOpen=>Closed

    Back to the top


    Powered by Savane 3.1-cleanup1