Messages in this thread | ![/](/images/icornerl.gif) | | Date | Wed, 22 May 2024 08:57:31 -0400 | From | Sasha Levin <> | Subject | Re: [PATCH AUTOSEL 5.10 5/9] net: qede: sanitize 'rc' in qede_add_tc_flower_fltr() |
| |
On Mon, May 13, 2024 at 09:46:02AM +0000, Asbjørn Sloth Tønnesen wrote: >Hi Pavel and Sasha, > >On 5/13/24 8:18 AM, Pavel Machek wrote: >>>Explicitly set 'rc' (return code), before jumping to the >>>unlock and return path. >>> >>>By not having any code depend on that 'rc' remains at >>>it's initial value of -EINVAL, then we can re-use 'rc' for >>>the return code of function calls in subsequent patches. >>> >>>Only compile tested. >> >>Only compile tested, and is a preparation for something we won't do in >>stable. Does not fix a bug, please drop. > >Please see the original thread about this series[1], this patch is a requirement for >two of the next patches, which does fix a few bugs with overruled error codes returned >to user space. > >I was originally going to ignore these AUTOSEL mails, since the whole series was already >added to the queued more than 24 hours earlier[2]. In the queue Sasha has also added "Stable-dep-of:'. > >So the weird thing is that AUTOSEL selected this patch, given that it was already in the queue.
Two different processes on my end, sorry for the noise!
-- Thanks, Sasha
| ![\](/images/icornerr.gif) |