diff options
| author | Ian Ray <ian.ray@ge.com> | 2019-07-04 19:13:25 -0400 |
|---|---|---|
| committer | Dmitry Torokhov <dmitry.torokhov@gmail.com> | 2019-07-05 13:17:07 -0400 |
| commit | f0dd687815f9546860fc3ac4379d55da045942c9 (patch) | |
| tree | 24b36f2d7ac16bbf267e7fac9ecb8c870826de27 | |
| parent | abbe3acd7d72ab4633ade6bd24e8306b67e0add3 (diff) | |
Input: atmel_mxt_ts - fix leak in mxt_update_cfg()
Fix leak (whose magnitude is the configuration file size) when the CRCs
match in mxt_update_cfg().
Signed-off-by: Ian Ray <ian.ray@ge.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
| -rw-r--r-- | drivers/input/touchscreen/atmel_mxt_ts.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c b/drivers/input/touchscreen/atmel_mxt_ts.c index 5c63d25ce84e..8b536778f1d8 100644 --- a/drivers/input/touchscreen/atmel_mxt_ts.c +++ b/drivers/input/touchscreen/atmel_mxt_ts.c | |||
| @@ -1526,7 +1526,8 @@ static int mxt_update_cfg(struct mxt_data *data, const struct firmware *fw) | |||
| 1526 | } else if (config_crc == data->config_crc) { | 1526 | } else if (config_crc == data->config_crc) { |
| 1527 | dev_dbg(dev, "Config CRC 0x%06X: OK\n", | 1527 | dev_dbg(dev, "Config CRC 0x%06X: OK\n", |
| 1528 | data->config_crc); | 1528 | data->config_crc); |
| 1529 | return 0; | 1529 | ret = 0; |
| 1530 | goto release_raw; | ||
| 1530 | } else { | 1531 | } else { |
| 1531 | dev_info(dev, "Config CRC 0x%06X: does not match file 0x%06X\n", | 1532 | dev_info(dev, "Config CRC 0x%06X: does not match file 0x%06X\n", |
| 1532 | data->config_crc, config_crc); | 1533 | data->config_crc, config_crc); |
