Branch: refs/heads/main
  Home:   https://github.com/seccomp/libseccomp
  Commit: 5731b3c338f8f18b1d2b3aa300bbcb97af0fb34c
      
https://github.com/seccomp/libseccomp/commit/5731b3c338f8f18b1d2b3aa300bbcb97af0fb34c
  Author: Kir Kolyshkin <[email protected]>
  Date:   2022-03-15 (Tue, 15 Mar 2022)

  Changed paths:
    M tests/54-live-binary_tree.c
    M tests/54-live-binary_tree.py

  Log Message:
  -----------
  tests: fix 54-live-binary_tree to use binary tree

Apparently, an early implementation of the binary tree optimization
used to enable the feature when the number of rules added was > 16.

The code was later changed to add and use SCMP_FLTATR_CTL_OPTIMIZE,
but the 54-live-binary_tree test case was left as is. So, despite
its name, it is not testing the binary tree.

Fix this, and remove the comment that referred to the old
implementation.

Fixes: 38f04da84748 ("tests: add tests for the binary tree")
Reviewed-by: Tom Hromatka <[email protected]>
Signed-off-by: Kir Kolyshkin <[email protected]>
[PM: restyled the "Fixes" tag]
Signed-off-by: Paul Moore <[email protected]>


-- 
You received this message because you are subscribed to the Google Groups 
"libseccomp" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/libseccomp/seccomp/libseccomp/push/refs/heads/main/7a28df-5731b3%40github.com.

Reply via email to