Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit c5e2c86a authored by Bas Nieuwenhuizen's avatar Bas Nieuwenhuizen Committed by Greg Kroah-Hartman
Browse files

drm/amdgpu: Actually check flags for all context ops. 



commit 0573a1e2ea7e35bff08944a40f1adf2bb35cea61 upstream.

Missing validation ...

Checked libdrm and it clears all the structs, so we should be
safe to just check everything.

Signed-off-by: default avatarBas Nieuwenhuizen <bas@basnieuwenhuizen.nl>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
(cherry picked from commit c6b86421f1f9ddf9d706f2453159813ee39d0cf9)
Cc: stable@vger.kernel.org
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 5d93f05e

drivers/gpu/drm/amd/amdgpu/amdgpu_ctx.c

+8 −0
Original line number Diff line number Diff line
@@ -288,16 +288,24 @@ int amdgpu_ctx_ioctl(struct drm_device *dev, void *data, 


	switch (args->in.op) {

	case AMDGPU_CTX_OP_ALLOC_CTX:

		if (args->in.flags)

			return -EINVAL;

		r = amdgpu_ctx_alloc(adev, fpriv, filp, priority, &id);

		args->out.alloc.ctx_id = id;

		break;

	case AMDGPU_CTX_OP_FREE_CTX:

		if (args->in.flags)

			return -EINVAL;

		r = amdgpu_ctx_free(fpriv, id);

		break;

	case AMDGPU_CTX_OP_QUERY_STATE:

		if (args->in.flags)

			return -EINVAL;

		r = amdgpu_ctx_query(adev, fpriv, id, &args->out);

		break;

	case AMDGPU_CTX_OP_QUERY_STATE2:

		if (args->in.flags)

			return -EINVAL;

		r = amdgpu_ctx_query2(adev, fpriv, id, &args->out);

		break;

	default: