[lwip-devel] [bug #52763] mdns: move data off the stack and dynamically allocate data.

[Erik Ekman]

Follow-up Comment #8, bug #52763 (project lwip):

Sorry, we reverted the async send commit, so your patch is out of date again.

mdns_service_free() has a copy-paste error:
 if (service) {
   mem_free(name);
 }

I don't really see the point of keeping the host name in a separately
allocated block. Its lifetime is the same as the surrounding struct, and it
only makes a few bytes difference. Also, the name might have to be adjusted so
it is good to know the amount of space available. This also applies to the
char arrays in the service struct.

You cannot count on any string functions being available, especially strndup
which does an allocation. lwIP must not have any dependencies outside of what
the port is expected to provide. 

Please update the function comments in the cases where you change the method
signature.

The late declaration of err_t res will not compile on stricter systems.

The old mdnsapi_* functions are still in the header file.

Please split out the LWIP_MDNS_RESPONDER_QUEUE_ANNOUNCEMENTS changes into a
separate patch.

Does this function still compile without the int declaration?
 mdns_resp_remove_netif(struct netif *netif)
 {
-  int i;


The dns_hdr struct is 12 bytes, that should be small enough to keep it on the
stack.

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?52763>

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