Hi!
On 2021-05-06 16:32, Atul Gopinathan wrote:
Apart from this, I don't see gdrom_get_last_session() being called anywhere. But I could be missing something obvious too.
If you don't mind, could you point out where gd.toc is being used in probe_gdrom() before it is kzalloc-ed in the same function.
You are very probably correct in your analysis, and I can't find it in me to spend the time to dig any further.
I simply thought it bad enough to hand off a pointer to a function that uses a stale pointer to some other driver. I never dug into that other module like you did. Relying on that other piece of code to not use the function that was just handed to it is way too subtle (for me at least). When you "register" with something else, you should be ready to get the calls.
This is true especially in the context of what we are fixing up here; broken shit related to people that are fond of weaknesses later to be activated by other innocuous commits.
Cheers, Peter