Ummm, why isnt this working?

[gp364481]

int main(void){
	int size, start;
	char *string = "Two words";

	size = (int) strlen(string);
	start = 0;

	reverseString(string, start, size - 1);

	return 0;
}
void reverseString(char *string, int start, int end){
	char temp;

	while(end > start){
		temp = string[start];
		string[start] = string[end];      /* <------ crashes here */
		string[end] = temp;

		start++; end--;
	}
	printf("reversed string: %s\n", string);
}

I'm trying to take a string: "Two words" and turn it into "sdrow owT".
I guess there is something wrong with my function though.
The program just hangs for a while after I execute it and then it crashes.
Any suggestions? Thanks!

[laserlight]

The problem may be due to the fact that you are trying to change a string literal. Instead of:

char *string = "Two words";

write:

char string[] = "Two words";

[gp364481]

The problem may be due to the fact that you are trying to change a string literal. Instead of:

char *string = "Two words";

write:

char string[] = "Two words";

Yes, you are correct. Thank you very much.
Do you care to explain why exactly that happens?

[laserlight]

Do you care to explain why exactly that happens?

Undefined behaviour happens since you are not allowed to modify a string literal.

[MK27]

Because you cannot modify a string literal, just like you cannot modify one of these:

#define string "two words"
const char string[] = "two words";

Whereas you can modify a non-const array:

char string[] = "Two words";

[gp364481]

Okay, thank you.