Hello Dan, On Wednesday, 23 April 2025 10:21:18 CEST Dan Carpenter wrote: > When the list_for_each_entry_reverse() exits without hitting a break > then the list cursor points to invalid memory. So this check for > if (c2a->fixed) is checking bogus memory. Fix it by using a "found" > variable to track if we found what we were looking for or not. IIUC the for loop ending condition in list_for_each_entry_reverse() is "!list_entry_is_head(pos, head, member);", so even if the loop runs to completion, the pointer should still be valid right? Thanks, -- Romain Gantois, Bootlin Embedded Linux and Kernel engineering https://bootlin.com
Attachment:
signature.asc
Description: This is a digitally signed message part.
