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

Janus getting crashed in audiobridge plugin #2432

Closed
ramprakash110109 opened this issue Nov 11, 2020 · 6 comments
Closed

Janus getting crashed in audiobridge plugin #2432

ramprakash110109 opened this issue Nov 11, 2020 · 6 comments

Comments

@ramprakash110109
Copy link

Hi,

During high load in production, I am facing janus crash in past few days. The crash happens in
“Line no. 6297 of janus_audiobridge.c of master branch“ I.e., g_snprintf(s_name, sizeof(s_name), "AudioBridge %s", participant->room->room_id_str);

GDB backtrace of the core dump file is present in https://pastebin.com/D05MWWT3

While in GDB, I printed the value of participant->room and it is NULL.
Is there any check missing ?

@atoppi
Copy link
Member

atoppi commented Nov 11, 2020

The source lines from gdb do not match the master revision.
Can you please provide a gdb trace from a recent version ?

@atoppi
Copy link
Member

atoppi commented Nov 11, 2020

This commit should avoid the crash in future, please test and report if it works.

@ramprakash110109
Copy link
Author

Oops, I have added few comments for my reference in janus_audiobridge.c. That's why the line numbers are different in gdb stack trace. I am using master branch of janus v0.10.4.

Thanks for the commit @atoppi 😊 . I will try this commit and update you.

@lminiero
Copy link
Member

@ramprakash110109 any update? Did you test Alessandro's commit? Does it fix the issue for you?

@ramprakash110109
Copy link
Author

ramprakash110109 commented Nov 17, 2020

Yes @lminiero . @atoppi 's commit solved the issue. Thanks for your support 😊

@lminiero
Copy link
Member

Ack, thanks for the update! Closing the issue then.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants