#109 Fix for corrupt test-pcap

已合併
carlos.garcia 6 年之前 將 1 次代碼提交從 stefan.schmidt/reg_test_fix合併至 SPIN/master

The usage of IP options by MembersMgmtCommAttack triggered a bug in libtins, which caused it to corrupt the contained UDP-header during the merging of the source and attack pcaps. This resulted in a partially overwritten source port in the UDP header, in this case, all packets containing the source port 36851 now had 243 as their source port (when you set the most significant byte of 36851 to zero you get 243).

The included result which the test compared to also contained this corruption, making the test break on systems unaffected by the bug. This commit simply replaces the corrupted pcap with a fixed one.

The bug was reported and fixed upstream but it will take time to trickle down to distribution packages. If the test-suite fails with AssertionError: Packets 40 differ in field UDP.sport: 243 != 36851, then you're running a version of libtins which contains the bug.

The usage of IP options by MembersMgmtCommAttack triggered a bug in libtins, which caused it to corrupt the contained UDP-header during the merging of the source and attack pcaps. This resulted in a partially overwritten source port in the UDP header, in this case, all packets containing the source port 36851 now had 243 as their source port (when you set the most significant byte of 36851 to zero you get 243). The included result which the test compared to also contained this corruption, making the test break on systems unaffected by the bug. This commit simply replaces the corrupted pcap with a fixed one. The bug was reported and fixed [upstream](https://github.com/mfontanini/libtins/issues/288) but it will take time to trickle down to distribution packages. If the test-suite fails with ```AssertionError: Packets 40 differ in field UDP.sport: 243 != 36851```, then you're running a version of libtins which contains the bug.
該合併請求已經成功合併!
Sign in to join this conversation.
未選擇標籤
Bug
未選擇里程碑
未指派成員
1 參與者
正在加載...
取消
保存
尚未有任何內容