Trace post of my spell loading crash, why is spell_us loaded twice? is this normal?

[Dakaar]

Update!
I put a debug message in my fileloadspdat() function.
During the run where I crash, it reaches the fgets(spell_line, sizeof(spell_line), sf);
twice, after the 2nd run, it crashes.

Is this because my SPDAT_RECORDS is > 0 and the spellspointer is equal to 0?

Okay I was looking at the trace again.

Only the second open has the following lines:
write(1, "[Status] FileLoadSPDat() Loading"..., 61) = 61
write(4, "17466 [09.24. - 19:10:40] FileLo"..., 7 = 78


Seems to me that its crashing in zone\net.cpp after the first line is read, and probably cant find the sp pointer or similar.

I wonder if i have a problem with the seperator object, as its not used in the dll load that may have something to do with it?

Is the first open the DLL open and the second the zone binary open? Are both supposed to load spells?

[Dakaar]

Fixed! Seperator sep and int y = 0 are redefined every step of the loot.

Declare y in the beginning of the function and seperator sep must become a pointer!

edit: i get a fault with my delete , what am I doing wrong?
I think the problem is that on the last line, the new seperator does not return a correct pointer.



Zone loads fine if i have no delete, I put it at the end of the while loop, so not entirely sure what i did wrong.

add:
+ Seperator *sep = NULL;
+ int y = 0;

in the top of the function

change
- Seperator sep(spell_line, '^', 200, 100, false, 0, 0, false);
+ sep = new Seperator(spell_line, '^', 200, 100, false, 0, 0, false);

+ if(sep != NULL)
+ delete sep;


change all sep.(whatever) to sep->(whatever)
and add an additional if(sep) delete sep; line at the end of the function after the loop exits.