「rsync 2.5.6 にバッファオーバーフローの脆弱性」記事へのコメント

分からん

[Anonymous Coward]

恥を承知で。

> あらら、これ rsync だけの問題で終わるのかしらん?

って言われても、パッチのコードがさっぱり読めん俺には何がまずいのかが分からんのですけど。
rsyncだけの問題で終わらんのですか？理由は？何で書かないの？

Re:分からん

[Anonymous Coward]

yoosee日記 [srad.jp]でフォローしてたのか....そんな所まで見に行かんっつー。

という訳で詳しい人おねがい。

Re:分からん

[Anonymous Coward]

疑問があるんなら記事本文に書かずにコメントで聞けてよバカ、って感じ。Linux の malloc() 系の関数に問題があるという話ではまったくないです。

Re:分からん

[yoosee (196)]

う、その通りですね。載せ方を明らかに間違ったと思います。
本文に書くにしても煽りっぽい書き方をする必要はないですね…。失礼しました。

> Linux の malloc() 系の関数に問題があるという話ではまったくないです。

ふーむ、単に場当たり的(と言っても必要十分)な対処をしただけ、ってことですか。
なんで MALLOC_MAX を (1GB程度の) 定数値で入る必要があるのかが気になったのと、realloc で !ptr をわざわざ見てるのが何故かと思ったんですが、コードの他の部分でそういう危なさげな状況が起こり得るのでその回避なのかな。

Re:分からん

[Anonymous Coward]

＞realloc で !ptr をわざわざ見てるのが何故かと思ったんですが

reallocじゃなくてrealloc_arrayですね？
単にreallocとの互換性のためでしょ。
reallocはヌルポインタ渡されたらmallocと同じ動作をする事になっているので。

Re:分からん

[T.Sawamoto (4142)]

昔痛い目に遭ったせいで、realloc前にポインタがNULLか判別する癖が今でも残ってる…。

# 実はfreeも(笑)