lwIP - A Lightweight TCP/IP stack - Bugs: bug #50325, DNS local host list exported...
You are not allowed to post comments on this tracker with your current authentication level.
bug #50325: DNS local host list exported functions lack versatility
Submitter: | Konstantin <tomcat_84> | ||
Submitted: | Thu 16 Feb 2017 09:54:25 AM UTC | ||
Category: | DNS | Severity: | 3 - Normal |
Item Group: | Change Request | Status: | Fixed |
Privacy: | Public | Assigned to: | goldsimon |
Open/Closed: | Closed | Planned Release: | None |
lwIP version: | 2.0.0 |
Jump to the original submission
Fri 17 Feb 2017 09:09:23 PM UTC, comment #11: |
Simon Goldschmidt <goldsimon> |
Fri 17 Feb 2017 02:31:24 PM UTC, comment #10: Thanks. |
Konstantin <tomcat_84> |
Fri 17 Feb 2017 09:10:20 AM UTC, comment #9: |
Simon Goldschmidt <goldsimon> |
Fri 17 Feb 2017 07:29:22 AM UTC, comment #8: Agree about TTL. It is not for locals list.
|
Konstantin <tomcat_84> |
Thu 16 Feb 2017 07:38:22 PM UTC, comment #7: To make it clear, I think TTL is out of scope for the "local dns" feature. It's meant to get locally configured, static entries into the table (much like the "hosts" file on some OSes). Anything beyond that should use DNS_LOOKUP_LOCAL_EXTERN(..).
|
Simon Goldschmidt <goldsimon> |
Thu 16 Feb 2017 03:48:41 PM UTC, comment #6: Of course, the user program could implement its own list (with add, remove, lookup and even TTLs), just using dns_local_addhost, when the user list grows, and dns_local_removehost, when entry expire due to TTL.
|
Konstantin <tomcat_84> |
Thu 16 Feb 2017 03:30:56 PM UTC, comment #5: Agree with undocoumented non-static as not a good choice.
|
Konstantin <tomcat_84> |
Thu 16 Feb 2017 02:37:12 PM UTC, comment #4: Undocumented non-static is a no-go. Good compilers warn you about that.
|
Simon Goldschmidt <goldsimon> |
Thu 16 Feb 2017 01:02:51 PM UTC, comment #3: Sorry for cofusing names, it was all about exported
|
Konstantin <tomcat_84> |
Thu 16 Feb 2017 11:38:39 AM UTC, comment #2: I can't say I understand what you mean...
|
Simon Goldschmidt <goldsimon> |
Thu 16 Feb 2017 10:11:52 AM UTC, comment #1: Sorry, for some reason the form posted without any contents except header, with no edit possibility found I post it as a following comment.
|
Konstantin <tomcat_84> |
Thu 16 Feb 2017 09:54:25 AM UTC, original submission:
|
Konstantin <tomcat_84> |
No files currently attached
Depends on the following items: None found
Items that depend on this one: None found
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 3 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2017-02-17 | goldsimon | Status | None | Fixed | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed |
Powered by Savane 3.13-f8d8.
Corresponding source code
I had a talk with Dirk about the iterator not providing an 'idx' in the callback. However, since the table can be 'const' (a const array in ROM) or 'dynamic' (a list with next pointers), we cannot just export the table (since it would be different for the const/dynamic cases.
If anyone has suggestions for improvement of the change, feel free to post here.