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

Replace double quote character in DHCP client ID. Issue #10295 #4211

Merged
merged 1 commit into from Mar 9, 2020

Conversation

vktg
Copy link
Contributor

@vktg vktg commented Feb 29, 2020

The following is allowed by the webgui in a static mapping: Client Identifier: 32" Sony Trinitron
That creates a configuration that cannot be loaded:

host s_lan_0 {
        hardware ethernet 01:02:03:04:05:06;
        option dhcp-client-identifier "32" Sony Trinitron";
    option host-name "32-Sony";
    option domain-name-servers 172.25.228.1;
}

This fix replaces the double quote character with '&quot';
There is no issue with other special characters (',./!@#$%^&*()~)

@jim-p
Copy link
Contributor

jim-p commented Feb 29, 2020

Does escaping the double quote as \" work? That seems like a better choice if it does.

@vktg
Copy link
Contributor Author

vktg commented Mar 5, 2020

It works with \" too
Replaced

@netgate-git-updates netgate-git-updates merged commit b23b28e into pfsense:master Mar 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants