(JP) tcache_create_column_store()内で確保したメモリの初期化が一部行われないことがある

[taiki-k]

#37 の調査中に発見しました。

tcache_create_column_store()で共有メモリからColumn Store用にメモリを確保していますが、
確保する際のサイズ(tcache.c:433)と、その直後に0で初期化する際のサイズ(tcache.c:436)が異なっていおり、
tcsのcdata以降の領域が初期化されことがあります。

直後のコードで具体的な値が代入されていくため、問題がないように見えましたが、
tcache.c:456からのfor文中のbms_is_member()がfalseを返却した場合、
cdataにゴミデータが残ったままとなってしまいます。

[kaigai]

確かに。tcache_column_store は可変長なので、offsetof(tcache_column_store, cdata[tupdesc->natts]) で初期化する必要がありますね。直しておきました。
ただ、length は tcs のヘッダおよびデータ領域を含むサイズ（数MB～数十MB）なので、これ全体をゼロクリアする必要はありません。

修正したのでクローズします。