PacketAnalyzer: Spawn Packets

[thefoof]

It looks like spawn packets aren't loading correctly now (tested with 57080 logs)



Source array was not long enough. Check srcIndex and length, and the array's lower bounds.


************** Exception Text **************
System.ArgumentException: Source array was not long enough. Check srcIndex and length, and the array's lower bounds.
at System.Array.Copy(Array sourceArray, Int32 sourceIndex, Array destinationArray, Int32 destinationIndex, Int32 length, Boolean reliable)
at System.Array.Copy(Array sourceArray, Int32 sourceIndex, Array destinationArray, Int32 destinationIndex, Int32 length)
at EQ2PacketAnalyzer.Common.UnpackSpawnPacket(TreeNodeDataChild node) in c:\Users\Chris\Desktop\EQ2_Emu\eq2tools\trunk\DevTools\PacketAnalyzer\PacketAnalyzer\Common.cs:line 450
at EQ2PacketAnalyzer.Main.LoadPacketStruct(TreeNode node) in c:\Users\Chris\Desktop\EQ2_Emu\eq2tools\trunk\DevTools\PacketAnalyzer\PacketAnalyzer\Main.cs:line 585
at EQ2PacketAnalyzer.Main.treeView_opcodes_AfterSelect(Object sender, TreeViewEventArgs e) in c:\Users\Chris\Desktop\EQ2_Emu\eq2tools\trunk\DevTools\PacketAnalyzer\PacketAnalyzer\Main.cs:line 525
at System.Windows.Forms.TreeView.OnAfterSelect(TreeViewEventArgs e)
at System.Windows.Forms.TreeView.TvnSelected(NMTREEVIEW* nmtv)
at System.Windows.Forms.TreeView.WmNotify(Message& m)
at System.Windows.Forms.TreeView.WndProc(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.OnMessage(Message& m)
at System.Windows.Forms.Control.ControlNativeWindow.WndProc(Message& m)
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

[John Adams]

Could the structs just be off? We shouldn't be crashing, but still the index and length are int32s, and if the values the struct offer are bigger than that, yeah it will explode

If I have time this weekend, I'll check it out.

[thefoof]

Okay I attempted some debugging and found that when I deleted the arrays from the structs, it was still loading these elements so I figured it was loading the old header, and sure enough it was . Set the new header as version 1 and the packets loaded up fine. So I guess it isn't loading the correct version of the spawn header for some reason, hope that helps.

[Cynnar]

Flagged to be archived.