diff options
| author | Enrico Bravi <enrico.bravi@polito.it> | 2024-05-29 20:14:34 +0200 |
|---|---|---|
| committer | Mimi Zohar <zohar@linux.ibm.com> | 2024-06-03 16:37:22 -0400 |
| commit | fbf06cee60876d50f259d0689e3c03940750f0d4 (patch) | |
| tree | 597000d6614e1c6409ace04612af4e937d406f1f /tools/testing/selftests/bpf/prog_tests/tcp_hdr_options.c | |
| parent | c3f38fa61af77b49866b006939479069cd451173 (diff) | |
ima: fix wrong zero-assignment during securityfs dentry remove
In case of error during ima_fs_init() all the dentry already created
are removed. {ascii, binary}_securityfs_measurement_lists are freed
calling for each array the remove_securityfs_measurement_lists(). This
function, at the end, assigns to zero the securityfs_measurement_list_count.
This causes during the second call of remove_securityfs_measurement_lists()
to leave the dentry of the array pending, not removing them correctly,
because the securityfs_measurement_list_count is already zero.
Move the securityfs_measurement_list_count = 0 after the two
remove_securityfs_measurement_lists() calls to correctly remove all the
dentry already allocated.
Fixes: 9fa8e7625008 ("ima: add crypto agility support for template-hash algorithm")
Signed-off-by: Enrico Bravi <enrico.bravi@polito.it>
Reviewed-by: Roberto Sassu <roberto.sassu@huawei.com>
Signed-off-by: Mimi Zohar <zohar@linux.ibm.com>
Diffstat (limited to 'tools/testing/selftests/bpf/prog_tests/tcp_hdr_options.c')
0 files changed, 0 insertions, 0 deletions