mirror of
https://github.com/hardkernel/linux.git
synced 2026-03-25 20:10:23 +09:00
media: vivid: fix assignment of dev->fbuf_out_flags
[ Upstream commit5cde22fcc7] Currently the chroma_flags and alpha_flags are being zero'd with a bit-wise mask and the following statement should be bit-wise or'ing in the new flag bits but instead is making a direct assignment. Fix this by using the |= operator rather than an assignment. Addresses-Coverity: ("Unused value") Fixes:ef834f7836("[media] vivid: add the video capture and output parts") Signed-off-by: Colin Ian King <colin.king@canonical.com> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
7d81aff289
commit
385470a358
@@ -1025,7 +1025,7 @@ int vivid_vid_out_s_fbuf(struct file *file, void *fh,
|
||||
return -EINVAL;
|
||||
}
|
||||
dev->fbuf_out_flags &= ~(chroma_flags | alpha_flags);
|
||||
dev->fbuf_out_flags = a->flags & (chroma_flags | alpha_flags);
|
||||
dev->fbuf_out_flags |= a->flags & (chroma_flags | alpha_flags);
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user