diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2022-08-01 13:20:12 +0300 |
|---|---|---|
| committer | Dan Williams <dan.j.williams@intel.com> | 2022-08-01 12:12:33 -0700 |
| commit | c7e3548cac4a8a8bc1ad183cc7c7249463dff55f (patch) | |
| tree | f288dd095c2354807dfbc116363c376d9baff822 /drivers/cxl/core | |
| parent | 88ab1dde792aa6a3902e2a30929e00acc99d351a (diff) | |
| download | lwn-c7e3548cac4a8a8bc1ad183cc7c7249463dff55f.tar.gz lwn-c7e3548cac4a8a8bc1ad183cc7c7249463dff55f.zip | |
cxl/region: prevent underflow in ways_to_cxl()
The "ways" variable comes from the user. The ways_to_cxl() function
has an upper bound but it doesn't check for negatives. Make
the "ways" variable an unsigned int to fix this bug.
Fixes: 80d10a6cee05 ("cxl/region: Add interleave geometry attributes")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Link: https://lore.kernel.org/r/Yueo3NV2hFCXx1iV@kili
[djbw: fixup interleave_ways_store() to only accept unsigned input]
Signed-off-by: Dan Williams <dan.j.williams@intel.com>
Diffstat (limited to 'drivers/cxl/core')
| -rw-r--r-- | drivers/cxl/core/region.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/cxl/core/region.c b/drivers/cxl/core/region.c index c80932bca667..516ba7e2de81 100644 --- a/drivers/cxl/core/region.c +++ b/drivers/cxl/core/region.c @@ -319,10 +319,11 @@ static ssize_t interleave_ways_store(struct device *dev, struct cxl_decoder *cxld = &cxlrd->cxlsd.cxld; struct cxl_region *cxlr = to_cxl_region(dev); struct cxl_region_params *p = &cxlr->params; - int rc, val, save; + unsigned int val, save; + int rc; u8 iw; - rc = kstrtoint(buf, 0, &val); + rc = kstrtouint(buf, 0, &val); if (rc) return rc; |