(In reply to Gabriele Svelto [:gsvelto] from comment #2) > I did some further digging and this is happening all over the place but > mostly in code we don't control. It's always `select()`'s fault though. > Daniel, this code is different from upstream WebRTC (which is also using > `select()` but in other places), is there a reason for this? I can cook up a > patch to switch this code to `poll()` instead but I wonder if it's the right > thing to do or if this should be fixed upstream first.
Hi Gabriele, this code is our local modification to webrtc.org that we have never merged upstream. As such, feel free to rewrite it to use poll() if that will fix things. I'm not the original author, I just happened to do the last upstream merge, so I can't comment on why select was chosen rather than poll. :jesup might know if you want to follow up with him. One thing to be careful of is that this code is handling cameras being plugged and unplugged which is not something we have coverage for in automation, so you'll have to do some manual testing of your changes. The STR from https://bugzilla.mozilla.org/show_bug.cgi?id=1581193#c0 might be helpful here. -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1849615 Title: WebRTC-related crashes To manage notifications about this bug go to: https://bugs.launchpad.net/firefox/+bug/1849615/+subscriptions -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
