diff options
author | Tomas Vanek <vanekt@fbl.cz> | 2021-11-18 09:37:23 +0100 |
---|---|---|
committer | Antonio Borneo <borneo.antonio@gmail.com> | 2021-12-03 21:59:20 +0000 |
commit | f735faa9319af324b94e4257444419c8dfec6592 (patch) | |
tree | 1d42acd12cc96d37e19b278d17926c6f2e1e98c3 /src/flash/nor/msp432.c | |
parent | 35f284fe7c51d29768156cfec172152d2539d98a (diff) | |
download | riscv-openocd-f735faa9319af324b94e4257444419c8dfec6592.zip riscv-openocd-f735faa9319af324b94e4257444419c8dfec6592.tar.gz riscv-openocd-f735faa9319af324b94e4257444419c8dfec6592.tar.bz2 |
target,flash: allow target_free_working_area on NULL area pointer
Standard C library free() allows NULL pointer as a parameter.
Change target_free_working_area() to conform this convention.
Remove NULL pointer tests before target_free_working_area() calls.
While on it add missing setting pointer to NULL after target_free_working_area().
Change-Id: I7c692ab04a9933398ba5bc614723ad0bdecb87b3
Signed-off-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-on: https://review.openocd.org/c/openocd/+/6712
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Diffstat (limited to 'src/flash/nor/msp432.c')
-rw-r--r-- | src/flash/nor/msp432.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/src/flash/nor/msp432.c b/src/flash/nor/msp432.c index 667349f..61741c8 100644 --- a/src/flash/nor/msp432.c +++ b/src/flash/nor/msp432.c @@ -335,8 +335,9 @@ static int msp432_init(struct flash_bank *bank) } /* Check for working area to use for flash helper algorithm */ - if (msp432_bank->working_area) - target_free_working_area(target, msp432_bank->working_area); + target_free_working_area(target, msp432_bank->working_area); + msp432_bank->working_area = NULL; + retval = target_alloc_working_area(target, ALGO_WORKING_SIZE, &msp432_bank->working_area); if (retval != ERROR_OK) |