> The change is good.
> The commit message would better simply describe the problem instead
> of describing the symptom. For example, something like:
>       The function _copy_to_user() is used to copy to address space.
>       As such, the destination pointer should be annotated with
>       '__user'.
>       However, the function has the annotation wrongly, on the
>       source instead of the destination (copy & paste error?).
>       Fix this by moving the __user annotation to the correct
>       argument.

FWIW, I've committed it with 

Fix misannotated out-of-line _copy_to_user()

Destination is a kernel pointer and source - a userland one
is _copy_from_user(); _copy_to_user() is the other way round.

Reply via email to