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

Lua Error: TvbRange:uint() does not handle 8 byte integers #8

Closed
stapelberg opened this issue Oct 31, 2016 · 2 comments
Closed

Lua Error: TvbRange:uint() does not handle 8 byte integers #8

stapelberg opened this issue Oct 31, 2016 · 2 comments

Comments

@stapelberg
Copy link

I’m using the following capture file:

repro_mlvpn.pcap.gz

This definition results in an error:

local wssdl = require 'wssdl'

pkt = wssdl.packet
{
  len : u16();
  version : bits(4);
  flags : bits(6);
  reorder : bits(1);
  unused : bits(5);
  timestamp : u16();
  timestamp_reply : u16();
  flow_id : u32();
  seq : u64();
}

wssdl.dissect {
  udp.port:add {
    [5080] = pkt:proto('mlvpn', 'multi-link VPN')
  };
}

Whereas this definition works (without the seq member):

local wssdl = require 'wssdl'

pkt = wssdl.packet
{
  len : u16();
  version : bits(4);
  flags : bits(6);
  reorder : bits(1);
  unused : bits(5);
  timestamp : u16();
  timestamp_reply : u16();
  flow_id : u32();
}

wssdl.dissect {
  udp.port:add {
    [5080] = pkt:proto('mlvpn', 'multi-link VPN')
  };
}

Am I doing something wrong?

@Snaipe
Copy link
Owner

Snaipe commented Oct 31, 2016

It seems that support for integers larger than 32-bits got broken along the line (wssdl should call uint64, not uint). This should be quickly fixable.

Incidentally, this also shows that I should find a solution to test those feature.

@Snaipe
Copy link
Owner

Snaipe commented Oct 31, 2016

This is now fixed. Until this hits the next release, you can use bits(64) as a workaround.

@Snaipe Snaipe closed this as completed in 9e55b00 Oct 31, 2016
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