Commit 0573a1e2 authored by Bas Nieuwenhuizen's avatar Bas Nieuwenhuizen Committed by Alex Deucher
Browse files

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



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 c6b86421)
Cc: stable@vger.kernel.org
parent e6c6bd62

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

+8 −0
Original line number Diff line number Diff line
@@ -685,16 +685,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;

	case AMDGPU_CTX_OP_GET_STABLE_PSTATE: