Browse Source

flashparams: Fix a null-pointer dereference crash

Fix a potential crash caused by calling erase_sector with a null
sector_descriptor (current_sector == 0).

Signed-off-by: Jukka Laitinen <jukkax@ssrc.tii.ae>
release/1.12
Jukka Laitinen 4 years ago committed by David Sidrane
parent
commit
b99e597a73
  1. 4
      src/lib/parameters/flashparams/flashfs.c

4
src/lib/parameters/flashparams/flashfs.c

@ -875,11 +875,13 @@ parameter_flashfs_write(flash_file_token_t token, uint8_t *buffer, size_t buf_si
} }
pf = (flash_entry_header_t *) current_sector->address; pf = (flash_entry_header_t *) current_sector->address;
}
if (!blank_check(pf, total_size)) { if (!blank_check(pf, total_size)) {
rv = erase_sector(current_sector, pf); rv = erase_sector(current_sector, pf);
} }
}
} }
flash_entry_header_t *pn = (flash_entry_header_t *)(buffer - sizeof(flash_entry_header_t)); flash_entry_header_t *pn = (flash_entry_header_t *)(buffer - sizeof(flash_entry_header_t));

Loading…
Cancel
Save