Fix for MSTP packet display issue

[ACodingfreak]

Fix for output formatting
fix for issue with MSTP packets

[mcr]

Can you provide a test case for these changes (a pcap file).
I guess that there was not any MSTP examples in tests/ before, so this would be very much appreciated.

[ACodingfreak]

I will add an MSTP pcap file and update the same in test case and will drop
a pull request

Regards,

Ajith

codingfreak.blogspot.com http://www.codingfreak.blogspot.com

On Sun, Mar 10, 2013 at 11:11 PM, Michael Richardson <
[email protected]> wrote:

Can you provide a test case for these changes (a pcap file).
I guess that there was not any MSTP examples in tests/ before, so this
would be very much appreciated.

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/55#issuecomment-14685283
.

[infrastation]

@ajithadapa, any new commits you add to this branch count in the original pull request, even if you rebase the branch.

[mcr]

Can you rebase this and add the pcap file? Thanks.

[ACodingfreak]

Sorry but I am a bit confused as I am new to git.

Should I pull "upstream" changes using commands git fetch upstream and then
git merge upstream/master and then update my fork tree with the MSTP pcap file ?

or

Just add the MSTP pcap file to my current fork tree and do a pull request ?

[infrastation]

git fetch upstream
git rebase upstream/master

[mcr]

Merged manually.

[mcr]

Well, actually, I didn't get it to compile properly, so I removed the code.
What is the length variable at line 338?