Why am I getting a segmentation fault in my code?

Question

When I compile my code with GCC and then I run it, when I call my function into my code, it prints out: "Segmentation fault (core dumped)".

I tried searching on google for solutions.

Here is my current code:

char ** saveLevelPositions() {
  int x, y;
  char ** positions;
  positions = malloc(sizeof(char *) * 25);

  for (y = 0; y < 25; y++) {
    positions[y] = malloc(sizeof(char) * 100);

    for (x = 0; x < 100; x++) {
      positions[x][y] = mvinch(y, x);
    }
  }

  return positions;
}

I expected the function to just run properly and it just gives a segmentation error.

EDIT: For a little bit of context, here is a link to the GitHub project: https://github.com/xslendix/rogue

That isn't a valid solution sadly, I tried it and I got the same problem. — Slendi
– Slendi, Commented Apr 5, 2019 at 17:36
It is definitely a valid part of a solution. If you still have a problem, then it is not in the posted code (mvinch)?. — Eugene Sh.
– Eugene Sh., Commented Apr 5, 2019 at 17:36
See How to create a Minimal, Complete, and Verifiable example. Also a debugger might be of some help. — sjsam
– sjsam, Commented Apr 5, 2019 at 17:39

Paul Ogilvie · Accepted Answer · 2019-04-05 18:08:34Z

5

As other answers and comments indicated, you should swap your use of x and y, so
positions[x][y] should be positions[y][x].

Also, you are not using the correct type to store the result of mvinch. In curses.h it says:

typedef unsigned long chtype;

so you should allocate memory as follows:

chtype ** positions;
positions = malloc(sizeof(chtype *) * 25);
positions[y] = malloc(sizeof(chtype) * 100);

And turn warnings of your compiler on, because the compiler should have flagged this error.

edited Apr 5, 2019 at 18:08

answered Apr 5, 2019 at 17:56

Paul Ogilvie

25.3k4 gold badges25 silver badges43 bronze badges

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

Collectives™ on Stack Overflow

Why am I getting a segmentation fault in my code?

1 Answer 1

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related