[RFC PATCH 1/3] lsm: cleanup the size counters in security_getselfattr()
Casey Schaufler
casey at schaufler-ca.com
Tue Oct 24 22:23:07 UTC 2023
On 10/24/2023 2:35 PM, Paul Moore wrote:
> Zero out all of the size counters in the -E2BIG case (buffer too
> small) to help make the current code a bit more robust in the face of
> future code changes.
I don't see how this change would have the described effect.
What it looks like it would do is change the return from -E2BIG
to 0, which would not have the desired result.
>
> Signed-off-by: Paul Moore <paul at paul-moore.com>
> ---
> security/security.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/security/security.c b/security/security.c
> index 988483fcf153..9c63acded4ee 100644
> --- a/security/security.c
> +++ b/security/security.c
> @@ -3951,8 +3951,9 @@ int security_getselfattr(unsigned int attr, struct lsm_ctx __user *uctx,
> continue;
> }
> if (rc == -E2BIG) {
> - toobig = true;
> + rc = 0;
> left = 0;
> + toobig = true;
> } else if (rc < 0)
> return rc;
> else
More information about the Linux-security-module-archive
mailing list