On Thu, Jan 18, 2018 at 1:20 PM, walter harms <wharms@xxxxxx> wrote:
> Am 18.01.2018 12:03, schrieb Dan Carpenter:
>> On Thu, Jan 18, 2018 at 11:50:30AM +0100, walter harms wrote:
>>>> out_unwind_strings:
>>>> - for (i = i-1; i > 0; i--) {
>>>> + for (i = i-1; i >= 0; i--) {
>>>> kfree(token_location_attrs[i].attr.name);
>>>> kfree(token_value_attrs[i].attr.name);
>>>> }
>>> would you mind to reverse order here ?
>>> you know programmers are terrible at couting backwards.
>>
>> I prefer to always unwind in reverse order so I'd prefer to leave it
>> as-is.
> It is just a comment from my side. we can leave the actual decision to the
> current maintainer.
Right.
And maintainer would like to see simple:
while (i--) {
kfree(token_location_attrs[i].attr.name);
kfree(token_value_attrs[i].attr.name);
}
--
With Best Regards,
Andy Shevchenko
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
