Junio C Hamano <gitster@xxxxxxxxx> writes:
> Having said all that ...
>
>> Make the call to `read_attr_from_buf` conditional on `buf` being
>> non-NULL, ensuring that `size` is not read if it's never set.
>
> ... this makes the logic at the caller crystal clear, so even if
> there are suboptimal checker that bothers us with false positives,
> the change itself justifies itself, I would say.
Well, "even if there were *no* MSAN or other issues wrt usage of size"
was what I wanted to say. Sorry for a noise.
>> } else {
>> buf = read_blob_data_from_index(istate, path, &size);
>> - stack = read_attr_from_buf(buf, size, path, flags);
>> + if (buf)
>> + stack = read_attr_from_buf(buf, size, path, flags);
>> }
>> return stack;
>
> Thanks.
