Google-Code-Fork / tibiaapi

Automatically exported from code.google.com/p/tibiaapi
MIT License
0 stars 0 forks source link

Proxy is unstable again #213

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
Who knows that?

What is the expected output? What do you see instead?
Proxy is freezing Tibia

What version of the product are you using? On what operating system?
Newest TibiaAPI and Windows XP SP3 x86 or Windows 7 x64

Please provide any additional information below.
After some time of playing proxy freezes Tibia.
Video: http://www.youtube.com/watch?v=Xs2NKIEZohQ

Original issue reported on code.google.com by muttle...@gmail.com on 30 Aug 2010 at 6:40

GoogleCodeExporter commented 9 years ago
Agree, i have this same issue.

Original comment by Piotr3k1...@gmail.com on 2 Sep 2010 at 5:55

GoogleCodeExporter commented 9 years ago
The strange thing is that it does work much better in VS 2008

Original comment by muttd...@gmail.com on 2 Sep 2010 at 7:07

GoogleCodeExporter commented 9 years ago
We aren't yet sure what the issue is here. How long does it take to reproduce? 
What is in the proxy log?

Original comment by ian320 on 6 Sep 2010 at 7:06

GoogleCodeExporter commented 9 years ago
Hmm, I'll take log now (I have disabled it)

Original comment by muttd...@gmail.com on 6 Sep 2010 at 7:12

GoogleCodeExporter commented 9 years ago
2010-09-06 21:19:50 System.ArgumentOutOfRangeException: Index and count must 
refer to a location within the buffer.
Parameter name: bytes
   at System.Text.EncodingNLS.GetString(Byte[] bytes, Int32 index, Int32 count)
   at Tibia.Packets.NetworkMessage.GetString() in H:\Downloady\programowanie\SmallBot\MuttleyBot\TibiaAPI\Packets\NetworkMessage.cs:line 174
   at Tibia.Packets.Incoming.CreatureSpeechPacket.ParseMessage(NetworkMessage msg, PacketDestination destination) in H:\Downloady\programowanie\SmallBot\MuttleyBot\TibiaAPI\Packets\Incoming\CreatureSpeechPacket.cs:line 60
   at Tibia.Packets.ProxyBase.ParsePacketFromServer(Client client, NetworkMessage msg, NetworkMessage outMsg) in H:\Downloady\programowanie\SmallBot\MuttleyBot\TibiaAPI\Packets\ProxyBase.cs:line 198
   at Tibia.Packets.Proxy.ServerReadCallBack(IAsyncResult ar) in H:\Downloady\programowanie\SmallBot\MuttleyBot\TibiaAPI\Packets\Proxy.cs:line 603
Last received packet types: CreatureSpeech, TileRemoveThing, ContainerOpen, 
TileAddThing, MoveEast, CreatureMove, MoveEast, CreatureMove, TileRemoveThing, 
TextMessage

Here you are

Original comment by muttd...@gmail.com on 6 Sep 2010 at 7:21

GoogleCodeExporter commented 9 years ago
And here is my proxy log.

2010-09-03 14:15:26 System.ArgumentOutOfRangeException: Index and count must 
refer to a location within the buffer.
Parameter name: bytes
   at System.Text.EncodingNLS.GetString(Byte[] bytes, Int32 index, Int32 count)
   at Tibia.Packets.NetworkMessage.GetString()
   at Tibia.Packets.Incoming.CreatureSpeechPacket.ParseMessage(NetworkMessage msg, PacketDestination destination)
   at Tibia.Packets.ProxyBase.ParsePacketFromServer(Client client, NetworkMessage msg, NetworkMessage outMsg)
   at Tibia.Packets.Proxy.ServerReadCallBack(IAsyncResult ar)
Last received packet types: CreatureSpeech, MagicEffect, CreatureMove, 
CreatureHealth, CreatureHealth, AnimatedText, MagicEffect, CreatureHealth, 
MagicEffect, CreatureMove

Original comment by Piotr3k1...@gmail.com on 6 Sep 2010 at 8:13

GoogleCodeExporter commented 9 years ago

Original comment by ian320 on 8 Sep 2010 at 4:00

GoogleCodeExporter commented 9 years ago
Sth changed in CreatureSpeech Packet

Original comment by muttd...@gmail.com on 8 Sep 2010 at 6:14

GoogleCodeExporter commented 9 years ago
Position: 38 Len: 31785 Length: 47

Friend tried to debug. Something is wrong with length reading. 
CreatureSpeechPacket

Original comment by muttd...@gmail.com on 8 Sep 2010 at 6:35

GoogleCodeExporter commented 9 years ago
Ian i tested newest revision of TibiaAPI and problem isn't fixed, here is log.

2010-09-09 08:18:50 System.ArgumentOutOfRangeException: Index and count must 
refer to a location within the buffer.
Parameter name: bytes
   at System.Text.EncodingNLS.GetString(Byte[] bytes, Int32 index, Int32 count)
   at Tibia.Packets.NetworkMessage.GetString()
   at Tibia.Packets.Incoming.CreatureSpeechPacket.ParseMessage(NetworkMessage msg, PacketDestination destination)
   at Tibia.Packets.ProxyBase.ParsePacketFromServer(Client client, NetworkMessage msg, NetworkMessage outMsg)
   at Tibia.Packets.Proxy.ServerReadCallBack(IAsyncResult ar)
Last received packet types: CreatureSpeech, CreatureMove, CreatureMove, 
TileTransformThing, CreatureMove, CreatureMove, CreatureMove, 
TileTransformThing, CreatureMove, TextMessage

Original comment by Piotr3k1...@gmail.com on 9 Sep 2010 at 1:34

GoogleCodeExporter commented 9 years ago
Works much better but crashed on PM
18:32 Saito Master [9]: leads me in the troll 

Original comment by muttd...@gmail.com on 9 Sep 2010 at 4:32

GoogleCodeExporter commented 9 years ago
I think that's the reason - tested 5 PMs, 5 freezed clients

Original comment by muttd...@gmail.com on 9 Sep 2010 at 4:52

GoogleCodeExporter commented 9 years ago
Should be fixed now. Tested with PMs and creature speech. Also made sure the 
freezes don't happen anymore even if protocol changes.

Original comment by ian320 on 12 Sep 2010 at 3:41

GoogleCodeExporter commented 9 years ago
Works nice, but don't close - Let's wait 3 days, if no freezes then close ;)

Original comment by muttd...@gmail.com on 12 Sep 2010 at 11:22

GoogleCodeExporter commented 9 years ago
Close, fixed

Original comment by muttle...@gmail.com on 17 Sep 2010 at 8:37