service/sm: Slightly more efficient string name validation

We can check the end of the string first for null-termination, rather
than the beginning of the string.
This commit is contained in:
Lioncash 2020-09-17 10:54:09 -04:00
parent 78b1bc3b61
commit 057aa6275d

View File

@ -27,11 +27,11 @@ void ServiceManager::InvokeControlRequest(Kernel::HLERequestContext& context) {
} }
static ResultCode ValidateServiceName(const std::string& name) { static ResultCode ValidateServiceName(const std::string& name) {
if (name.size() <= 0 || name.size() > 8) { if (name.empty() || name.size() > 8) {
LOG_ERROR(Service_SM, "Invalid service name! service={}", name); LOG_ERROR(Service_SM, "Invalid service name! service={}", name);
return ERR_INVALID_NAME; return ERR_INVALID_NAME;
} }
if (name.find('\0') != std::string::npos) { if (name.rfind('\0') != std::string::npos) {
LOG_ERROR(Service_SM, "A non null terminated service was passed"); LOG_ERROR(Service_SM, "A non null terminated service was passed");
return ERR_INVALID_NAME; return ERR_INVALID_NAME;
} }