Skip to content

accept LLDP messages containing padding#6151

Merged
poettering merged 2 commits intosystemd:masterfrom
speakinghedge:6150-LLDP-messages-containing-padding-are-dropped
Jun 23, 2017
Merged

accept LLDP messages containing padding#6151
poettering merged 2 commits intosystemd:masterfrom
speakinghedge:6150-LLDP-messages-containing-padding-are-dropped

Conversation

@speakinghedge
Copy link

fixes #6150

@poettering
Copy link
Member

Any chance you can rework this adding a comment as replacement for the old check, that says something like:

/* Note that we permit trailing garbage here, as that's mandated by … */

i.e. just so that the next one reading this isn't suprised about the lax checks....

@poettering
Copy link
Member

looks goot otherwise

@poettering poettering added the reviewed/needs-rework 🔨 PR has been reviewed and needs another round of reworks label Jun 21, 2017
@poettering poettering merged commit a877b1d into systemd:master Jun 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lldp reviewed/needs-rework 🔨 PR has been reviewed and needs another round of reworks

Development

Successfully merging this pull request may close these issues.

LLDP messages containing padding are dropped

2 participants