diff options
author | Andy Shevchenko <andriy.shevchenko@linux.intel.com> | 2022-02-23 17:47:18 +0200 |
---|---|---|
committer | Miguel Ojeda <ojeda@kernel.org> | 2022-03-03 00:30:31 +0100 |
commit | 9ed331f8a0fb674f4f06edf05a1687bf755af27b (patch) | |
tree | 58387ee9ec6cf300bcac857aa2820ebd827ade8a /net/psample | |
parent | 898c0a15425a5bcaa8d44bd436eae5afd2483796 (diff) | |
download | linux-9ed331f8a0fb674f4f06edf05a1687bf755af27b.tar.bz2 |
auxdisplay: lcd2s: Use proper API to free the instance of charlcd object
While it might work, the current approach is fragile in a few ways:
- whenever members in the structure are shuffled, the pointer will be wrong
- the resource freeing may include more than covered by kfree()
Fix this by using charlcd_free() call instead of kfree().
Fixes: 8c9108d014c5 ("auxdisplay: add a driver for lcd2s character display")
Cc: Lars Poeschel <poeschel@lemonage.de>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
Diffstat (limited to 'net/psample')
0 files changed, 0 insertions, 0 deletions