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

ipidseq.nse uses packet.u16 function that is marked as local #2464

Closed
rainoko opened this issue Apr 10, 2022 · 4 comments
Closed

ipidseq.nse uses packet.u16 function that is marked as local #2464

rainoko opened this issue Apr 10, 2022 · 4 comments
Assignees
Labels

Comments

@rainoko
Copy link

@rainoko rainoko commented Apr 10, 2022

table.insert(ipids, packet.u16(layer3, 4))

packet.u16(...) is marked as local (it is right thing to do to achieve correct encapsulation) and cannot be used anymore.
Maybe separate module for helper functions should be introduced.

@nnposter nnposter self-assigned this Apr 10, 2022
@nnposter nnposter added bug NSE script labels Apr 10, 2022
@nnposter
Copy link

@nnposter nnposter commented Apr 10, 2022

Thank you for reporting the issue and identifying the root cause. A fix has been committed as r38371.

@LiJulle
Copy link

@LiJulle LiJulle commented Apr 29, 2022

Hi,
I think I am encountering the same error.
With root and the command "nmap -p80 -d2 --script ipidseq -iR10" I keep getting the error "variable 'u16' is not declared". I am not able to find r38371 is it a new version? I am running the newest version that comes with Kali.
I am probably doing a stupid mistake but I'm new to nmap and help would be much appreciated.
Thanks
Jules

@nnposter
Copy link

@nnposter nnposter commented Apr 29, 2022

r38371 is not a version but an SVN commit. There is not yet an official version of Nmap that contains this update. Instead, you can simply replace your local copy of the file with what is in GitHub or SVN.

@LiJulle
Copy link

@LiJulle LiJulle commented May 3, 2022

Yes thank you now it’s working fine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants