Re: [PATCH] drm/scheduler: set entity to NULL in drm_sched_entity_pop_job()

2023-04-11 Thread Luben Tuikov
On 2023-04-11 14:13, Danilo Krummrich wrote: > On 4/5/23 19:39, Luben Tuikov wrote: >> On 2023-03-31 01:59, Christian König wrote: >>> Am 31.03.23 um 02:06 schrieb Danilo Krummrich: It already happend a few times that patches slipped through which implemented access to an entity through

Re: [PATCH] drm/scheduler: set entity to NULL in drm_sched_entity_pop_job()

2023-04-11 Thread Danilo Krummrich
On 4/5/23 19:39, Luben Tuikov wrote: On 2023-03-31 01:59, Christian König wrote: Am 31.03.23 um 02:06 schrieb Danilo Krummrich: It already happend a few times that patches slipped through which implemented access to an entity through a job that was already removed from the entities queue.

Re: [PATCH] drm/scheduler: set entity to NULL in drm_sched_entity_pop_job()

2023-04-05 Thread Luben Tuikov
On 2023-03-31 01:59, Christian König wrote: > Am 31.03.23 um 02:06 schrieb Danilo Krummrich: >> It already happend a few times that patches slipped through which >> implemented access to an entity through a job that was already removed >> from the entities queue. Since jobs and entities might have

Re: [PATCH] drm/scheduler: set entity to NULL in drm_sched_entity_pop_job()

2023-03-31 Thread Luben Tuikov
On 2023-03-31 01:59, Christian König wrote: > Am 31.03.23 um 02:06 schrieb Danilo Krummrich: >> It already happend a few times that patches slipped through which >> implemented access to an entity through a job that was already removed >> from the entities queue. Since jobs and entities might have

Re: [PATCH] drm/scheduler: set entity to NULL in drm_sched_entity_pop_job()

2023-03-30 Thread Christian König
Am 31.03.23 um 02:06 schrieb Danilo Krummrich: It already happend a few times that patches slipped through which implemented access to an entity through a job that was already removed from the entities queue. Since jobs and entities might have different lifecycles, this can potentially cause UAF

[PATCH] drm/scheduler: set entity to NULL in drm_sched_entity_pop_job()

2023-03-30 Thread Danilo Krummrich
It already happend a few times that patches slipped through which implemented access to an entity through a job that was already removed from the entities queue. Since jobs and entities might have different lifecycles, this can potentially cause UAF bugs. In order to make it obvious that a jobs