From: Ian Ray <ian.ray@ge.com>
Date: Thu, 4 Jul 2019 23:13:25 +0000 (-0700)
Subject: Input: atmel_mxt_ts - fix leak in mxt_update_cfg()
X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=f0dd687815f9546860fc3ac4379d55da045942c9;p=linux.git

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>
---

diff --git a/drivers/input/touchscreen/atmel_mxt_ts.c b/drivers/input/touchscreen/atmel_mxt_ts.c
index 5c63d25ce84eb..8b536778f1d8e 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)
 		} else if (config_crc == data->config_crc) {
 			dev_dbg(dev, "Config CRC 0x%06X: OK\n",
 				 data->config_crc);
-			return 0;
+			ret = 0;
+			goto release_raw;
 		} else {
 			dev_info(dev, "Config CRC 0x%06X: does not match file 0x%06X\n",
 				 data->config_crc, config_crc);