Log when SystemMessage has SystemMessageId with null parameter.
This commit is contained in:
@@ -393,7 +393,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -406,7 +405,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -393,7 +393,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -406,7 +405,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -400,7 +400,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -413,7 +412,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -400,7 +400,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -413,7 +412,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -400,7 +400,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -413,7 +412,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -400,7 +400,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -413,7 +412,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -400,7 +400,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -413,7 +412,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -400,7 +400,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -413,7 +412,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -400,7 +400,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -413,7 +412,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -400,7 +400,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -413,7 +412,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -400,7 +400,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -413,7 +412,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -407,7 +407,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -420,7 +419,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -407,7 +407,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -420,7 +419,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -407,7 +407,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -420,7 +419,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
@@ -400,7 +400,6 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
{
|
{
|
||||||
params[i] = _params[i].getValue();
|
params[i] = _params[i].getValue();
|
||||||
}
|
}
|
||||||
|
|
||||||
packet.writeH(SystemMessageId.S1_2.getId());
|
packet.writeH(SystemMessageId.S1_2.getId());
|
||||||
packet.writeC(0x01);
|
packet.writeC(0x01);
|
||||||
packet.writeC(TYPE_TEXT);
|
packet.writeC(TYPE_TEXT);
|
||||||
@@ -413,7 +412,14 @@ public class SystemMessage implements IClientOutgoingPacket
|
|||||||
packet.writeC(_params.length);
|
packet.writeC(_params.length);
|
||||||
for (SMParam param : _params)
|
for (SMParam param : _params)
|
||||||
{
|
{
|
||||||
|
if (param == null)
|
||||||
|
{
|
||||||
|
LOGGER.warning("Found null parameter for SystemMessageId " + _smId);
|
||||||
|
continue;
|
||||||
|
}
|
||||||
|
|
||||||
packet.writeC(param.getType());
|
packet.writeC(param.getType());
|
||||||
|
|
||||||
switch (param.getType())
|
switch (param.getType())
|
||||||
{
|
{
|
||||||
case TYPE_ELEMENT_NAME:
|
case TYPE_ELEMENT_NAME:
|
||||||
|
Reference in New Issue
Block a user