C言語でAbort trap:6 というエラーメッセージが出て対処法がわからず調べています。

Question

###前提・実現したいこと
C言語で、少数データを入力させ、負の少数が入力されたときに今まで入力された少数をまとめて表示するプログラムを作りたいです。reallocを用いて領域サイズを変更し、データの個数に上限は設けないものとします。

###発生している問題・エラーメッセージ

$./a.out
Input number:1.2
Input number:2.3
a.out(1501,0x7fffddcbf3c0) malloc: *** error for object 0x7ffd9b402738: pointer being realloc'd was not allocated
*** set a breakpoint in malloc_error_break to debug
Abort trap: 6

###該当のソースコード

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

int main(void)
{
    int i=1;
    double *a, *b, *c, x, *A;
    a = (double *)malloc(sizeof(double));
    
    if(a == NULL)
    {
        free(a);
        exit(1);
    }
    
    A = a;
    
    do
    {
        b = a;          /*現在のポインタを保存*/
        printf("Input number:");
        scanf("%lf", &x);
        *a = x;
        i++;
        c = (double *)realloc(a, sizeof(double)*i);
        if(c == NULL)
        {
            free(c);
            exit(1);
        }
        a = c;          /*cを用いて領域を確保*/
        a = b;          /*ポインタを戻す*/
        a++;
    }while(x >= 0);
        
    for(;i>0;i--)
    {
        printf("%f\n", *A);
        A++;
    }
    return 0;
}

###補足情報(言語/FW/ツール等のバージョンなど)
xcodeを使用しました。

Answer 1

こんにちは。

c = (double *)realloc(a, sizeof(double)*i);

を行うと、cとaが一致する保証はありません。
一致しなかった場合に何が起こっていたかと言うと、以下の通りです。

c = (double*)malloc(sizeof(double)*i);
memcpy(c, a, aの領域サイズ);
free(a);

したがって、

b = a; /現在のポインタを保存/
(中略)
aをrealloc
(中略)
a = b; /ポインタを戻す/

とすると、free(a)した時のaの値が回復されてしまいます。
つまり、ポインタを戻した後は、aは開放された領域を指してます。
その内容をアクセスすると不正アクセスになっても不思議なことではありません。

Answer 2

a ポインタを変更してしまっているので、 2 周目以降では malloc() で最初に確保した領域の末端から realloc() しようとしてしまい、「そんな領域を確保した覚えはない」と怒られているようです。
せっかく最初に確保した領域は A に保存しているので、 A から realloc() してみては如何でしょう。

Comments

[youryoukun]

ありがとうございます。
とてもわかりやすく、Aに書き換えるだけで動くようになりました！

[argparse]

ごめんなさい、 Chironian さんのご指摘が正しいですね。こちらの回答は誤りです。

Answer 3

以下のreallocのaをAに書き換えたら正常に動作しました。
最初にmallocしたポインタと変わってしまっていたんですね。
ありがとうございました。

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

int main(void)
{
    int i=1;
    double *a, *b, *c, x, *A;
    a = (double *)malloc(sizeof(double));
    
    if(a == NULL)
    {
        free(a);
        exit(1);
    }
    
    A = a;              /*スタート位置を保存*/
    
    do
    {
        b = a;          /*現在のポインタを保存*/
        printf("Input number:");
        scanf("%lf", &x);
        *a = x;
        i++;
        c = (double *)realloc(A, sizeof(double)*i);
        if(c == NULL)
        {
            free(c);
            exit(1);
        }
        a = c;          /*cを用いて領域を確保*/
        a = b;          /*ポインタを戻す*/
        a++;
    }while(x >= 0);
        
    for(;i>1;i--)
    {
        printf("%f\n", *A);
        A++;
    }
    return 0;
}