mallocで配列を動的確保するときのコードの書き方

Question

####解決したいこと
下にあるソースコードですが，
x = (double *)malloc(sizeof(x) * 10);
y = (double *)malloc(sizeof(double) * 10);
でxの書き方は正しいのでしょうか？
windowsのPCではコンパイルは通るのですが，プログラムを実行すると無限ループのようになってしまい，終了しません．
Macだと期待通りの動作をしました．
エラーメッセージが表示されないので，動作しない理由がわからずつまづいています．

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

int main(int argc, const char * argv[]) {

    int i;
    double *x, *y;
    
    x = (double *)malloc(sizeof(x) * 10);
    y = (double *)malloc(sizeof(double) * 10);
    
    for (i=0; i<10; i++) {
        x[i] = i;
        y[i] = i;
        printf("i: %d, x: %.0lf, y; %.0lf\n", i, x[i], y[i]);
    }
    
    free(x);
    free(y);
    return 0;
}

Answer 1

xはポインタなので、sizeof(x)ではなくてsizeof(*x)ですね。

ポインタサイズとdoubleのサイズが同じ環境だとたまたまうまく動いて、異なる環境だと割り当て範囲外にアクセスしてしまいます。

Comments

[RaitoN]

回答ありがとうございます．スッキリできました．
とても助かります．
double x[10]
のようにdouble型の配列を10個確保する目的ならば,
x = (double *)malloc(sizeof(double) * 10);
が正しいのですね．

[pepperleaf]

32bit環境だと、 (sizeof(double) = 64) ≠ (double* = 32) でアウトみたいですね。

[otn]

8と4ですね。

[pepperleaf]

失礼、 sizeof だと バイト数でした....