「東証の派生売買システムで障害、4時間取引停止」記事へのコメント

ちょっと違う

[Anonymous Coward]

>データ容量の上限を 2万銘柄とすべきところ、88銘柄に設定していた
データ容量のサイズが28000銘柄分のはずが88銘柄分しか確保できてなかった。

http://itpro.nikkeibp.co.jp/article/NEWS/20080722/311271/

Re:

[hylom (27448)]

なるほど。データのサイズを誤っていたのですね。しかし、1280バイトのはずのデータを、4バイトに間違えるってどんなミスだよ……。

板情報を配信するプログラムは本来、1銘柄当たり1280バイトの作業用メモリー領域を2万8000銘柄分、合計3万5000Kバイト確保するよう記述しなければならない。だが、1銘柄当たりのメモリー領域を誤って4バイトとしてしまったため、プログラムは本来の320分の1の109.375Kバイトしか確保しなかった。結果として89銘柄以上の板情報の問い合わせが同時に発生すると、作業用メモリーが足りなくなり、情報配信システムがダウンした。

（東証のシステム障害、設定ミスをテストでも見抜けず：ITpro [nikkeibp.co.jp]より）

Re:

[taka2 (14791)]

> 1280バイトのはずのデータを、4バイトに間違える

sizeof(DataType) のつもりが sizeof(DataType*) になってたとか?

…っていうのは、やっちゃったことあります。マクロ化してて、引数の型を間違えたりとか…
#define alloc(type,count) malloc(sizeof(type)*(count))
DataType *p = alloc(p, count);
ってな感じのコードだったかな。

Re:ちょっと違う

[Anonymous Coward]

そのマクロ定義だと、たしかに起きがちですね。
そういう間違いを防ぐため、以下のようなマクロを使うことにしてます。

#define MALLOC1(p) ((p) = malloc(sizeof(*(p))))
#define MALLOCN(p,n) ((p) = malloc_array((n), sizeof(*(p))))

なお関数 malloc_array() 内では、整数オーバーフローのチェックが必要です。
親記事の alloc() マクロは、その点でも問題がありますね。