Re: [PATCH -next] evm: Support small xattr in dump_security_xattr()

From: Mimi Zohar
Date: Wed Jan 18 2023 - 15:24:11 EST


Hi Xiu,

Thank you for cleaning up the code. The purpose of this patch is to
remove code duplication. Perhaps rename the patch to "evm: call
dump_security_xattr() in all cases to remove code duplication".

On Mon, 2022-12-26 at 18:24 +0800, Xiu Jianfeng wrote:
> Currently the debug function of dumping xattr is splited into two parts,
> when the length of xattr is less than 64 bytes, it uses pr_debug()
> directly. Merge it into dump_security_xattr() to simplify the code, no
> functional changes here.
>
> Signed-off-by: Xiu Jianfeng <xiujianfeng@xxxxxxxxxx>

Please reword the patch description in terms of removing code
duplication.

--
thanks,

Mimi