Re: [PATCH v3] lockdown: Initialize array before use

From
Paul Moore <paul@paul-moore.com>
Date
2025-01-05 17:48:43
Also in
lkml
Reply to
Tanya Agarwal, 2025-01-05
On Jan  5, 2025 Tanya Agarwal [off-list ref] wrote:
The static code analysis tool "Coverity Scan" pointed the following
details out for further development considerations:
CID 1486102: Uninitialized scalar variable (UNINIT)
uninit_use_in_call: Using uninitialized value *temp when calling
strlen.

Conclusion:
Initialize array before use in lockdown_read() to satisfy the static
analyzer.

Fixes: 000d388ed3bb ("security: Add a static lockdown policy LSM")
Signed-off-by: Tanya Agarwal <redacted>
I removed the 'Fixes:' tag as the code isn't actually broken as we
discussed in previous iterations of this patch.  I also edited the
subject line and commit description a bit, but otherwise this is
fine.  Merged into lsm/dev.

--
paul-moore.com
Thread (4 messages) 4 messages, 3 authors, last reply 2025-01-05
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help