メッセージの有無なんかよりも
動作するかどうか
の方がはるかに重要

メモリ確保が失敗する可能性があるところで
失敗が許されない動作に対してmallocを使うのが間違い