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

From
Tanya Agarwal <hidden>
Date
2025-01-05 17:52:30
Also in
lkml
Reply to
Paul Moore, 2025-01-05
On Sun, Jan 5, 2025 at 11:18 PM Paul Moore [off-list ref] wrote:
On Jan  5, 2025 Tanya Agarwal [off-list ref] wrote:
quoted
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

Thank you, Paul.

Regards,
Tanya
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