Comment 23 for bug 1489478

Revision history for this message
Paweł Stołowski (stolowski) wrote :

Michi is right about the thread safety problem in the forwarder code. The original old buffered result forwarder (in mediascanner scope) had this issue and afair since we didn't have that code in the scopes library at that time, it was copied to some other aggregator scopes (namely Today scope). We now have BufferedResultForwarder in scopes API, but it uses somewhat different semantics than the original implementation. For a quickfix I think all you need to do is make sure that access to 'ready' state flag is synchronized.