+
Skip to content

Sign of a bad copy/paste in logging of usserSessionLimitsAuthenticator #40060

Closed
@LM4O322

Description

@LM4O322

Description

Hi, guys!
I found some minor bugs in the code while running SAST. There are some signs of bad copy/paste.
In particular, the code was changed in Commit 2c238b9 and this part of code is still in the main branch.

logger.infof("Too many sessions related to the current client for this user. Session count: %s", userSessionCountForRealm);

It looks like userSessionCountForClient should be used instead of userSessionCountForRealm.
This code is in a block that compares userSessionCountForClient to determine if userClientLimit has been exceeded, so it would make sense to output the value of userSessionCountForClient to the logs.

Discussion

No response

Motivation

No response

Details

No response


Found by Linux Verification Center with SVACE

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载