[lwip-devel] [patch #8480] Fix handling of dns_seqno wraparound

[Kevin Cernekee]

URL:
  <http://savannah.nongnu.org/patch/?8480>

                 Summary: Fix handling of dns_seqno wraparound
                 Project: lwIP - A Lightweight TCP/IP stack
            Submitted by: cernekee
            Submitted on: Sat 14 Jun 2014 10:29:40 PM GMT
                Category: DNS
                Priority: 5 - Normal
                  Status: None
                 Privacy: Public
             Assigned to: None
        Originator Email: 
             Open/Closed: Open
         Discussion Lock: Any
         Planned Release: None

    _______________________________________________________

Details:

In dns_enqueue(), lseq represents the number of DNS transactions that have
occurred since dns_table[lseqi] was started.  This should be a positive
number.  But gcc promotes (dns_seqno - pEntry->seqno) to a signed int, causing
lseq to be compared against negative values in some situations:

  dns_seqno = 1
  dns_table[0].seqno =   0, state = DNS_STATE_ASKING
  dns_table[1].seqno = 253, state = DNS_STATE_DONE
  dns_table[2].seqno = 254, state = DNS_STATE_DONE
  dns_table[3].seqno = 255, state = DNS_STATE_DONE

In this case, lseqi == 0 and the function returns ERR_MEM, despite the fact
that entries 1..3 are available for reuse.

Fix this by explicitly casting the "age" result to a u8_t.




    _______________________________________________________

File Attachments:


-------------------------------------------------------
Date: Sat 14 Jun 2014 10:29:40 PM GMT  Name:
0001-dns-Fix-handling-of-dns_seqno-wraparound.patch  Size: 1kB   By: cernekee

<http://savannah.nongnu.org/patch/download.php?file_id=31547>

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/patch/?8480>

_______________________________________________
  Message sent via/by Savannah
  http://savannah.nongnu.org/