Skip to content

Update for compatibility with latest WireShark, and fix MSTNAK handling bug#5

Open
dchauran wants to merge 2 commits intomarrold:masterfrom
dchauran:master
Open

Update for compatibility with latest WireShark, and fix MSTNAK handling bug#5
dchauran wants to merge 2 commits intomarrold:masterfrom
dchauran:master

Conversation

@dchauran
Copy link
Copy Markdown

While the existing code mostly works in the current version of WireShark there are a few issues that either cause errors or outright break some scenarios.

  1. The current version of LUA does not support info(). This was causing failures with MSTNAK packet processing. The fix is to use print().
  2. The current version of LUA does not support math.pow(), causing errors in calculating BER. The fix is to use ^ instead.
  3. There was a null (nil in LUA) reference issue with MSTNAK packets where we sometimes did not set the message, but then tried to reference it for the info column. The logic has been updated to consistently set this.

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

Successfully merging this pull request may close these issues.

1 participant