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

Fix LLDP packet format #25192

Merged

Conversation

jukkar
Copy link
Member

@jukkar jukkar commented May 11, 2020

The LLDP net_pkt was not created as the code did not properly check the return value of net_pkt_write() call.

Fixes #25184

Make sure that net_pkt_write() sets the LLDPDU data correctly
in net_pkt.

Fixes zephyrproject-rtos#25184

Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
Copy link
Collaborator

@tbursztyka tbursztyka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

stuff like that are never caught since we don't have lldp unit tests

@jukkar
Copy link
Member Author

jukkar commented May 11, 2020

stuff like that are never caught since we don't have lldp unit tests

Yep, the LLDP stuff was bitrotted. We could now have tests for it using the docker image that @pfl created. @laperie can you allocate some task for this?

@jukkar jukkar added this to the v2.3.0 milestone May 11, 2020
Make sure that we have proper network interface which support
VLAN and LLDP in this sample.

Signed-off-by: Jukka Rissanen <jukka.rissanen@linux.intel.com>
@jukkar jukkar force-pushed the issue-25184-lldp-pkt-write branch from 09f5d7a to edde183 Compare May 11, 2020 12:51
@carlescufi carlescufi merged commit 5c62afd into zephyrproject-rtos:master May 11, 2020
@jukkar jukkar deleted the issue-25184-lldp-pkt-write branch May 12, 2020 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

lldp: lldp_send includes bug
5 participants