13 Aug
2025
13 Aug
'25
4:38 p.m.
On 13/08/2025 04:41, David Gibson wrote:
@@ -127,6 +127,7 @@ size_t iov_to_buf(const struct iovec *iov, size_t iov_cnt, /* copying data */ for (copied = 0; copied < bytes && i < iov_cnt; i++) { size_t len = MIN(iov[i].iov_len - offset, bytes - copied); + /* NOLINTNEXTLINE(clang-analyzer-core.NonNullParamChecker) */ This suppression worries me slightly - I don't really understand why clang would be complaining here in the first place.
Honestly, I think it's a bug in clang analyzer, there is no reason to guess that iov[i].iov_base will be NULL. It's why I put NOLINTNEXTLINE. To be sure, I'm going to put an ASSERT(iov[i].iov_base) to shutdown the warning. Thanks, Laurent