diff options
| author | John Garry <[email protected]> | 2018-11-05 12:35:15 +0000 |
|---|---|---|
| committer | Herbert Xu <[email protected]> | 2018-11-09 09:35:43 +0000 |
| commit | 0b0cf6af3f3151c26c27e8e51def5527091c3e69 (patch) | |
| tree | bed8cd167637cda4537c79cae21371a4966c9aed /tools/perf/scripts/python/check-perf-trace.py | |
| parent | crypto: hisilicon - Fix NULL dereference for same dst and src (diff) | |
| download | kernel-0b0cf6af3f3151c26c27e8e51def5527091c3e69.tar.gz kernel-0b0cf6af3f3151c26c27e8e51def5527091c3e69.zip | |
crypto: hisilicon - Fix reference after free of memories on error path
coccicheck currently warns of the following issues in the driver:
drivers/crypto/hisilicon/sec/sec_algs.c:864:51-66: ERROR: reference preceded by free on line 812
drivers/crypto/hisilicon/sec/sec_algs.c:864:40-49: ERROR: reference preceded by free on line 813
drivers/crypto/hisilicon/sec/sec_algs.c:861:8-24: ERROR: reference preceded by free on line 814
drivers/crypto/hisilicon/sec/sec_algs.c:860:41-51: ERROR: reference preceded by free on line 815
drivers/crypto/hisilicon/sec/sec_algs.c:867:7-18: ERROR: reference preceded by free on line 816
It would appear than on certain error paths that we may attempt reference-
after-free some memories.
This patch fixes those issues. The solution doesn't look perfect, but
having same memories free'd possibly from separate functions makes it
tricky.
Fixes: 915e4e8413da ("crypto: hisilicon - SEC security accelerator driver")
Reviewed-by: Jonathan Cameron <[email protected]>
Cc: <[email protected]>
Signed-off-by: John Garry <[email protected]>
Signed-off-by: Herbert Xu <[email protected]>
Diffstat (limited to 'tools/perf/scripts/python/check-perf-trace.py')
0 files changed, 0 insertions, 0 deletions