Skip to content

riscv: modules: Fix module loading error handling

Charlie Jenkins linux-riscv lkml 3 versions – Applied

Status from this message: This series was applied to riscv/linux.git (for-next)

Get series with b4
b4 am [email protected] change-id 20231213-module_loading_fix-3ac6d4ea8129

Versions newest first

  1. v3 Latest3 patches 1 replyApplied 3/3
  2. v24 patches 2 repliesSuperseded
  3. v11 patch 0 repliesSuperseded

Changes in v3 from the cover letter

- Drop patch using do-while
- Link to v2: https://lore.kernel.org/r/[email protected]

Patches in v3 All 3 applied

#Subject and filesSizeRepliesApplied
1/3riscv: Fix module loading free orderarch/riscv/kernel/module.c+1 -10 repliesApplied
2/3riscv: Correctly free relocation hashtable on errorarch/riscv/kernel/module.c+7 -30 repliesApplied
3/3riscv: Fix relocation_hashtable sizearch/riscv/kernel/module.c+1 -10 repliesApplied

Reviews trailers per patch

None of the 3 patches has a Reviewed-by or Acked-by of their own; the series has one for all of them.

PatchDan Carpenter
SeriesR-b
1/3
2/3
3/3

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v3 thread

Cover letter [PATCH v3 0/3] riscv: modules: Fix module loading error handling

When modules are loaded while there is not ample allocatable memory, there was previously not proper error handling. This series fixes a use-after-free error and a different issue that caused a non graceful exit after memory was not properly allocated.

---
Changes in v3:
- Drop patch using do-while
- Link to v2: https://lore.kernel.org/r/[email protected]

Changes in v2:
- Split changes across multiple patches
- Link to v1: https://lore.kernel.org/r/[email protected]

---
Charlie Jenkins (3):
riscv: Fix module loading free order
riscv: Correctly free relocation hashtable on error
riscv: Fix relocation_hashtable size

 arch/riscv/kernel/module.c | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

---
base-commit: a39b6ac3781d46ba18193c9dbb2110f31e9bffe9
change-id: 20231213-module_loading_fix-3ac6d4ea8129

Signature · 1 line
- Charlie