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

Crash for unknown reasons #2300

Closed
648540858 opened this issue Jul 27, 2020 · 12 comments
Closed

Crash for unknown reasons #2300

648540858 opened this issue Jul 27, 2020 · 12 comments

Comments

@648540858
Copy link

this is the log;
https://gist.github.com/648540858/44135642522fe4fdb6f15865581c4c9a

I added the field displayName. The modified file is in the log。I'm not sure that's what caused it。

@lminiero
Copy link
Member

Clearly not master, as line numbers don't match. As clearly explained in the guidelines, you have to try master first, or we won't look at the issue.

@648540858
Copy link
Author

@lminiero
Copy link
Member

Looks like a race condition where both sides are closed more or less at the same time: this commit a6731ba should help address it. Closing for the time being, let us know if it's still not fixed.

@648540858
Copy link
Author

@lminiero
Copy link
Member

I wouldn't say "the problem remains", since it happens in a completely different place, which means the original issue has been solved. Busy now, I'll try to find time later or in the next few days.

@648540858
Copy link
Author

OK, I'm careless。

@lminiero lminiero reopened this Jul 28, 2020
@lminiero
Copy link
Member

Mh, looks like it's a similar issue: the peer is freed while we're using its structure. I guess we're missing sessions referencing each other when they're in a call to each other (or trying to establish a call), which might help prevent this. I'll try to work on a fix and let you know when there's a commit to try.

@lminiero
Copy link
Member

Please check the above commit, to see if it fixes it for you.

@648540858
Copy link
Author

The crash took place further forward,The content is relatively small, I posted it directly

(gdb) bt
#0  0x00007ff05ffb33eb in __GI_raise (sig=sig@entry=6) at ../sysdeps/unix/sysv/linux/raise.c:50
#1  0x00007ff05ff92899 in __GI_abort () at abort.c:79
#2  0x00007ff0607332fd in  () at /lib/x86_64-linux-gnu/libglib-2.0.so.0
#3  0x00007ff0500f6c00 in janus_videocall_handler (data=<optimized out>) at plugins/janus_videocall.c:1285
#4  0x00007ff060711181 in  () at /lib/x86_64-linux-gnu/libglib-2.0.so.0
#5  0x00007ff060167669 in start_thread (arg=<optimized out>) at pthread_create.c:479
#6  0x00007ff06008f2b3 in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

@lminiero
Copy link
Member

My fault, there was a copy/paste typo: ef954f9 should fix it.

@lminiero
Copy link
Member

@648540858 any update? Can this be closed?

@648540858
Copy link
Author

I've been testing it for a day and it hasn't happened again. It's great

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

2 participants