Andreas Jaeger <[EMAIL PROTECTED]> writes:

| Gabriel Dos Reis <[EMAIL PROTECTED]> writes:
| 
| > Hi again,
| >
| > I just hit this one from tree-ssa-into.c:rewrite_into_ssa()
| >
| >   /* Initialize dominance frontier.  */
| >   dfs = (bitmap *) xmalloc (last_basic_block * sizeof (bitmap *));
| >                                                        ^^^^^^^^
| >                                                                 
| > If the sizeof operand really supposed to be "bitmap*" and not "bitmap"?
| 
| This indeed looks wrong - and could lead to real memory corruption
| :-(.  If this is wrong, he should be changed on the 4.0 branch as
| well,

And if it is wrong, it would be interesting to know wheter we already
have PRs related to that memory corruption -- but I guess it would be
hard to know :-)

Parenthetically, I was wondering who is freeing those extensive
regions of storage xmalloc/xcalloc()ed here and there?

-- Gaby

Reply via email to