diff options
author | Andy Walls <awalls@radix.net> | 2009-01-17 11:37:36 -0500 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2009-03-30 11:42:25 -0400 |
commit | be411df610a970a0ab90e3b02d025ab2a3554b61 (patch) | |
tree | a55d7c3b85f2ebb971a4ea71904427179a2e084e /drivers/media/video/cx18 | |
parent | 888cdb07741ab0098ccb8d9feff3f98cad048c26 (diff) |
V4L/DVB (10283): cx18: Call request_module() with proper argument types.
request_module() needs to be called with a string literal for a
format string or with 1 or more variable arguments to avoid
compiler warnings and possible exploits, if someone could cause
us to get a format string with a '%' code in the format string
when we make the call.
Reported-by: Brandon Jenkins <bcjenkins@tvwhere.com>
Signed-off-by: Andy Walls <awalls@radix.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video/cx18')
-rw-r--r-- | drivers/media/video/cx18/cx18-driver.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/video/cx18/cx18-driver.c b/drivers/media/video/cx18/cx18-driver.c index 450c48dd995a..1d3a865b9c6d 100644 --- a/drivers/media/video/cx18/cx18-driver.c +++ b/drivers/media/video/cx18/cx18-driver.c | |||
@@ -728,7 +728,7 @@ static u32 cx18_request_module(struct cx18 *cx, u32 hw, | |||
728 | { | 728 | { |
729 | if ((hw & id) == 0) | 729 | if ((hw & id) == 0) |
730 | return hw; | 730 | return hw; |
731 | if (request_module(name) != 0) { | 731 | if (request_module("%s", name) != 0) { |
732 | CX18_ERR("Failed to load module %s\n", name); | 732 | CX18_ERR("Failed to load module %s\n", name); |
733 | return hw & ~id; | 733 | return hw & ~id; |
734 | } | 734 | } |