|Summary:||Crashes if you poke Hold or CallStates before RequestStreams|
|Product:||Telepathy||Reporter:||Will Thompson <will>|
|Component:||gabble||Assignee:||Telepathy bugs list <telepathy-bugs>|
|Status:||RESOLVED FIXED||QA Contact:||Telepathy bugs list <telepathy-bugs>|
|i915 platform:||i915 features:|
|Bug Depends on:|
Description Will Thompson 2009-05-29 10:10:08 UTC
The Hold and CallStates code assumes that a media channel has a JingleSession. This is not true before you actually initiate the call. It's not immediately clear to me what should happen if you RequestHold() on an uninitiated call. Should it fail? Or should it succeed, and send <hold/> as soon as you initiate? Regardless, calling GetCallStates and RequestHold does crash Gabble; I'd like to fix this before releasing 0.7.28, so setting this bug to blocker.
Comment 1 Will Thompson 2009-06-01 10:21:06 UTC
The GetCallState() crash has been fixed, and will be in 0.7.28. Downgrading the bug to critical: Hold-before-initiate crashing is not a new issue, and isn't actively causing problems, but should still be fixed ASAP.
Comment 2 Will Thompson 2009-06-02 07:18:31 UTC
My branch hold-before-call-starts addresses the remaining issue, as well as addressing #22031 (by forbidding answering calls you've put on hold).
Comment 3 Guillaume Desmottes 2009-06-10 03:03:58 UTC
Looks good. Very clear patches, nice done! Don't forget to upgrade NEWS.
Comment 4 Will Thompson 2009-06-10 03:47:48 UTC
Merged; will be in 0.7.29. Thanks!