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

Submitter:  Simon Goldschmidt <goldsimon>
Submitted:  Fri 01 Jul 2016 11:17:20 AM UTC
   
 
Category:  None Severity:  3 - Normal
Item Group:  Crash Error Status:  Fixed
Privacy:  Public Assigned to:  goldsimon
Open/Closed:  Closed Planned 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>
Group administrator
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>
Group administrator

 

(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 iosen (Posted a comment)
  • -email is unavailable- added by goldsimon (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
    2016-07-04 goldsimon Item GroupNone Crash Error
        StatusNone Fixed
        Assigned toNone goldsimon
        Open/ClosedOpen Closed

    Back to the top

    Powered by Savane 3.13-02a9.
    Corresponding source code