#109 Fix for corrupt test-pcap

Fusionado
carlos.garcia fusionados 1 commits de stefan.schmidt/reg_test_fix en SPIN/master hace 6 años

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.
¡Este Pull Request se ha completado con éxito!
Inicie sesión para unirse a esta conversación.
Sin etiquetas
Bug
Sin Milestone
Sin asignado
1 participantes
Cargando...
Cancelar
Guardar
Aún no existe contenido.