Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Multiple message invoked #6664

Open
Neloy262 opened this issue Dec 11, 2024 · 2 comments
Open

Multiple message invoked #6664

Neloy262 opened this issue Dec 11, 2024 · 2 comments
Labels
bug A bug (error) in the software needs-more-input stale-no-response This issue hasn't seen any activity in recent time and will probably be closed soon. triage This issue is waiting to be triaged by one of the project members

Comments

@Neloy262
Copy link

Description

In the mumble server two users with the same name can be connected at once. In this case we are seeing that the user that joined last is getting sent the same proto msg twice from the server. If three users with the same name are connected then the 3rd user is getting the proto msg thrice and so on.

Steps to reproduce

  1. Have multiple users with the same name join the server.

Mumble version

1.5.0

Mumble component

Both

OS

Linux

Reproducible?

Yes

Additional information

No response

Relevant log output

No response

Screenshots

No response

@Neloy262 Neloy262 added bug A bug (error) in the software triage This issue is waiting to be triaged by one of the project members labels Dec 11, 2024
@Krzmbrzl
Copy link
Member

Krzmbrzl commented Dec 11, 2024

User's shouldn't be able to use the same username

ServerUser *uOld = nullptr;
foreach (ServerUser *u, qhUsers) {
if (u == uSource)
continue;
if (((u->iId >= 0) && (u->iId == uSource->iId)) || (u->qsName.toLower() == uSource->qsName.toLower())) {
uOld = u;
break;
}
}
// Allow reuse of name from same IP
if (ok && uOld && (uSource->iId == -1)) {
if ((uOld->peerAddress() != uSource->peerAddress())
&& (uSource->qsHash.isEmpty() || (uSource->qsHash != uOld->qsHash))) {
reason = "Username already in use";
rtType = MumbleProto::Reject_RejectType_UsernameInUse;
ok = false;
}
}

Could you give specifics on your exact server setup and what usernames you are using (and whether a user with that username is registered on that server)?

Copy link

This issue has been marked as stale, because our request for more information has thus far not been fulfilled.

If no further action occurs, this issue will be closed within 7 days.

@github-actions github-actions bot added the stale-no-response This issue hasn't seen any activity in recent time and will probably be closed soon. label Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug A bug (error) in the software needs-more-input stale-no-response This issue hasn't seen any activity in recent time and will probably be closed soon. triage This issue is waiting to be triaged by one of the project members
Projects
None yet
Development

No branches or pull requests

2 participants