Question

I am new to C programming and I am unable to figure out why this is happening.

char *example(char *temp) {
      char *xyz;
      memset(xyz,0,strlen(temp));
      strncpy(xyz,temp,3);
      strcat(xyz,"XX);
      return xyz;
      }

int main() {
      char temp[] = "ABCDEF"
      char *xyz;
      xyz = example(temp);
      printf("Returned string is: %s",xyz);
      return 0;
      }

I just want to know what all I am doing wrong cause I tried to run GDB and it showed I am trying to access some unreachable part of memory but am not able to figure it out. Thanks in advance.

Answer 1

The problem is here:

char *xyz;
memset(xyz,0,strlen(temp));

You never initialized xyz. Therefore it’s an invalid pointer.

What you need to do it allocate something for it:

char *xyz = malloc(strlen(temp) + 1);   //  Allocate, +1 is for the '\0'.
memset(xyz,0,strlen(temp));

and free it when you’re done with it:

int main() {
    char temp[] = "ABCDEF"
    char *xyz;
    xyz = example(temp);
    printf("Returned string is: %s",xyz);

    free(xyz);   //  Free

    return 0;
}

EDIT : Other problems:

These 3 lines here are very dangerous:

  memset(xyz,0,strlen(temp));
  strncpy(xyz,temp,3);   //  The nul is not copied in strncpy.
  xyz[3] = '\0';         //  Add the nul manually.
  strcat(xyz,"XX");      //  You need to ensure that xyz is large enough.