Todd Kjos 551a785c26 binder: fix handling of error during copy ...

[ Upstream commit fe6b186924 ]

If a memory copy function fails to copy the whole buffer,
a positive integar with the remaining bytes is returned.
In binder_translate_fd_array() this can result in an fd being
skipped due to the failed copy, but the loop continues
processing fds since the early return condition expects a
negative integer on error.

Fix by returning "ret > 0 ? -EINVAL : ret" to handle this case.

Fixes: bb4a2e48d5 ("binder: return errors from buffer copy functions")
Suggested-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Christian Brauner <christian.brauner@ubuntu.com>
Signed-off-by: Todd Kjos <tkjos@google.com>
Link: https://lore.kernel.org/r/20211130185152.437403-2-tkjos@google.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>

2022-01-27 10:54:06 +01:00

..

binder_alloc_selftest.c

binder: print warnings when detecting oneway spamming.

2020-09-03 18:24:41 +02:00

binder_alloc.c

binder: fix async_free_space accounting for empty parcels

2022-01-05 12:40:33 +01:00

binder_alloc.h

binder: add flag to clear buffer on txn complete

2020-12-30 11:54:09 +01:00

binder_internal.h

binder: prevent UAF for binderfs devices II

2020-03-03 19:58:37 +01:00

binder_trace.h

…

binder.c

binder: fix handling of error during copy

2022-01-27 10:54:06 +01:00

binderfs.c

binderfs: make symbol 'binderfs_fs_parameters' static

2020-09-03 18:24:39 +02:00

Kconfig

treewide: replace '---help---' in Kconfig files with 'help'

2020-06-14 01:57:21 +09:00

Makefile

…