Greetings oh confused one, having looked at your code snippet and read Edwin's response to you, and your answer to him... I feel the need to but in :) -dons flame-proof underwear-
> > > save_char_obj( ch ); > > > id = ch->id; > > > d = ch->desc; > > > d->host = str_dup(ch->pcdata->host); > > > extract_char( ch, TRUE ); > > > > That line with the str_dup() is not stock Rom. Why have you > > added it? > > the line mentioned above uses a member of pcdata that stock does _not_ have. namely the host member. look through your code to the definition of the pcdata structure and see if your version actually does have a member named host in it. if not remove the offending str_dup line and life will be good. you mention below running it through the MSVC++ debugger, that will _not_ modify your code unless you do :) nothing gets changed automatically to the best of my knowledge. you will most likely find that it does not exist, if on the other hand, it does exist then you need some check to make sure that you are not dealing with invalid variables. check for d != NULL and ch->pcdata->host !=NULL before trying to copy the line. unless you have a wildly different do_quit than stock, the line you are worried about does nothing for you. later in the function, d is reassigned to check the descriptor_list, so copying the pcdata->host to the d->host makes no sense. > I am not exactly certain. I was not the one who put it there, as I am using > code called "Startermud". it is stock rom with ... <snipped> > The only thing I can think of, is that perhaps running it in the MSVC++ > debugger did something to it...but for the life of me I can not even fathom > what it might be. good luck with your coding sir/ma'am -goes back to lurking- /-------------------------------------------------------------\ | Steve Boleware | Beginning Coder | | [EMAIL PROTECTED] | Student of Life | | ICQ: 11120901 | | \-------------------------------------------------------------/
