[GitHub] [incubator-nuttx-apps] normanr commented on pull request #904: netutils/dhcpc: add non-blocking interface

[GitBox <gi...@apache.org>]

normanr commented on PR #904:
URL: https://github.com/apache/incubator-nuttx-apps/pull/904#issuecomment-1166455202

   fyi: I tried to use this code (adding a `dhcpc` daemon based on the `ntpc` daemon), but it has a couple of nasty bugs:
   - it doesn't renew dhcp leases, but rather does full re-discovery and request (as if it had never seen the network before),
   - you can't start `dhcpc_request_async` from `netinit_net_bringup`, and later call `dhcpc_close` from nsh:
     - the socket isn't closed properly, not sure why (it works fine when calling `dhcpc_request_async` from nsh),
   - the interface name stored in `dhcpc_state_s` isn't duplicated, so the name is lost when the caller frees the underlying string,
   - sometimes it sees a double-offer, it it will decline it even if it's just a duplicate of the offer that it's just received.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org