23

I am wondering why am I getting segmentation fault in the below code.

int main(void)
{
        char str[100]="My name is Vutukuri";
        char *str_old,*str_new;

        str_old=str;
        strcpy(str_new,str_old);

        puts(str_new);

        return 0;
}
Teja
  • 13,214
  • 36
  • 93
  • 155

3 Answers3

41

You haven't initialized *str_new so it is just copying str_old to some random address. You need to do either this:

char str_new[100];

or

char * str = (char *) malloc(100);

You will have to #include <stdlib.h> if you haven't already when using the malloc function.

Oli4
  • 489
  • 1
  • 9
  • 22
seanwatson
  • 1,053
  • 1
  • 9
  • 12
9

str_new is an uninitialized pointer, so you are trying to write to a (quasi)random address.

geekosaur
  • 59,309
  • 11
  • 123
  • 114
3

Because str_new doesn't point to valid memory -- it is uninitialized, contains garbage, and likely points into memory that's not even mapped if you're getting a segmentation error. You have to make str_new point to a valid block of memory large enough to hold the string of interest -- including the \0 byte at the end -- before calling strcpy().

Ernest Friedman-Hill
  • 80,601
  • 10
  • 150
  • 186