Messages in this thread | | | Date | Sun, 4 Aug 2013 18:30:02 +0200 | From | Oleg Nesterov <> | Subject | Re: [PATCH 0/3] exec: minor cleanups + minor fix |
| |
Andrew,
please ignore this and the next series, I'll resend everything with a couple of cosmetic changes.
Plus I believe audit_bprm() is not right too, this needs another patch.
On 08/01, Oleg Nesterov wrote: > > Andrew, > > By discussion with Zach, please drop > > 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 > > Zach is working on v2. > > Meanwhile, can't we cleanup search_binary_handler() a bit? It > doesn't look nice imho. And we can certainly cleanup it more > (perhaps I'll try tomorrow). The error handling, request_module() > logic and even "for (try=0; try<2; try++)" looks horrible imho. > > Oleg.
| |