Re: [PATCH net-next 1/2] tools: ynl: check for null ptr on dump free
Jakub Kicinski <[email protected]> Tue, 4 Aug 2026 14:25:53 -0700
| Newsgroups | gmane.linux.kernel,gmane.linux.network |
|---|---|
| Message-ID | <[email protected]> |
On Mon, 3 Aug 2026 20:16:51 +0000 Thaison Phan wrote: > Static analysis What's the tool? I guess we can do this but I'd factor it out to a separate check at the start of the function. Otherwise it reads as if there could be a NULL on the list itself. FWIW NULL is not an empty list, it's a dump error. So the static tool / LLM is definitely wrong. But I guess free(NULL) is supposed to be a noop in general so whatever.