On Fri Nov 18 00:25:03 2022 +0000, Zebediah Figura wrote:
This can still access an invalid presentation descriptor if the media source is concurrently shut down. There are other, similar cases, e.g. in Start() and other state change methods, we can access an invalid command queue. In a sense this patch is really just protecting concurrent access to the "state" field, but I don't think that makes much sense by itself, especially given the nature of the problem we're trying to solve here.
Yeah, you're right. I don't know how I convinced myself that that would be enough. Will fix.