On Mon, 2019-12-16 at 15:44 -0800, Lakshmi Ramasubramanian wrote:
> On 12/16/2019 4:30 AM, Mimi Zohar wrote:
>
> >> +
> >> + if (!list_empty(&ima_keys)) {
> >> + list_for_each_entry_safe(entry, tmp, &ima_keys, list)
> >> + list_move_tail(&entry->list, &temp_ima_keys);
> >> + process = true;
> >> + }
> >> + }
> >> +
> >> + mutex_unlock(&ima_keys_mutex);
> >> +
> >> + if (!process)
> >> + return;
> >
> > The new changes - checking if the list is empty and this test - are
> > unnecessary, as you implied earlier.
> >
> > Mimi
>
> Do you want me to remove this check? I feel it is safer to have this
> check - use a local flag "process" to return if no keys were moved to
> the temp list. Would like to leave it as is - if you don't mind.
Sure
