Указатель, являющийся realloc'd, не был выделен, установите точку останова в malloc_error_break для отладки

Question

Я разрабатывал эту очень простую C-игру для Windows, где я использовал команду gcc matches.c -o ./matches для ее компиляции. Я импортировал код на своем Mac и перекомпилировал его, используя gcc и clang. Используя оба метода, программа полностью падает, иногда закрывая сеанс терминала, и выводит это.

matches(54122,0x1137715c0) malloc: *** error for object 0x7ffee9e8ba40: pointer being realloc'd was not allocated
matches(54122,0x1137715c0) malloc: *** set a breakpoint in malloc_error_break to debug
Abort trap: 6

Broadcast Message from _appleevents@(myname).local                                
        (no tty) at 20:33 CET...                                               

matches(54122,0x1137715c0) malloc: *** error for object 0x7ffee9e8ba40: pointer
 being realloc'd was not allocated

Код не содержит ошибок в Windows.

Я думаю, это связано с xcode или чем-то подобным. Кто-нибудь знает, как решить эту проблему?

Кстати, вот код. Сбой программы в функции настройки при getline()

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

void show_matches(int n);
void setup(int *numberOfPlayersA, int *gamemodeA, int *numberOfMatchesA);
void changePlayersTurn(int *currentPlayerA, int numberOfPlayers);
int random_move();

int main(void) {
  char *input;
  int currentPlayer = 1;
  size_t n = 0;
  int numberOfPlayers = 0;
  int gamemode = 0;
  int numberOfMatches = 0;
  int move = 0;

  setup(&numberOfPlayers, &gamemode, &numberOfMatches);
  show_matches(numberOfMatches);

  while (numberOfMatches >= 1) {
    printf("\033[1;31m");
    printf("\n\nPlayer %d> ", currentPlayer);
    printf("\033[0m");

    if (gamemode == 2 || currentPlayer == 1) {
      getline(&input, &n, stdin);
      move = atoi(input);

      if (move > 3 || move < 1 ) {
           move = 1;
      }
    } else {
      int randomMove = random_move();
      move = randomMove;
      printf("%d", randomMove);
    }



    numberOfMatches -= move;
    show_matches(numberOfMatches);


    if (numberOfMatches >= 1) {
      changePlayersTurn(&currentPlayer, numberOfPlayers);
    }
  }


  printf("\n\nPlayer %d lost\n\n", currentPlayer);

  return 0;
}

void setup(int *numberOfPlayersA, int *gamemodeA, int *numberOfMatchesA) {
  char *input;
  size_t n = 0;

  printf("--The matches--\n\n");
  printf("Do you plan on playing against:\n\t1. The computer\n\t2. Other persons\n\n(1 / 2) > ");
  getline(&input, &n, stdin);
  printf("1");
  *gamemodeA = atoi(input);
  printf("2");
  if (*gamemodeA == 2) {
    printf("\n\nPlease enter the number of players: ");
    getline(&input, &n, stdin);
    *numberOfPlayersA = atoi(input);
  }
  printf("Enter the number of matches: ");
  getline(&input, &n, stdin);
  *numberOfMatchesA = atoi(input);
  *numberOfPlayersA = 2;
  printf("4");
}

void changePlayersTurn(int *currentPlayerA, int numberOfPlayers) {
  if (*currentPlayerA == numberOfPlayers) {
    *currentPlayerA = 1;
  } else {
    *currentPlayerA += 1;
  }
}

void show_matches(int n) {
  for (int i = 0; i < n; i++) {
    printf("|");
  }
}

int random_move() {
  int num = (rand() % (3 - 1 + 1)) + 1;
  return num;
}

Answer 1

In main :

 char *input;
 ...
 getline(&input, &n, stdin);

вы вызываете getline без инициализации input , если (неопределенное) значение не равно NULL getline освободит его и потому что это не адрес выделенный блок у вас есть ошибка

вам нужно, например:

  input = NULL;
  getline(&input, &n, stdin);
  move = atoi(input);

  if (move > 3 || move < 1 ) {
       move = 1;
  }
  free(input);

У вас есть похожая ошибка в setup :

char *input;
...
getline(&input, &n, stdin);
...
getline(&input, &n, stdin);

вам нужно установить input в NULL перед первым вызовом getline , и вам нужно free после последнего вызова, например:

char *input;
...
input = NULL;
getline(&input, &n, stdin);
...
getline(&input, &n, stdin);
*numberOfMatchesA = atoi(input);
free(input);

Дополнительные замечания:

• вам не хватает проверить результат getline
• использовать atoi небезопасно, вы получаете 0, если пользователь вводит что-то не так, лучше использовать strtod или sscanf
• в main вместо форсирования 1, когда значение ввода, если не между 1 и 3, кажется, лучше повторить ввод