On 9/12/07, Sungwon Jung <[EMAIL PROTECTED]> wrote:
> thank you reply.
>
> doDecode() implementation is below.
>
> --
> public class ACUPacketReqAnalysis extends CumulativeProtocolDecoder
> {
> Logger logger = Logger.getLogger( this.getClass() );
>
> private int device = Constants.DEVICE_NONE;
>
> protected boolean doDecode(IoSession session, ByteBuffer in,
> ProtocolDecoderOutput out) throws Exception {
> int start = in.get( 0 ) & 0xFF;
>
> if( start == PacketDefineACU.START ) {
> session.setAttribute( Constants.SESSION_DEVICE,
> Constants.DEVICE_ACU );
>
> ACUMessageReq acu_request = new ACUMessageReq(
> session, in, out, start );
> out.write( acu_request );
>
> return true;
> } else {
> return false;
> }
> }
> }
It seems like you don't move the position of the input buffer after
decoding a message. Please more the buffer position to the end of the
message (note that it's not the end of the buffer).
And you shall not use absolute addressing (e.g. in.get(0)) because you
cannot assume that the offset (position) of the first byte is always
zero.
HTH,
Trustin
--
what we call human nature is actually human habit
--
http://gleamynode.net/
--
PGP Key ID: 0x0255ECA6
