From f2f47f774f3d9c5da1a38b5f20cbfe59c2ff66af Mon Sep 17 00:00:00 2001 From: Shauren Date: Wed, 18 Mar 2020 23:20:11 +0100 Subject: Core/PacketIO: Add a validating string helper class for use in packet classes --- src/server/shared/Packets/ByteBuffer.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) (limited to 'src/server/shared/Packets/ByteBuffer.cpp') diff --git a/src/server/shared/Packets/ByteBuffer.cpp b/src/server/shared/Packets/ByteBuffer.cpp index 30557a05af1..0406da061e7 100644 --- a/src/server/shared/Packets/ByteBuffer.cpp +++ b/src/server/shared/Packets/ByteBuffer.cpp @@ -53,16 +53,16 @@ ByteBufferSourceException::ByteBufferSourceException(size_t pos, size_t size, message().assign(ss.str()); } -ByteBufferInvalidValueException::ByteBufferInvalidValueException(char const* type, size_t pos) +ByteBufferInvalidValueException::ByteBufferInvalidValueException(char const* type, char const* value) { - message().assign(Trinity::StringFormat("Invalid %s value found in ByteBuffer at pos " SZFMTD, type, pos)); + message().assign(Trinity::StringFormat("Invalid %s value (%s) found in ByteBuffer", type, value)); } ByteBuffer& ByteBuffer::operator>>(float& value) { value = read(); if (!std::isfinite(value)) - throw ByteBufferInvalidValueException("float", _rpos - sizeof(float)); + throw ByteBufferInvalidValueException("float", "infinity"); return *this; } @@ -70,7 +70,7 @@ ByteBuffer& ByteBuffer::operator>>(double& value) { value = read(); if (!std::isfinite(value)) - throw ByteBufferInvalidValueException("double", _rpos - sizeof(double)); + throw ByteBufferInvalidValueException("double", "infinity"); return *this; } @@ -85,7 +85,7 @@ std::string ByteBuffer::ReadCString(bool requireValidUtf8 /*= true*/) value += c; } if (requireValidUtf8 && !utf8::is_valid(value.begin(), value.end())) - throw ByteBufferInvalidValueException("string", _rpos - value.length() - 1); + throw ByteBufferInvalidValueException("string", value.c_str()); return value; } -- cgit v1.2.3