Skip to content

Commit efe759d

Browse files
Dan Carpentergregkh
Dan Carpenter
authored andcommitted
drm/msm/gem: Fix error code msm_parse_deps()
[ Upstream commit 0b305b7 ] The SUBMIT_ERROR() macro turns the error code negative. This extra '-' operation turns it back to positive EINVAL again. The error code is passed to ERR_PTR() and since positive values are not an IS_ERR() it eventually will lead to an oops. Delete the '-'. Fixes: 866e43b ("drm/msm: UAPI error reporting") Signed-off-by: Dan Carpenter <[email protected]> Patchwork: https://patchwork.freedesktop.org/patch/637625/ Signed-off-by: Rob Clark <[email protected]> Signed-off-by: Sasha Levin <[email protected]>
1 parent 98adf63 commit efe759d

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

drivers/gpu/drm/msm/msm_gem_submit.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -509,7 +509,7 @@ static struct drm_syncobj **msm_parse_deps(struct msm_gem_submit *submit,
509509
}
510510

511511
if (syncobj_desc.flags & ~MSM_SUBMIT_SYNCOBJ_FLAGS) {
512-
ret = -SUBMIT_ERROR(EINVAL, submit, "invalid syncobj flags: %x", syncobj_desc.flags);
512+
ret = SUBMIT_ERROR(EINVAL, submit, "invalid syncobj flags: %x", syncobj_desc.flags);
513513
break;
514514
}
515515

0 commit comments

Comments
 (0)