2 Nov
2020
2 Nov
'20
8:27 a.m.
On 2020/11/2 下午3:24, Marek Behun wrote:
On Mon, 2 Nov 2020 08:27:16 +0800 Qu Wenruo wqu@suse.com wrote:
Thus I really tend to believe it's just a bug in coverity. All locations accessing @ii all have its key.type checked to ensure it get filled in the first place.
If this is a bug in coverity, we should fix coverity, not add extra code to U-Boot, even if it is just one instruction. That simply stinks the same way like when systemd crashed when "debug" parameter was present in /proc/cmdline, and they sent a patch to kernel which removed the "debug" parameter, instead of fixing systemd. IMO.
Yep, you're right.
Please discard this patch.
Thanks, Qu
Marek