diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2011-09-23 06:23:22 (GMT) |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2011-09-27 00:43:44 (GMT) |
| commit | de0920b1fedb4d694fe4732cfc3cccfcd4743f84 (patch) | |
| tree | 599e30f3d96b9855c3474d033a144cd6e3576acb /drivers/staging/vt6656 | |
| parent | 43664e14a029ab458cba8af43c65853b079b282f (diff) | |
| download | linux-de0920b1fedb4d694fe4732cfc3cccfcd4743f84.tar.xz | |
Staging: vt6655-6: potential info leak in private_ioctl()
Smatch has a new check for Rosenberg type information leaks where
structs are copied to the user with uninitialized stack data in them.
In this path, the .uLinkRate member doesn't get initialized so I've
set it to zero.
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/staging/vt6656')
| -rw-r--r-- | drivers/staging/vt6656/ioctl.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/staging/vt6656/ioctl.c b/drivers/staging/vt6656/ioctl.c index cfe9c95..1cb9a7d 100644 --- a/drivers/staging/vt6656/ioctl.c +++ b/drivers/staging/vt6656/ioctl.c @@ -291,6 +291,7 @@ int private_ioctl(PSDevice pDevice, struct ifreq *rq) { } else { sLinkStatus.bLink = FALSE; + sLinkStatus.uLinkRate = 0; } if (copy_to_user(pReq->data, &sLinkStatus, sizeof(SCmdLinkStatus))) { result = -EFAULT; |