(Please CC me, I'm not on the list)
According to:
http://undocumented.ntinternals.net/UserMode/Undocumented%20Functions/Execut...
The parameter order for LdrGetProcedureAddress is (Handle, Function, Ordinal, Address)
The file dlls/ntdll/module.c has the order as (Handle, Ordinal, Function, Address)
This of course works well because kernel32.dll is the only caller (and it indeed uses, if not the correct, then at least the same order).
I haven't attached a patch, because I'm not actually sure which version is right. Anyone?