Skip to content

Commit f340d5a

Browse files
MrVanmathieupoirier
authored andcommitted
remoteproc: elf_loader: skip segment with memsz as zero
Per elf specification, p_filesz: This member gives the number of bytes in the file image of the segment; it may be zero. p_memsz: This member gives the number of bytes in the memory image of the segment; it may be zero. There is a case that i.MX DSP firmware has segment with PT_LOAD and p_memsz/p_filesz set to zero. Such segment needs to be ignored, otherwize rproc_da_to_va would report error. Signed-off-by: Peng Fan <peng.fan@nxp.com> Link: https://lore.kernel.org/r/20220413033038.1715945-2-peng.fan@oss.nxp.com Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
1 parent b30026f commit f340d5a

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/remoteproc/remoteproc_elf_loader.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ int rproc_elf_load_segments(struct rproc *rproc, const struct firmware *fw)
181181
bool is_iomem = false;
182182
void *ptr;
183183

184-
if (type != PT_LOAD)
184+
if (type != PT_LOAD || !memsz)
185185
continue;
186186

187187
dev_dbg(dev, "phdr: type %d da 0x%llx memsz 0x%llx filesz 0x%llx\n",

0 commit comments

Comments
 (0)