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

incompatible restrict options for Solaris 10 #60

Closed
Phil-Friderici opened this issue Feb 10, 2015 · 3 comments
Closed

incompatible restrict options for Solaris 10 #60

Phil-Friderici opened this issue Feb 10, 2015 · 3 comments

Comments

@Phil-Friderici
Copy link
Contributor

The module is using restrict options that are incompatible with Solaris 10. Because of that the complete line for restrict is ignored leading to an completly unrestricted NTP server.

Error messages show that the restrict lines are ignored (at all)

Feb  5 15:49:21 sol10 xntpd[35808]: [ID 504011 daemon.error] getnetnum: "-4" invalid host number, line ignored
Feb  5 15:49:21 sol10 xntpd[35808]: [ID 504011 daemon.error] getnetnum: "-6" invalid host number, line ignored
Feb  5 15:49:21 sol10 xntpd[35808]: [ID 504011 daemon.error] getnetnum: "::1" invalid host number, line ignored
Feb  5 15:49:21 sol10 xntpd[35808]: [ID 269952 daemon.error] can't open key file /etc/inet/ntp.keys: No such file or directory
Feb  5 15:49:21 sol10 xntpd[35808]: [ID 241925 daemon.error] configure: keyword "tinker" unknown, line ignored
Feb  5 15:49:24 sol10 xntpd[35808]: [ID 373468 daemon.error] adj_frequency: ntp_adjtime failed: Not owner

To restrict ntp on Solaris 10 these line would be sufficient:

restrict default noserve noquery
restrict 127.0.0.1

Should I create a PR with (new) valid defaults for Solaris 10 ?

@Phil-Friderici
Copy link
Contributor Author

wanna see the PR first ?

@ghoneycutt
Copy link
Owner

Any PR's in this area are of course certainly welcomed. :)

I don't have testing ability on Solaris 10, so if you are seeing errors, let's get 'em fixed!

@Phil-Friderici
Copy link
Contributor Author

@ghoneycutt thanks for merging :)

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

2 participants