patchlwIP - A Lightweight TCP/IP stack - Patches: patch #9429, mdns: added support for searching...


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

patch #9429: mdns: added support for searching services

Submitted by:  David GIRAULT <dgirault>
Submitted on:  Mon 07 Aug 2017 01:30:45 PM UTC  
Category:  DNS Priority:  5 - Normal
Status:  None Privacy:  Public
Assigned to:  None Open/Closed:  Open
Planned Release:  None

Wed 10 Jan 2018 02:57:07 PM UTC, comment #5:

Erik, after reading the rfc, I saw the point. My mdns_search() need to use another udp_pcb to send the request. A new temporary udp_pcb with the same recv handle should do the job.

David GIRAULT <dgirault>
Tue 09 Jan 2018 01:17:56 PM UTC, comment #4:

This patch allow search for services like `_workstation._tcp.local` using PTR type query.

DNS module only support querying IP for a specified hostname (A or AAAA queries).

David GIRAULT <dgirault>
Mon 08 Jan 2018 09:02:43 PM UTC, comment #3:

The DNS module already supports sending queries for the .local. domain to the MDNS multicast address. What does this add?

It sends queries with source port 5353 even though it is not a full MDNS capable querier. This is not allowed by the RFC:
(rfc6762 section 5.1)
.... these queries MUST NOT be sent using UDP source port 5353, since using UDP source port 5353 signals the presence of a fully compliant Multicast DNS querier, as described below.

Erik Ekman <yarrick>
Project Member
Mon 08 Jan 2018 01:40:47 PM UTC, comment #2:

Since some mDNS work are ongoing, I thinks this path should be considered.

David GIRAULT <dgirault>
Mon 07 Aug 2017 01:53:02 PM UTC, comment #1:

Second patch, `0002-mdns-early-uncompress-domain-name-since-cannot-be-do.patch` is the only addition to original #9339 patch.

David GIRAULT <dgirault>
Mon 07 Aug 2017 01:30:45 PM UTC, original submission:

This new patchset was previously #9339, but this new one include only the search part (responder part already merged).

All comments previously posted on #9339 were already addressed.

David GIRAULT <dgirault>



Depends on the following items: None found

Items that depend on this one: None found


Carbon-Copy List
  • -email is unavailable- added by yarrick (Posted a comment)
  • -email is unavailable- added by dgirault (Submitted the item)

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

    Only logged-in users can vote.




    Follow 2 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2017-08-07 dgirault Attached File- => Added 0001-mdns-added-support-for-searching-services.patch, #41441
        Attached File- => Added 0002-mdns-early-uncompress-domain-name-since-cannot-be-do.patch, #41442

    Back to the top

    Powered by Savane 3.3