diff options
author | Kees Cook <keescook@chromium.org> | 2014-10-20 18:49:04 -0300 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-10-21 08:49:11 -0200 |
commit | 7383159f413b8b07fe689a25ee55915f86cf2f36 (patch) | |
tree | 8ca26f868966a062a2c3766b884eda8e67d8a7b4 /drivers/media/usb/dvb-usb-v2/anysee.c | |
parent | a1ecf3c4560ef8260f73ffb85b425f3a046cd419 (diff) | |
download | op-kernel-dev-7383159f413b8b07fe689a25ee55915f86cf2f36.zip op-kernel-dev-7383159f413b8b07fe689a25ee55915f86cf2f36.tar.gz |
[media] anysee: make sure loading modules is const
Make sure that loaded modules are const char strings so we don't
load arbitrary modules in the future, nor allow for format string
leaks in the module request call.
Signed-off-by: Kees Cook <keescook@chromium.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/usb/dvb-usb-v2/anysee.c')
-rw-r--r-- | drivers/media/usb/dvb-usb-v2/anysee.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/drivers/media/usb/dvb-usb-v2/anysee.c b/drivers/media/usb/dvb-usb-v2/anysee.c index d3c5f23..ae917c0 100644 --- a/drivers/media/usb/dvb-usb-v2/anysee.c +++ b/drivers/media/usb/dvb-usb-v2/anysee.c @@ -630,8 +630,8 @@ error: return ret; } -static int anysee_add_i2c_dev(struct dvb_usb_device *d, char *type, u8 addr, - void *platform_data) +static int anysee_add_i2c_dev(struct dvb_usb_device *d, const char *type, + u8 addr, void *platform_data) { int ret, num; struct anysee_state *state = d_to_priv(d); @@ -659,7 +659,7 @@ static int anysee_add_i2c_dev(struct dvb_usb_device *d, char *type, u8 addr, goto err; } - request_module(board_info.type); + request_module("%s", board_info.type); /* register I2C device */ client = i2c_new_device(adapter, &board_info); |