On Mon, 2025-12-01 at 13:40 -0800, Tony Nguyen wrote: > > On 11/23/2025 11:40 PM, Ally Heev wrote: > > Uninitialized pointers with `__free` attribute can cause undefined > > behavior as the memory assigned randomly to the pointer is freed > > automatically when the pointer goes out of scope. > > > > We could just fix it by initializing the pointer to NULL, but, as usage of > > cleanup attributes is discouraged in net [1], trying to achieve cleanup > > using goto > > These two drivers already have multiple other usages of this. All the > other instances initialize to NULL; I'd prefer to see this do the same > over changing this single instance. >
Other usages are slightly complicated to be refactored and might need good testing. Do you want me to do it in a different series? Regards, Ally
