Merge pull request #6736
05ad4fa39
epee: further defending against exceptions in command handlers (moneromooo-monero)
This commit is contained in:
commit
43a4fd9e16
@ -291,6 +291,7 @@ namespace epee
|
|||||||
#define BEGIN_INVOKE_MAP2(owner_type) \
|
#define BEGIN_INVOKE_MAP2(owner_type) \
|
||||||
template <class t_context> int handle_invoke_map(bool is_notify, int command, const epee::span<const uint8_t> in_buff, std::string& buff_out, t_context& context, bool& handled) \
|
template <class t_context> int handle_invoke_map(bool is_notify, int command, const epee::span<const uint8_t> in_buff, std::string& buff_out, t_context& context, bool& handled) \
|
||||||
{ \
|
{ \
|
||||||
|
try { \
|
||||||
typedef owner_type internal_owner_type_name;
|
typedef owner_type internal_owner_type_name;
|
||||||
|
|
||||||
#define HANDLE_INVOKE2(command_id, func, type_name_in, typename_out) \
|
#define HANDLE_INVOKE2(command_id, func, type_name_in, typename_out) \
|
||||||
@ -336,7 +337,13 @@ namespace epee
|
|||||||
LOG_ERROR("Unknown command:" << command); \
|
LOG_ERROR("Unknown command:" << command); \
|
||||||
on_levin_traffic(context, false, false, true, in_buff.size(), "invalid-command"); \
|
on_levin_traffic(context, false, false, true, in_buff.size(), "invalid-command"); \
|
||||||
return LEVIN_ERROR_CONNECTION_HANDLER_NOT_DEFINED; \
|
return LEVIN_ERROR_CONNECTION_HANDLER_NOT_DEFINED; \
|
||||||
|
} \
|
||||||
|
catch (const std::exception &e) { \
|
||||||
|
MERROR("Error in handle_invoke_map: " << e.what()); \
|
||||||
|
return LEVIN_ERROR_CONNECTION_TIMEDOUT; /* seems kinda appropriate */ \
|
||||||
|
} \
|
||||||
}
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user