diff options
author | Alan Cox <alan@linux.intel.com> | 2012-07-30 17:41:30 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-07-30 20:25:17 -0400 |
commit | 261eba73353edd48b0c0cb7aad59553dfc712ebc (patch) | |
tree | 0bc64028f987e7777ed5002f0b4debf4422c28ad | |
parent | 40251b8eb46e48c011939a3ddf056fe13a223319 (diff) |
drivers/message/i2o/i2o_config.c: bound allocation
Fix a case where users can try to allocate arbitarily large amounts of
memory. 64K is overkill for a config request so apply an upper bound.
Signed-off-by: Alan Cox <alan@linux.intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | drivers/message/i2o/i2o_config.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/drivers/message/i2o/i2o_config.c b/drivers/message/i2o/i2o_config.c index 098de2b35784..9a49c243a6ac 100644 --- a/drivers/message/i2o/i2o_config.c +++ b/drivers/message/i2o/i2o_config.c | |||
@@ -188,6 +188,13 @@ static int i2o_cfg_parms(unsigned long arg, unsigned int type) | |||
188 | if (!dev) | 188 | if (!dev) |
189 | return -ENXIO; | 189 | return -ENXIO; |
190 | 190 | ||
191 | /* | ||
192 | * Stop users being able to try and allocate arbitary amounts | ||
193 | * of DMA space. 64K is way more than sufficient for this. | ||
194 | */ | ||
195 | if (kcmd.oplen > 65536) | ||
196 | return -EMSGSIZE; | ||
197 | |||
191 | ops = memdup_user(kcmd.opbuf, kcmd.oplen); | 198 | ops = memdup_user(kcmd.opbuf, kcmd.oplen); |
192 | if (IS_ERR(ops)) | 199 | if (IS_ERR(ops)) |
193 | return PTR_ERR(ops); | 200 | return PTR_ERR(ops); |