[seccomp/libseccomp] 861e2d: tests: fix 54-live-binary_tree to use binary tree

2 views
Skip to first unread message

Kir Kolyshkin

unread,
Mar 15, 2022, 10:40:40 PM3/15/22
to libse...@googlegroups.com
Branch: refs/heads/release-2.5
Home: https://github.com/seccomp/libseccomp
Commit: 861e2dc0945c1c7994e3e18ac5f1249aac3c2c1e
https://github.com/seccomp/libseccomp/commit/861e2dc0945c1c7994e3e18ac5f1249aac3c2c1e
Author: Kir Kolyshkin <koly...@gmail.com>
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 <tom.hr...@oracle.com>
Signed-off-by: Kir Kolyshkin <koly...@gmail.com>
[PM: restyled the "Fixes" tag]
Signed-off-by: Paul Moore <pa...@paul-moore.com>

(imported from commit 5731b3c338f8f18b1d2b3aa300bbcb97af0fb34c)


Reply all
Reply to author
Forward
0 new messages