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

citrix-enum-servers-xml can return error #2192

Closed
tatulea opened this issue Nov 24, 2020 · 1 comment
Closed

citrix-enum-servers-xml can return error #2192

tatulea opened this issue Nov 24, 2020 · 1 comment
Assignees

Comments

@tatulea
Copy link

@tatulea tatulea commented Nov 24, 2020

Describe the bug
I have run the following scan:

NSE: citrix-enum-servers-xml against wordpress.com (192.0.78.9:443) threw an error!
/usr/local/bin/../share/nmap/nselib/citrixxml.lua:50: bad argument #1 to 'char' (value out of range)
stack traceback:
	[C]: in function 'string.char'
	/usr/local/bin/../share/nmap/nselib/citrixxml.lua:50: in function 'citrixxml.decode_xml_document'
	(...tail calls...)
	...al/bin/../share/nmap/scripts/citrix-enum-servers-xml.nse:37: in function <...al/bin/../share/nmap/scripts/citrix-enum-servers-xml.nse:35>
	(...tail calls...)

It seems like the citrixxml.lua has an error. For the scan run above the value of escaped_val is &#8212; and the value of hexval is 8212, which can not to converted to char using string.char.

Version info (please complete the following information):

  • OS: Ubuntu 20.04
  • Output of nmap --version: 7.80
@tatulea tatulea added the Nmap label Nov 24, 2020
@nnposter nnposter self-assigned this Nov 25, 2020
@nnposter nnposter added bug NSE and removed Nmap labels Nov 25, 2020
@nnposter
Copy link

@nnposter nnposter commented Nov 25, 2020

Thank you for reporting the issue. A fix has been committed as r38146. The implemented change will leave character references with codepoints higher than 255 untouched.

Loading

@nmap-bot nmap-bot closed this in c00f054 Nov 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants