Thread: Segmentation Fault

I'm getting a segmentation fault and I don't know how to fix it. I have narrowed it down to where it's occurring but I don't know how I am causing it. From my understanding a Segmentation Fault has something to do with the program either accessing illegal data or trying to illegally overwrite data.

I'm going to post my code but be warned, it's really long and I understand if you don't want to help.

There are 4 files (2 header and 2 .cpp). The first two are the driver programs that I was given. The last two are the files that I created to allow the driver program to work.

Here's where the problem is. In:

Code:

//strdrv.cpp (driver program)

void test10()

    {

    system("cls");

    cout << "10. Testing: String concatenation." << endl << endl;

    csis << "10. Testing: String concatenation." << endl << endl;

    String s10("DEF");

    String t10('H');

    String u10("ABC" + s10 + "G" + t10 + 'I');

    u10.setName("u10");

    u10.print();

    String v10('X' + u10);

    v10.setName("v10");

    v10.print();

    wait();

    }

In bold is where the Seg Fault is occuring. Adding all of strings, objects, and char's together is calling some friend functions that I created in //string.cpp.

>> length = strlen(&oneCharBuf);
If it is a single char, then the length must be 1. Also, strlen won't work because it expects a null terminated array, not a pointer to a single character.

Also, in your destructor, you delete [] name. But you don't initialize name in any of your constructors. I didn't look at all the code to see if you are using it, but if you are, you should initialize it to null (0) if you don't assign new memory to it. That way delete [] name won't fail.

Thanks so far. I'm going to try out all that you suggested. I'm also going to update the code that I wrote to reflect the changes. In all the + overloaded operators I just linked them to the += operator instead of having all of the excess code. That part seems to be working for now.

I just edited my code posts with update code. I changed a lot of stuff and commented other stuff out.

The problem I'm having right now is with "test 17" in the driver program:

Code:

void test17()

    {

    system("cls");

    cout << "17. Testing: Substr function." << endl << endl;

    csis << "17. Testing: Substr function." << endl << endl;

    String s17("Jai Guru Dev"), t17;

    s17.setName("s17");

    t17.setName("t17");

    t17 = s17.substr(4, 8);

    s17.print();

    t17.print();

    wait();

    }

I highlighted the probelm areas. The "substr(x,y)" function is supposed to take the string from the object, start at x position, and continue for y spaces making a new string with those characters. So in this instance t17 should equal "Guru Dev". For some reason when I compile it and run it t17 is equal to "Guru Dev Morgan". I have no idea how "Morgan" got there.

I did some debugging and found that when I try and make a new character pointer to hold the memory something screws up and it makes a bigger character array and just fills it with junk. I've been trying to fix it but I can't figure out how. If you could help me that would be awesome!!!

The reason is that you never append the terminating 0 character to the substring.

Code:

string[0] = '\n';
//fill string here

This is useless, because you'll overwrite the 0-character after all.

Thank you! I just figured it would just be taken care of by itself. I didn't initially mean for the program to take care of it but it just slipped my mind because I haven't been appending the null character to the end of any of the strings.