diff options
| author | Dan Carpenter <error27@gmail.com> | 2010-10-21 01:46:58 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2010-11-09 16:32:36 -0500 |
| commit | eccbf04a904fc99c54ab37c29a2a4dedcec66e33 (patch) | |
| tree | fe6d1142f644afe0984bdd3570b89ede21bc1540 /net/unix/sysctl_net_unix.c | |
| parent | 4fc718a4b0cdf3803f370e323ea5252a3d76f52d (diff) | |
Staging: bcm: use get_user() to access user pointers
This fixes some places that dereference user pointers directly instead
of using get_user().
Please especially check my changes to IOCTL_BCM_GET_CURRENT_STATUS. The
original code modified the struct which "arg" was pointing to. I think
this was a bug in the original code and that we only wanted to write to
the OutputBuffer. Also with the original code you could read as much
memory as you wanted so I had to put a cap on OutputLength. The only
value of OutputLength that makes sense is sizeof(LINK_STATE) so now if
OutputLength is not sizeof(LINK_STATE) it returns -EINVAL.
Signed-off-by: Dan Carpenter <error27@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'net/unix/sysctl_net_unix.c')
0 files changed, 0 insertions, 0 deletions