Bug 33292

Summary: bug in setting up call state at startup
Product: Telepathy Reporter: Kai Vehmanen <kai.vehmanen>
Component: ringAssignee: Pekka Pessi <pekka.pessi>
Status: NEW --- QA Contact: Telepathy bugs list <telepathy-bugs>
Severity: normal    
Priority: medium    
Version: git master   
Hardware: All   
OS: All   
Whiteboard:
i915 platform: i915 features:

Description Kai Vehmanen 2011-01-20 05:35:12 UTC
STEPS:
1. do not start tpring
2. make a call to number X with ofono
3. start tpring
4. tpring detects the ongoing calls and creates channels for them
5. try to create a new call to X

EXPECTED OUTCOME:
a new, separate channel is established to X (and the original call is put on hold)

ACTUAL OUTCOME:
assert hit:
--cut--
ERROR:call.c:648:modem_call_set_handler: assertion failed: (success)
--cut--
Comment 1 Pekka Pessi 2011-01-20 05:50:02 UTC
oFono version? It should work in 0.37 or never, earlier versions fail.
Comment 2 Kai Vehmanen 2011-01-20 06:03:27 UTC
(In reply to comment #1)
> oFono version? It should work in 0.37 or never, earlier versions fail.

For ofono, git master of today (754be710d53178088e2668902cd3bd3711691d7b). The channels for existing calls are created succesfully, but something goes wrong afterwords.

I can take a look at this later, but I'm currently looking at some many things already, I wanted to file this just so that I don't forget.
Comment 3 Pekka Pessi 2011-01-21 08:16:37 UTC
The assert gets hit if the object returned from Dial is already associated with a channel.

oFono may emit a CallAdded signal before the Dial method returns.

There is no straightforward way to associated the Dial method call and the subsequent CallAdded signal. It may result from Dial, or then not.
Comment 4 Pekka Pessi 2011-02-03 11:01:52 UTC
This particular situation seems to be a bug in oFono.

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.