segmentation fault (core dump)

Question

I'm getting a segmentation error (core dump) when I try to run this. It compiles perfectly but I get the error, and I don't know why. I've tried to edit my code in all possible ways, but am still getting this error. I'm out of ideas already. Any help would be great. Thanks!

    unsigned short *reg = NULL;

    int byte;
    int i;
    for (byte = 0; byte < num_bytes; byte++){
        unsigned int next_byte = (unsigned int) message[byte];
        crc_byte(reg, key, next_byte);
    }

    for (i = 0; i < 16; i++){
        crc_bit(reg, key, 0);
    }

    return *reg;
}

Please have a look at this: stackoverflow.com/questions/5115613/core-dump-file-analysis — Axel
– Axel, Commented Feb 10, 2014 at 7:30
Downvoted because answers are all grasping for straws. The posted code is nowhere near complete enough to know why a seg fault is occurring. People answering can only guess. — shawn1874
– shawn1874, Commented Aug 16, 2023 at 17:25

ooga · Accepted Answer · 2014-02-10 07:35:02Z

5

Compile with debugging info:

> gcc -o myprog myprog.c -ggdb

Run in a debugger

> gdb myprog
(gdb) run

Debugger tells you where the segfault occurred:

Program received signal SIGSEGV, Segmentation fault.
0x0040133d in crc_bit (reg=0x0, key=12345, next_bit=0) at rrr.c:4
4           unsigned int msb = (*reg >> (sizeof(*reg)-1)) & 1;

Note that reg is 0 (i.e., NULL) and you dereference it.

answered Feb 10, 2014 at 7:35

ooga

15.6k2 gold badges23 silver badges23 bronze badges

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

Comments

NPE · Accepted Answer · 2014-02-10 07:29:26Z

2

You are passing a NULL reg into crc_byte(), which passes it to crc_bit(), which then tries to dereference it.

Change the function like so:

unsigned short reg = 0;  /* replace 0 with whatever value is appropriate */
...

for (byte = 0; byte < num_bytes; byte++){
    ...
    crc_byte(&reg, key, next_byte);  /* added the ampersand */
}

for (i = 0; i < 16; i++){
    crc_bit(&reg, key, 0);  /* added the ampersand */
}

return reg;  /* removed the asterisk */

answered Feb 10, 2014 at 7:29

NPE

503k114 gold badges970 silver badges1k bronze badges

Comments

nomoney29 · Accepted Answer · 2014-02-10 07:48:12Z

0

For me, your segmentation fault problem comes from the reg pointer which is NULL. This means that you will modify an unisgned hsort value located at address zero. On most operating systems, this is not allowed.

Why don't you do the following thing ?

unsigned short crc_message(unsigned int key, char *message, int num_bytes) {

unsigned short reg;

int byte;
int i;
for (byte = 0; byte < num_bytes; byte++){
    unsigned int next_byte = (unsigned int) message[byte];
    crc_byte(&reg, key, next_byte);
}

for (i = 0; i < 16; i++){
    crc_bit(&reg, key, 0);
}

return reg;

}

answered Feb 10, 2014 at 7:48

nomoney29

1261 gold badge2 silver badges5 bronze badges

Comments

ssm · Accepted Answer · 2014-02-10 07:32:22Z

0

reg is NULL in crc_message. This gets passed on to crc_byte which gets passed on to crc_bit. Then use access a location which has an address NULL.

answered Feb 10, 2014 at 7:32

ssm

5,4031 gold badge26 silver badges43 bronze badges

Collectives™ on Stack Overflow

segmentation fault (core dump)

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related