code review 6782100: cmd/gc: check malloc return value

cmd/gc: check malloc return value

Check the return value from malloc - do not assume that we were
allocated memory just because we asked for it.

Update issue 4415.

[minux1, 2012-11-23 15:09:38]

found more unchecked mallocs:
https://codereview.appspot.com/6856080

[jsing, 2012-11-25 13:08:26]

Hello minux.ma@gmail.com (cc: golang-dev@googlegroups.com),

I'd like you to review this change to
https://go.googlecode.com/hg/

[DMorsing, 2012-11-25 14:19:33]

Why not just use mal()? That function already bails on allocation failure.

[remyoudompheng, 2012-11-25 14:23:00]

because this malloc has a corresponding free.

Rémy.

2012/11/25, daniel.morsing@gmail.com <daniel.morsing@gmail.com>:
> Why not just use mal()? That function already bails on allocation
> failure.
>
> https://codereview.appspot.com/6782100/
>

[rsc, 2012-11-25 16:21:30]

LGTM

[jsing, 2012-11-26 11:03:40]

*** Submitted as http://code.google.com/p/go/source/detail?r=3d70c606eb75 ***

cmd/gc: check malloc return value

Check the return value from malloc - do not assume that we were
allocated memory just because we asked for it.

Update issue 4415.

R=minux.ma, daniel.morsing, remyoudompheng, rsc
CC=golang-dev
http://codereview.appspot.com/6782100