diff options
| author | Vitaly Lavrov <vel21ripn@gmail.com> | 2022-03-08 02:20:56 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-03-08 00:20:56 +0100 |
| commit | a1451935b8653adc830ee4cb827def3622fb02d6 (patch) | |
| tree | 7056ae6059f3a4126afec650420654cba0f44e66 /.github | |
| parent | c345b3c7af89957ef4bc55e2ccf1b1a4bc724f3a (diff) | |
Errors fixed (#1482)
Fixed errors for bigendian platforms in ndpiReader.
All address and port comparisons and hash calculations are done with
endian in mind.
The get_ndpi_flow_info() function searched for an existing flow for the
forward and reverse direction of the packet.
The ndpi_workflow_node_cmp() function looked for a flow regardless of
the packet's direction. This is what led to an error in determining the
direction of transmission of the packet.
Fixed error in "synscan" test: the number of packets in the forward and
reverse direction is incorrectly defined (verified via tcpdump).
Fixed bug with icmp protocol checksum check for big endian platforms.
Diffstat (limited to '.github')
| -rw-r--r-- | .github/workflows/build.yml | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 8fa5655cd..cb48a6456 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -305,10 +305,10 @@ jobs: "uname -a && lscpu | grep Endian " - - name: Configure and compile (no tests) using qemu for the specified architecture (s390x - big endian) + - name: Configure and compile using qemu for the specified architecture (s390x - big endian) if: startsWith(matrix.os, 'ubuntu') && startsWith(matrix.arch, 's390x') uses: docker://multiarch/ubuntu-core:s390x-bionic - with: #./tests/do.sh disabled because we know we have some problems with big-endian machines + with: args: > bash -c "apt-get -y update && @@ -318,5 +318,6 @@ jobs: make -C example ndpiSimpleIntegration && make -C rrdtool && make -C python && + ./tests/do.sh && ./tests/do-unit.sh " |