[GitHub] [incubator-dubbo] 46557783 opened issue #2619: is there a problem in NettyBackedChannelBuffer.setBytes(...)?

["46557783 (GitHub)" <gi...@apache.org>]

hi,I see the  code for method setBytes in class com.alibaba.dubbo.remoting.transport.netty4.NettyBackedChannelBuffer.java which is below,is there a problem ?

![image](https://user-images.githubusercontent.com/20593632/46648646-90eaba00-cbc8-11e8-896d-9a1d235d7e0b.png)



[ Full content available at: https://github.com/apache/incubator-dubbo/issues/2619 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org

[GitHub] [incubator-dubbo] beiwei30 closed issue #2619: is there a problem in NettyBackedChannelBuffer.setBytes(...)?

["beiwei30 (GitHub)" <gi...@apache.org>]

[ issue closed by beiwei30 ]

[ Full content available at: https://github.com/apache/incubator-dubbo/issues/2619 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org

[GitHub] [incubator-dubbo] qinliujie commented on issue #2619: is there a problem in NettyBackedChannelBuffer.setBytes(...)?

["qinliujie (GitHub)" <gi...@apache.org>]

This is really a bug, but it won't affect it at the moment, because I searched it, there is no place to use this method. please @zonghaishang  check again

[ Full content available at: https://github.com/apache/incubator-dubbo/issues/2619 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org

[GitHub] [incubator-dubbo] zonghaishang commented on issue #2619: is there a problem in NettyBackedChannelBuffer.setBytes(...)?

["zonghaishang (GitHub)" <gi...@apache.org>]

I have checked that, @46557783 Could you make a pull request to fix it?

[ Full content available at: https://github.com/apache/incubator-dubbo/issues/2619 ]
This message was relayed via gitbox.apache.org for notifications@dubbo.apache.org