This is an automated email from Gerrit.

"Daniel Anselmi <danse...@gmx.ch>" just uploaded a new patch set to Gerrit, 
which you can find at https://review.openocd.org/c/openocd/+/7243

-- gerrit

commit 28214cefc2a8a56833f65a93ce8fe43fa9dd4577
Author: Daniel Anselmi <danse...@gmx.ch>
Date:   Sun Oct 2 17:15:41 2022 +0200

    fix leaky file-handle in virtex2 driver
    
    Change-Id: I2784a66c42be71f2982dff7746f9fb2eb1dc8ca6
    Signed-off-by: Daniel Anselmi <danse...@gmx.ch>

diff --git a/src/pld/xilinx_bit.c b/src/pld/xilinx_bit.c
index 7911b8dc14..792b3375b6 100644
--- a/src/pld/xilinx_bit.c
+++ b/src/pld/xilinx_bit.c
@@ -96,31 +96,37 @@ int xilinx_read_bit_file(struct xilinx_bit_file *bit_file, 
const char *filename)
        read_count = fread(bit_file->unknown_header, 1, 13, input_file);
        if (read_count != 13) {
                LOG_ERROR("couldn't read unknown_header from file '%s'", 
filename);
+               fclose(input_file);
                return ERROR_PLD_FILE_LOAD_FAILED;
        }
 
        if (read_section(input_file, 2, 'a', NULL, &bit_file->source_file) != 
ERROR_OK) {
                xilinx_free_bit_file(bit_file);
+               fclose(input_file);
                return ERROR_PLD_FILE_LOAD_FAILED;
        }
 
        if (read_section(input_file, 2, 'b', NULL, &bit_file->part_name) != 
ERROR_OK) {
                xilinx_free_bit_file(bit_file);
+               fclose(input_file);
                return ERROR_PLD_FILE_LOAD_FAILED;
        }
 
        if (read_section(input_file, 2, 'c', NULL, &bit_file->date) != 
ERROR_OK) {
                xilinx_free_bit_file(bit_file);
+               fclose(input_file);
                return ERROR_PLD_FILE_LOAD_FAILED;
        }
 
        if (read_section(input_file, 2, 'd', NULL, &bit_file->time) != 
ERROR_OK) {
                xilinx_free_bit_file(bit_file);
+               fclose(input_file);
                return ERROR_PLD_FILE_LOAD_FAILED;
        }
 
        if (read_section(input_file, 4, 'e', &bit_file->length, 
&bit_file->data) != ERROR_OK) {
                xilinx_free_bit_file(bit_file);
+               fclose(input_file);
                return ERROR_PLD_FILE_LOAD_FAILED;
        }
 

-- 

Reply via email to