exec: cleanup search_binary_handler()
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 Latest8 patches 0 repliesNew
- v13 patches 8 repliesSuperseded
Patches in v2 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/8 | exec: introduce exec_binprm() for "depth == 0" codefs/ | +22 | 0 replies | |
| 2/8 | exec: kill "int depth" in search_binary_handler()fs/ include/ | +5 | 0 replies | |
| 3/8 | exec: proc_exec_connector() should be called only oncefs/ | +1 | 0 replies | |
| 4/8 | exec: move allow_write_access/fput to exec_binprm()fs/ | +6 | 0 replies | |
| 5/8 | exec: kill ->load_binary != NULL check in search_binary_handler()fs/ | +3 | 0 replies | |
| 6/8 | exec: cleanup the CONFIG_MODULES logicfs/ | +31 | 0 replies | |
| 7/8 | exec: don't retry if request_module() failsfs/ | +2 | 0 replies | |
| 8/8 | exec: cleanup the error handling in search_binary_handler()fs/ | +3 | 0 replies |
Reviews trailers per patch
All 8 patches have a Reviewed-by or Acked-by.
| Patch | Kees Cook |
|---|---|
| 1/8 | A-b |
| 2/8 | A-b |
| 3/8 | A-b |
| 4/8 | A-b |
| 5/8 | A-b |
| 6/8 | A-b |
| 7/8 | A-b |
| 8/8 | A-b |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v2 0/8] exec: cleanup search_binary_handler()
To remind, by discussion with Zach
fs-binfmts-add-a-name-field-to-the-binfmt-struct.patch fs-binfmts-better-handling-of-binfmt-loops.patch fs-binfmts-whitespace-fixes-with-scripts-cleanfile.patch
should be dropped. Zach is working on the new version on top of these changes.
Changes:
2/8: add the comment about recursion_depth
4/8: nullify bprm->file to catch the potential use after free
Add the acks from Kees (thanks!)
Oleg.
fs/exec.c | 120 +++++++++++++++++++++++------------------------ include/linux/binfmts.h | 2 +- 2 files changed, 60 insertions(+), 62 deletions(-)