Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

int lwip_getaddrinfo() error (IDFGH-13115) #70

Open
greentd opened this issue Jun 24, 2024 · 3 comments
Open

int lwip_getaddrinfo() error (IDFGH-13115) #70

greentd opened this issue Jun 24, 2024 · 3 comments

Comments

@greentd
Copy link

greentd commented Jun 24, 2024

The function lwip_getaddrinfo(const char *nodename, const char *servname, const struct addrinfo *hints, struct addrinfo **res)
@https://github.com/espressif/esp-lwip/blob/4297782bf9e614be1bb8605f30d46af8697fab17/src/api/netdb.c
is wrong,

It causes the following test routine to fail to run

https://github.com/espressif/esp-modbus/tree/master/test/tcp/mb_tcp_slave

@github-actions github-actions bot changed the title int lwip_getaddrinfo() error int lwip_getaddrinfo() error (IDFGH-13115) Jun 24, 2024
@espressif-abhikroy
Copy link
Collaborator

@greentd Thanks for reporting this issue. We are fixing this issue and the fix will be available soon.

@AxelLin
Copy link
Contributor

AxelLin commented Aug 6, 2024

@greentd Thanks for reporting this issue. We are fixing this issue and the fix will be available soon.

@espressif-abhikroy How is the status now?

@espressif-abhikroy
Copy link
Collaborator

@greentd Thanks for reporting this issue. We are fixing this issue and the fix will be available soon.

@espressif-abhikroy How is the status now?

The issue was with handling of IPADDR_ANY or NULL in netdb api's. This bug was introduced with the code changes for the "multiple ip support in dns" feature(f174681).
The fix is under review and will be merged soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants