On 2019/1/15 下午11:07, Petr Vorel wrote:
> Hi Jia, Mimi,
>
> ...
>> +reset_printk_ratelimit()
>> +{
>> + [ "$PRINTK_RATE_LIMIT" != "0" ] && \
>> + sysctl -wq kernel.printk_ratelimit=$PRINTK_RATE_LIMIT
>> +}
> ...
>> @@ -151,6 +163,8 @@ test3()
>
>> validate $num_violations $count $search
>
>> + reset_printk_ratelimit
> This should be called in cleanup function.
> Actually as it's called only once I'd put it into cleanup function.
I recognized that ima_setup.sh already defines a cleanup() so I cannot
define another cleanup function in ima_violations.sh. Do you mean moving
reset_printk_ratelimit into cleanup() defined in ima_setup.sh?
Thanks,
Jia
> No need to resent the patchset, if it's the only change, I change it before
> merge.
>
> @Mimi: do you agree with this change?
> I've seen some issues on some distros when not using auditd daemon,
> but now cannot reproduce them any more.
>
> Kind regards,
> Petr
>
