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.

Sign up for GitHub

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

Jump to bottom

MDNS_NAME_BUF_LEN need to be in include/mdns.h (IDFGH-14326) #724

Open
3 tasks done
jonsmirl opened this issue Dec 31, 2024 · 1 comment
Open
3 tasks done

MDNS_NAME_BUF_LEN need to be in include/mdns.h (IDFGH-14326) #724

jonsmirl opened this issue Dec 31, 2024 · 1 comment
Labels
Status: Opened Issue is new

Comments

@jonsmirl
Copy link

Answers checklist.

  • I have read the documentation for esp-protocols components and the issue is not addressed there.
  • I have updated my esp-protocols branch (master or release) to the latest version and checked that the issue is present there.
  • I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report

image
image
@espressif-bot espressif-bot added the Status: Opened Issue is new label Dec 31, 2024
@github-actions github-actions bot changed the title MDNS_NAME_BUF_LEN need to be in include/mdns.h MDNS_NAME_BUF_LEN need to be in include/mdns.h (IDFGH-14326) Dec 31, 2024
@david-cermak
Copy link
Collaborator

This is correct, need to provide the maximum size in public headers if we require users to allocate buffers for this API.
Thank for the report, will fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Status: Opened Issue is new
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jonsmirl @david-cermak @espressif-bot