[PATCH 4/4] fs: btrfs: initialize @ii in show_dir() to make coverity happy
Qu Wenruo
wqu at suse.com
Mon Nov 2 08:27:59 CET 2020
On 2020/11/2 下午3:24, Marek Behun wrote:
> On Mon, 2 Nov 2020 08:27:16 +0800
> Qu Wenruo <wqu at 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
>
More information about the U-Boot
mailing list