C Pointer for Struct - Segmentation fault

Question

I am having problems with this program. It's very simply. I need to assign values to my struct from the pointers I created, but I keep getting a segmentation fault. Any ideas what I'm doing wrong:

#include <stdio.h>
#include <stdlib.h>

struct problem37
{
    int a;
    int b;
    int c;
};

int main()
{
    printf("Problem 37\n");

    //create struct
    struct problem37 myStruct;

    //create the pointer
    int* p;
    int* q;
    int* r;

    *p = 1;
    *q = 5;
    *r = 8;

    //read the data into the struct using the pointers
    myStruct.a = *p;
    myStruct.b = *q;
    myStruct.c = *r;

    printf("%d\n", myStruct.a);
    printf("%d\n", myStruct.b);
    printf("%d\n", myStruct.c);

    return 0;
}

And where do you think your pointers point to?

ruslik
– ruslik

2010-11-14 22:46:42 +00:00
Commented Nov 14, 2010 at 22:46 — ruslik
– ruslik, Commented Nov 14, 2010 at 22:46

peoro · Accepted Answer · 2010-11-14 22:47:23Z

7

Your are assigning a value to *p, *q and *r, but they are not initialized: they're pointers pointing to random memory.

You need to initialize them, either assigning them a new value allocated in the heap (with malloc):

int *p = (int*) malloc( sizeof(int) );
*p = 1;

or making them point to an already existent value:

int x;
int *p = &x;
*p = 1; // it's like doing x=1

answered Nov 14, 2010 at 22:47

peoro

26.2k21 gold badges102 silver badges153 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

ThiefMaster · Accepted Answer · 2010-11-14 22:46:52Z

7

Your problem is that you write at random memory locations since you do not initialize your pointers nor allocate memory.

You could do the following:

int* p = malloc(sizeof(int));
int* q = malloc(sizeof(int));
int* r = malloc(sizeof(int));

Obviously you need to free them when you are done using them:

free(p);
free(q);
free(r);

answered Nov 14, 2010 at 22:46

ThiefMaster

320k85 gold badges608 silver badges648 bronze badges

Comments

user400055 · Accepted Answer · 2010-11-14 22:46:15Z

3

You're not allocating memory to the pointer. Therefore when you're doing *p and *q and *r you're dereferencing a null pointer (or a random pointer). This leads to a segmentation fault. Use p = malloc(sizeof(int)); when you declare the variables.

answered Nov 14, 2010 at 22:46

user400055

Collectives™ on Stack Overflow

C Pointer for Struct - Segmentation fault

3 Answers 3

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related