When I try:
char *string;
strcpy(string, "hello");
...It works.
But when I do:
char *strings[10];
strcpy(strings[0], "hello");
...The program crashes.
I know I can of course do:
strings[0] = "hello";
...but in this application, the memory it will point to will be overwritten in later calls.
Any ideas?
Code:#include <cmath> #include <complex> bool euler_flip(bool value) { return std::pow ( std::complex<float>(std::exp(1.0)), std::complex<float>(0, 1) * std::complex<float>(std::atan(1.0) *(1 << (value + 2))) ).real() < 0; }
what about if you just continue to do it this way?
strcpy(string, "Hello")
It worked for me and didn't crash.
NEVER PET YOUR DOG WHILE IT'S ON FIRE!
> char *string;
> strcpy(string, "hello");
> ...It works.
string was unallocated, you were just being lucky
Try it with char *string = NULL;
Either way, it's wrong - you need to allocate memory before the strcpy
> char *strings[10];
> strcpy(strings[0], "hello");
> ...The program crashes.
Same dice - pointers are still uninitialised, just that this time you lost.
> ...but in this application, the memory it will point to will be overwritten in later calls.
strings[0] = malloc( strlen("hello") + 1 );
strcpy ( strings[0], "hello" );
Is the only way to go
And remember, strcpy() is looking for the address of a dataspace, not an address to an address to a dataspace...
Learn how to use '(' and ')' to help your compiler correctly build dereferencing statements.
Hi!
Use the function strncpy()!
So you can prevent a buffer-overflow!!
klausi
When I close my eyes nobody can see me...
