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.

Sign up for GitHub

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

Jump to bottom

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

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

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

rainoko opened this issue Apr 10, 2022 · 5 comments
Assignees
@nnposter
Labels
bug NSE script NSE script

Comments

@rainoko
Copy link

rainoko commented Apr 10, 2022

nmap/scripts/ipidseq.nse

Line 234 in 705ac8d

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 NSE script labels Apr 10, 2022
@nnposter
Copy link

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

@nmap nmap deleted a comment Apr 11, 2022
nmap-bot pushed a commit that referenced this issue Apr 14, 2022
@bonsaiviking
Refactor to avoid using packet.lua local functions. See #2464
2543f2d
@LiJulle
Copy link

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

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 commented May 3, 2022

Yes thank you now it’s working fine.

@bhanu5678
Copy link

what i have to do solve that problem. please explain me how to copy to local file

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@nnposter nnposter

Labels
bug NSE script NSE script
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@rainoko @nnposter @LiJulle @bhanu5678